Skip to content

Extend PrepareBatchExtraInput to work with HoVerNet#5448

Merged
Nic-Ma merged 7 commits intoProject-MONAI:devfrom
KumoLiu:prepare-batch-hovernet
Nov 3, 2022
Merged

Extend PrepareBatchExtraInput to work with HoVerNet#5448
Nic-Ma merged 7 commits intoProject-MONAI:devfrom
KumoLiu:prepare-batch-hovernet

Conversation

@KumoLiu
Copy link
Copy Markdown
Contributor

@KumoLiu KumoLiu commented Nov 2, 2022

Signed-off-by: KumoLiu [email protected]

Fixes #5028.

Description

Since the output for HoVerNet is a dictionary, we already have PrepareBatchExtraInput to support extra input data for the network, but it still can't meet the requirements. This PR is to extend PrepareBatchExtraInput to make the label can be a dictionary.

Types of changes

  • Non-breaking change (fix or new feature that would not break existing functionality).
  • 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: KumoLiu <[email protected]>
@KumoLiu KumoLiu requested review from Nic-Ma, bhashemian and wyli November 2, 2022 10:46
Signed-off-by: KumoLiu <[email protected]>
Signed-off-by: KumoLiu <[email protected]>
@Nic-Ma
Copy link
Copy Markdown
Contributor

Nic-Ma commented Nov 3, 2022

/black

@Nic-Ma
Copy link
Copy Markdown
Contributor

Nic-Ma commented Nov 3, 2022

/build

@Nic-Ma Nic-Ma enabled auto-merge (squash) November 3, 2022 08:09
@wyli
Copy link
Copy Markdown
Contributor

wyli commented Nov 3, 2022

/build

1 similar comment
@wyli
Copy link
Copy Markdown
Contributor

wyli commented Nov 3, 2022

/build

@Nic-Ma Nic-Ma merged commit c38d503 into Project-MONAI:dev Nov 3, 2022
@KumoLiu KumoLiu deleted the prepare-batch-hovernet branch November 8, 2022 02:43
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.

Make HoVerNet work SuperviseTrainer

4 participants