Skip to content

2415 - add json config model support for mmar loading#2416

Merged
wyli merged 2 commits intoProject-MONAI:devfrom
wyli:fixes-path-mmar
Jun 23, 2021
Merged

2415 - add json config model support for mmar loading#2416
wyli merged 2 commits intoProject-MONAI:devfrom
wyli:fixes-path-mmar

Conversation

@wyli
Copy link
Contributor

@wyli wyli commented Jun 22, 2021

Signed-off-by: Wenqi Li wenqil@nvidia.com

Fixes #2415

Description

adds support of model config from json

Status

Ready

Types of changes

  • Non-breaking change (fix or new feature that would not break existing functionality).
  • New tests added to cover the changes.
  • Quick tests passed locally by running ./runtests.sh --quick --unittests.
  • In-line docstrings updated.
  • Documentation updated, tested make html command in the docs/ folder.

@wyli wyli requested a review from Nic-Ma June 22, 2021 13:02
@wyli wyli changed the title 2415 - add json config model support 2415 - add json config model support for mmar loading Jun 22, 2021
@wyli wyli force-pushed the fixes-path-mmar branch 2 times, most recently from bfea4a5 to 6605c55 Compare June 22, 2021 21:54
Copy link
Contributor

@Nic-Ma Nic-Ma left a comment

Choose a reason for hiding this comment

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

Thanks for the enhancement, I put some minor comments.
Others look good to me.

Thanks.

Signed-off-by: Wenqi Li <wenqil@nvidia.com>
@wyli wyli force-pushed the fixes-path-mmar branch from 6605c55 to cfccabf Compare June 23, 2021 11:00
Signed-off-by: Wenqi Li <wenqil@nvidia.com>
@wyli wyli force-pushed the fixes-path-mmar branch from cfccabf to c2ebe78 Compare June 23, 2021 11:01
@wyli wyli requested a review from Nic-Ma June 23, 2021 11:02
Copy link
Contributor

@Nic-Ma Nic-Ma left a comment

Choose a reason for hiding this comment

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

Thanks for the quick update.
Looks good to me.

@wyli wyli merged commit ea028f5 into Project-MONAI:dev Jun 23, 2021
@wyli wyli deleted the fixes-path-mmar branch June 23, 2021 13:33
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.

load_from_mmar to support model config from the mmar json

2 participants