Skip to content

Workflow Fix#4944

Merged
ericspod merged 6 commits intoProject-MONAI:devfrom
ericspod:workflow-fix
Aug 20, 2022
Merged

Workflow Fix#4944
ericspod merged 6 commits intoProject-MONAI:devfrom
ericspod:workflow-fix

Conversation

@ericspod
Copy link
Copy Markdown
Member

@ericspod ericspod commented Aug 19, 2022

Description

Very minor fix to how workflows decide what the best metric result is such that the first epoch always chooses whatever its metric was. BaseWorkflow was also removed since it wasn't used anywhere, this can be put back if somewhere else needs it.

Status

Ready

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.

@ericspod ericspod requested review from Nic-Ma and wyli August 19, 2022 17:35
@ericspod ericspod enabled auto-merge (squash) August 20, 2022 10:42
@wyli
Copy link
Copy Markdown
Contributor

wyli commented Aug 20, 2022

/build

@ericspod ericspod merged commit 1941283 into Project-MONAI:dev Aug 20, 2022
@Nic-Ma
Copy link
Copy Markdown
Contributor

Nic-Ma commented Aug 20, 2022

Hi @ericspod ,

The BaseWorkflow is useful in the NVIDIA Clara Train project to ask developers to follow the signature if they want to develop their own workflow.

Thanks.

@ericspod
Copy link
Copy Markdown
Member Author

Could that be put into the clara train package then? If it's not used in MONAI I would say it shouldn't be here, but I can put it back in another PR.

@ericspod ericspod deleted the workflow-fix branch August 20, 2022 15:45
@Nic-Ma
Copy link
Copy Markdown
Contributor

Nic-Ma commented Aug 21, 2022

Hi @ericspod ,

I think we are fine now, Clara Train supposes to work with fixed MONAI version.

Thanks.

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