Skip to content
This repository has been archived by the owner on Oct 24, 2024. It is now read-only.

Name permanence via shallow copy #172

Merged

Conversation

TomNicholas
Copy link
Member

Closes #116 using this suggestion. Builds on #171.

  • Closes Name permanence #116
  • Tests added
  • Passes pre-commit run --all-files
  • New functions/methods are listed in api.rst
  • Changes are summarized in docs/source/whats-new.rst

@TomNicholas TomNicholas added the enhancement New feature or request label Dec 30, 2022
@TomNicholas TomNicholas enabled auto-merge (squash) December 31, 2022 00:27
@TomNicholas TomNicholas merged commit f2fbf44 into xarray-contrib:main Dec 31, 2022
@TomNicholas TomNicholas deleted the name_permanence_via_shallow_copy branch December 31, 2022 00:34
flamingbear pushed a commit to flamingbear/rewritten-datatree that referenced this pull request Jan 19, 2024
* added descendants property

* tests for descendants, lineage, ancestors, subtree

* added descendants to API docs

* whatsnew

* rerun tests

* rewrote copy method

* remove outdated mypy ignore error

* changed tests to reflect new expected behaviour

* shallow copy on insertion

* update test for checking isomorphism from root

* whatsnew
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Name permanence
1 participant