pull_requests: 712223776
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 |
|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
| 712223776 | MDExOlB1bGxSZXF1ZXN0NzEyMjIzNzc2 | 5704 | open | 0 | Allow in-memory arrays with open_mfdataset | 14371165 | The docstring seems to imply that it's possible to get in-memory arrays: https://github.com/pydata/xarray/blob/4bb9d9c6df77137f05e85c7cc6508fe7a93dc0e4/xarray/backends/api.py#L732 But it doesn't seem possible because of: https://github.com/pydata/xarray/blob/4bb9d9c6df77137f05e85c7cc6508fe7a93dc0e4/xarray/backends/api.py#L899 This PR removes that `or` check, changes the default to `chunk={}`, and fixes the failing tests. <!-- Feel free to remove check-list items aren't relevant to your change --> - [x] Noticed in #5689 - [ ] Closes #7792 - [x] Tests added - [x] Passes `pre-commit run --all-files` - [x] User visible changes (including notable bug fixes) are documented in `whats-new.rst` | 2021-08-13T09:50:26Z | 2023-04-29T06:58:26Z | 2c505d513d75bc6c1c30854a5860343dd5daabe8 | 0 | 34442811c68fc38ddcda2370eb7b00bb943af73e | d1e4164f3961d7bbb3eb79037e96cae14f7182f8 | MEMBER | 13221727 | https://github.com/pydata/xarray/pull/5704 |
Links from other tables
- 1 row from pull_requests_id in labels_pull_requests