pull_requests: 705713175
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 |
|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
| 705713175 | MDExOlB1bGxSZXF1ZXN0NzA1NzEzMTc1 | 5680 | open | 0 | ENH: Add default fill values for decode_cf | 44147817 | - [x] Closes #2374 - [x] Tests added - [x] Passes `pre-commit run --all-files` - [ ] User visible changes (including notable bug fixes) are documented in `whats-new.rst` This is a work in progress, mostly so that I can ask some clarifying questions. I see that `netCDF4` is an optional dependency for `xarray`, so probably `import netCDF4` can't be used. Should `xarray` simply hard-code default fill values ? From the issue's conversation, it wasn't clear to me whether an argument should control the use of the default fill value. Since some tests fail now I guess the answer is yes. | 2021-08-06T19:54:05Z | 2022-06-09T14:50:16Z | f46be84bd01a42419a8936df8d8e7f97d2151ab4 | 0 | 26f1b3224c5c7777765bf37999b700c34ff9ee04 | d1e4164f3961d7bbb3eb79037e96cae14f7182f8 | CONTRIBUTOR | 13221727 | https://github.com/pydata/xarray/pull/5680 |
Links from other tables
- 0 rows from pull_requests_id in labels_pull_requests