pull_requests: 776620818
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 |
|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
| 776620818 | PR_kwDOAMm_X84uSksS | 5961 | closed | 0 | [Experimental] Refactor Dataset to store variables in a manifest | 35968931 | This PR is part of an experiment to see how to integrate a `DataTree` into xarray. What is does is refactor `Dataset` to store variables in a `DataManifest` class, which is also capable of maintaining a ledger of child tree nodes. The point of this is to prevent name collisions between stored variables and child datatree nodes, as first mentioned in https://github.com/TomNicholas/datatree/issues/38 and explained further in https://github.com/TomNicholas/datatree/issues/2. ("Manifest" in the old sense, of a noun meaning "a document giving comprehensive details of a ship and its cargo and other contents") - [x] Would eventually close https://github.com/TomNicholas/datatree/issues/38 - [ ] Tests added - [x] Passes `pre-commit run --all-files` - [ ] User visible changes (including notable bug fixes) are documented in `whats-new.rst` | 2021-11-09T14:51:03Z | 2023-12-06T17:38:53Z | 2023-12-06T17:38:52Z | 0c4c6baf99c63a92355fefe944afdd84c1c78078 | 0 | 183b92be0c55e26e28913c06c7d9b249572aa207 | d1e4164f3961d7bbb3eb79037e96cae14f7182f8 | MEMBER | 13221727 | https://github.com/pydata/xarray/pull/5961 |
Links from other tables
- 2 rows from pull_requests_id in labels_pull_requests