pull_requests: 90853585
This data as json
| id | node_id | number | state | locked | title | user | body | created_at | updated_at | closed_at | merged_at | merge_commit_sha | assignee | milestone | draft | head | base | author_association | auto_merge | repo | url | merged_by |
|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
| 90853585 | MDExOlB1bGxSZXF1ZXN0OTA4NTM1ODU= | 1060 | closed | 0 | Remove obsolete NetCDF4 Error catch | 10050469 | [edit: this original PR was replaced with another change: "remove obsolete NetCDF4 Error catch"] This allows to give a customized NetCDF4 object to the DataStore. I needed this for Salem's diagnostic variables: https://github.com/fmaussion/salem/blob/master/salem/sio.py#L737-L751 My workaround (a very [shallow](https://github.com/fmaussion/salem/blob/master/salem/sio.py#L653) subclass of NetCDF4DataStore) is fine for me, so I won't be offended if you decide not to merge. On a related issue, the IndexError catch which happens [here](https://github.com/pydata/xarray/blob/master/xarray/backends/netCDF4_.py#L52) is also giving me trouble, since it hides bugs in my code. I know I'm kind of misusing the backend, but I find it very useful the way it is. I'm open to any suggestion to make it more elegant. Thanks! | 2016-10-25T17:25:33Z | 2017-05-22T08:58:21Z | 2017-05-22T08:58:21Z | 1ee3bb85cc5489a1df52e17ad690cd992590b8a7 | 0 | f24e8805b77682a21d086c6e28a8041b79c035b2 | a4f5ec2263ee96059f87daec4c217d425ca94e51 | MEMBER | 13221727 | https://github.com/pydata/xarray/pull/1060 |
Links from other tables
- 0 rows from pull_requests_id in labels_pull_requests