pull_requests: 488908129
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 |
|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
| 488908129 | MDExOlB1bGxSZXF1ZXN0NDg4OTA4MTI5 | 4432 | closed | 0 | Fix optimize for chunked DataArray | 1312546 | Previously we generated in invalidate Dask task graph, becuase the lines removed here dropped keys that were referenced elsewhere in the task graph. The original implementation had a comment indicating that this was to cull: https://github.com/pydata/xarray/blob/502a988ad5b87b9f3aeec3033bf55c71272e1053/xarray/core/variable.py#L384 Just spot-checking things, I think we're OK here though. Something like `dask.visualize(arr[[0]], optimize_graph=True)` indicates that we're OK. <!-- Feel free to remove check-list items aren't relevant to your change --> - [x] Closes #3698 - [x] Tests added - [x] Passes `isort . && black . && mypy . && flake8` - [x] User visible changes (including notable bug fixes) are documented in `whats-new.rst` - [x] New functions/methods are listed in `api.rst` | 2020-09-17T20:16:08Z | 2020-09-18T13:20:45Z | 2020-09-17T23:19:23Z | 2020-09-17T23:19:23Z | 9a8a62ba551e737dc87e39aded2f7cc788ff118d | 0 | 381aaf8cc37502907506011f8cb9f4149e229d2d | b0d8d93665dbb6d28e33dfd28ad27036c20c60bf | MEMBER | 13221727 | https://github.com/pydata/xarray/pull/4432 |
Links from other tables
- 0 rows from pull_requests_id in labels_pull_requests