pull_requests: 408964137
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 |
---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
408964137 | MDExOlB1bGxSZXF1ZXN0NDA4OTY0MTM3 | 4007 | open | 0 | Allow DataArray.to_series() without invoking sparse.COO.todense() | 1634164 | This adds some code (from iiasa/ixmp#317) that allows DataArray.to_series() to be called without invoking sparse.COO.todense() when that is the backing data type. I'm aware this needs some improvement to meet the standard of the existing codebase, so I hope I could ask for some guidance on how to address the following points (including whom to ask about them): - [ ] Make the same improvement in {DataArray,Dataset}.to_dataframe(). - [ ] Possibly move the code out of dataarray.py to a more appropriate location (where?). - [ ] Possibly check for sparse.COO explicitly instead of xarray.core.pycompat.sparse_array_type. Other SparseArray subclasses, e.g. DOK, may not have the same attributes. Standard items: - [ ] Tests added. - [x] Passes `isort -rc . && black . && mypy . && flake8` (Sort of: these wanted to modify 7 files beyond the one I touched; didn't commit these changes.) - [ ] Fully documented, including `whats-new.rst` for all changes and `api.rst` for new API. | 2020-04-25T20:15:16Z | 2022-06-09T14:50:17Z | 97367f000d06c906a778dfd2868cda39d937e87d | 0 | b5581fcd49264992acf53e05ea17957f7f2be61b | d1e4164f3961d7bbb3eb79037e96cae14f7182f8 | FIRST_TIME_CONTRIBUTOR | 13221727 | https://github.com/pydata/xarray/pull/4007 |
Links from other tables
- 0 rows from pull_requests_id in labels_pull_requests