Skip to content

Fixed issues in nnunetv2runner#6470

Merged
wyli merged 16 commits intoProject-MONAI:devfrom
dongyang0122:fix_nnunetv2runner
May 7, 2023
Merged

Fixed issues in nnunetv2runner#6470
wyli merged 16 commits intoProject-MONAI:devfrom
dongyang0122:fix_nnunetv2runner

Conversation

@dongyang0122
Copy link
Copy Markdown
Collaborator

Description

Fixed issues in nnunetv2runner

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: dongy <dongy@nvidia.com>
@dongyang0122 dongyang0122 requested review from mingxin-zheng and wyli May 5, 2023 06:38
@dongyang0122 dongyang0122 self-assigned this May 5, 2023
Comment thread monai/apps/nnunet/nnunetv2_runner.py Outdated
Comment thread monai/apps/nnunet/nnunetv2_runner.py
Comment thread monai/apps/nnunet/nnunetv2_runner.py
Comment thread monai/apps/nnunet/nnunetv2_runner.py Outdated
Comment thread monai/apps/nnunet/nnunetv2_runner.py Outdated
Comment thread monai/apps/nnunet/nnunetv2_runner.py Outdated
dongy and others added 13 commits May 5, 2023 08:10
Signed-off-by: dongy <dongy@nvidia.com>
Signed-off-by: dongy <dongy@nvidia.com>
Signed-off-by: dongy <dongy@nvidia.com>
Signed-off-by: dongy <dongy@nvidia.com>
Signed-off-by: dongy <dongy@nvidia.com>
Signed-off-by: dongy <dongy@nvidia.com>
Signed-off-by: dongy <dongy@nvidia.com>
Signed-off-by: dongy <dongy@nvidia.com>
Signed-off-by: dongy <dongy@nvidia.com>
Signed-off-by: dongy <dongy@nvidia.com>
Signed-off-by: dongy <dongy@nvidia.com>
Comment thread monai/apps/nnunet/nnunetv2_runner.py
@mingxin-zheng
Copy link
Copy Markdown
Contributor

Hi @wyli , can you please also take a look in the updated APIs for nnunet? Thanks!

@wyli
Copy link
Copy Markdown
Contributor

wyli commented May 6, 2023

/build

@wyli wyli enabled auto-merge (squash) May 6, 2023 17:10
@mingxin-zheng
Copy link
Copy Markdown
Contributor

@dongyang0122 there is a flake8 issue. Can you fix that thanks!

Signed-off-by: Mingxin Zheng <18563433+mingxin-zheng@users.noreply.github.com>
@wyli
Copy link
Copy Markdown
Contributor

wyli commented May 7, 2023

/build

@wyli wyli merged commit ee90481 into Project-MONAI:dev May 7, 2023
wyli pushed a commit that referenced this pull request May 10, 2023
Fixes #6496 .

### Description

Fix the test after nnunet API change in #6470 

### Types of changes
<!--- Put an `x` in all the boxes that apply, and remove the not
applicable items -->
- [x] Non-breaking change (fix or new feature that would not break
existing functionality).
- [x] Integration tests passed locally by running `./runtests.sh -f -u
--net --coverage`.
- [x] In-line docstrings updated.

---------

Signed-off-by: Mingxin Zheng <18563433+mingxin-zheng@users.noreply.github.com>
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