Skip to content

Make MONAI robust to non-default meta dict names #8411

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Draft
wants to merge 2 commits into
base: dev
Choose a base branch
from

Conversation

atbenmurray
Copy link
Contributor

@atbenmurray atbenmurray commented Apr 4, 2025

This PR adds a field to the meta dict indicating which tensor it belongs to, and a method get_meta_dict_key to find the meta_dict when it is present in a dictionary.

I have addressed many of the obvious places where there might be issues if the default meta_dict extension is not used but there are likely more that I need to find.

Description

A few sentences describing the changes proposed in this pull request.

Types of changes

  • Non-breaking change (fix or new feature that would not break existing functionality).
  • Breaking change (fix or new feature that would cause existing functionality to change).
  • New tests added to cover the changes.
  • Integration tests passed locally by running ./runtests.sh -f -u --net --coverage.
  • Quick tests passed locally by running ./runtests.sh --quick --unittests --disttests.
  • In-line docstrings updated.
  • Documentation updated, tested make html command in the docs/ folder.

…ct names

Signed-off-by: Ben Murray <atbenmurray@github.com>
@atbenmurray atbenmurray changed the title Initial commit to make codebase robust to use of non-standard meta_dict names Make MONAI robust to non-default meta dict names Apr 4, 2025
Signed-off-by: Ben Murray <atbenmurray@github.com>
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant