Skip to content

6371 enhance warning messages when modifying applied operations#6372

Merged
wyli merged 7 commits intoProject-MONAI:devfrom
wyli:fixes-invert-lazy
Apr 18, 2023
Merged

6371 enhance warning messages when modifying applied operations#6372
wyli merged 7 commits intoProject-MONAI:devfrom
wyli:fixes-invert-lazy

Conversation

@wyli
Copy link
Copy Markdown
Contributor

@wyli wyli commented Apr 16, 2023

Fixes #6371

raise a warning message early when the applied_operations is modified, this only becomes a critical issue when we actually use the applied_operations (InvertD is one of the use cases).

fixes #6390
fixes #6392

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.

Signed-off-by: Wenqi Li <[email protected]>
Signed-off-by: Wenqi Li <[email protected]>
@wyli
Copy link
Copy Markdown
Contributor Author

wyli commented Apr 17, 2023

/build

@wyli
Copy link
Copy Markdown
Contributor Author

wyli commented Apr 18, 2023

/integration-test

wyli added 3 commits April 18, 2023 19:14
Signed-off-by: Wenqi Li <[email protected]>
Signed-off-by: Wenqi Li <[email protected]>
Signed-off-by: Wenqi Li <[email protected]>
@wyli
Copy link
Copy Markdown
Contributor Author

wyli commented Apr 18, 2023

/build

@wyli wyli enabled auto-merge (squash) April 18, 2023 19:17
@wyli wyli merged commit c30a5b9 into Project-MONAI:dev Apr 18, 2023
@wyli wyli deleted the fixes-invert-lazy branch April 20, 2023 22:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

2 participants