Skip to content

Remove deprecated args and class for v1.4 - #8079

Merged
KumoLiu merged 5 commits into
Project-MONAI:devfrom
KumoLiu:deprecated
Sep 16, 2024
Merged

KumoLiu merged 5 commits into
Project-MONAI:devfrom
KumoLiu:deprecated

Conversation

@KumoLiu

@KumoLiu KumoLiu commented Sep 11, 2024

Copy link
Copy Markdown
Contributor

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: YunLiu <55491388+KumoLiu@users.noreply.github.com>
@KumoLiu
KumoLiu requested review from Nic-Ma and ericspod September 11, 2024 10:33
Signed-off-by: YunLiu <55491388+KumoLiu@users.noreply.github.com>
@KumoLiu

KumoLiu commented Sep 11, 2024

Copy link
Copy Markdown
Contributor Author

/build

@RahulVadisetty91 RahulVadisetty91 left a comment •

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

0999k

Correct or remove the last line of the docstring as the default is None

@KumoLiu

KumoLiu commented Sep 16, 2024

Copy link
Copy Markdown
Contributor Author

/build

2 similar comments
@KumoLiu

KumoLiu commented Sep 16, 2024

Copy link
Copy Markdown
Contributor Author

/build

@KumoLiu

KumoLiu commented Sep 16, 2024

Copy link
Copy Markdown
Contributor Author

/build

@KumoLiu
KumoLiu merged commit 3282e45 into Project-MONAI:dev Sep 16, 2024
@KumoLiu
KumoLiu deleted the deprecated branch September 16, 2024 04:48
@KumoLiu KumoLiu mentioned this pull request Sep 25, 2024
31 tasks done
@KumoLiu KumoLiu mentioned this pull request Apr 11, 2025
28 of 31 tasks
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.

3 participants