id,node_id,number,title,user,state,locked,assignee,milestone,comments,created_at,updated_at,closed_at,author_association,active_lock_reason,draft,pull_request,body,reactions,performed_via_github_app,state_reason,repo,type
1048697792,PR_kwDOAMm_X84uSksS,5961,[Experimental] Refactor Dataset to store variables in a manifest,35968931,closed,0,,,7,2021-11-09T14:51:03Z,2023-12-06T17:38:53Z,2023-12-06T17:38:52Z,MEMBER,,0,pydata/xarray/pulls/5961,"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`","{""url"": ""https://api.github.com/repos/pydata/xarray/issues/5961/reactions"", ""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,,13221727,pull