Skip to content

2355 parameter groups#2356

Merged
wyli merged 1 commit intoProject-MONAI:devfrom
wyli:2355-param-groups
Jun 11, 2021
Merged

2355 parameter groups#2356
wyli merged 1 commit intoProject-MONAI:devfrom
wyli:2355-param-groups

Conversation

@wyli
Copy link
Contributor

@wyli wyli commented Jun 10, 2021

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

Fixes #2355

Description

params groups requires parameters() and is not compatible with named_parameters(),
this PR fixes the issue by adding a map

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.

Signed-off-by: Wenqi Li <wenqil@nvidia.com>
@wyli wyli merged commit 0b599ef into Project-MONAI:dev Jun 11, 2021
@wyli wyli deleted the 2355-param-groups branch June 11, 2021 00:30
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.

paramer group format error

2 participants