Skip to content

Fix copy/deepcopy for AsdfFile - #2133

Merged
sydduckworth merged 3 commits into
asdf-format:mainfrom
sydduckworth:asdf-file-deepcopy
Sep 4, 2026
Merged

Fix copy/deepcopy for AsdfFile#2133
sydduckworth merged 3 commits into
asdf-format:mainfrom
sydduckworth:asdf-file-deepcopy

Conversation

@sydduckworth

@sydduckworth sydduckworth commented Sep 4, 2026

Copy link
Copy Markdown
Member

Description

Closes #2103

  • Fixed copy.deepcopy not working with AsdfFile
  • Fixed AsdfFile.copy not working for files with ndarrays when lazy_tree=True

AI Disclosure

No AI tools used

Tasks

  • run prek on your machine
  • run pytest on your machine
  • Does this PR add new features and / or change user-facing code / API? (if not, label with no-changelog-entry-needed)
    • write news fragment(s) in changes/: echo "changed something" > changes/<PR#>.<changetype>.rst (see below for change types)
    • update relevant docstrings and / or docs/ page
    • for any new features, add unit tests
news fragment change types...
  • changes/<PR#>.feature.rst: new feature
  • changes/<PR#>.bugfix.rst: bug fix
  • changes/<PR#>.doc.rst: documentation change
  • changes/<PR#>.removal.rst: deprecation or removal of public API
  • changes/<PR#>.general.rst: infrastructure or miscellaneous change

@sydduckworth
sydduckworth requested a review from a team as a code owner September 4, 2026 16:50

@braingram braingram left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

@sydduckworth
sydduckworth merged commit ea93570 into asdf-format:main Sep 4, 2026
29 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

AsdfFile instances cannot be deepcopied

2 participants