New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
3482 Add ConfigComponent
for config parsing
#3720
Conversation
merge master
merge master
merge master
merge master
merge master
Signed-off-by: Nic Ma <nma@nvidia.com>
/black |
/build |
I tried to cover all the possible config cases in the unit tests. Thanks. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
(Still under discussion)
ConfigComponent
for config parsingConfigComponent
for config parsing
/build |
ConfigComponent
for config parsingConfigComponent
for config parsing
/build |
I'm still looking into the existing implementations outside of monai... for this pr,
|
Hydra does do some of the things we want to include here, do we want to add it as a dependency? I'm not sure it could be an optional dependency since the model archiving functionality would need it to be there and it's a pretty important that it's present in MONAI. |
this one also looks interesting https://github.com/jmespath/jmespath.py |
Something structured like this looks like a good idea, I do like the "foo.baz" syntax. It's rather more complex that we need for this application perhaps, again though the concern is adding another dependency. I have a bit of code in a tool for generating random data that instantiates objects from a YAML schema. This function to creates objects from the dictionary derived from the schema recursively so that new objects can be used as constructor arguments when instantiating others. The find_type_def is pretty well the same as I'm good with this going forward if we want to keep the momentum going on the model archive. |
ConfigComponent
for config parsingConfigComponent
for config parsing
ConfigComponent
for config parsingConfigComponent
for config parsing
/black |
/build |
Signed-off-by: Nic Ma <nma@nvidia.com>
e92a3ee
to
aa5f95d
Compare
/build |
1 similar comment
/build |
Signed-off-by: Wenqi Li <wenqil@nvidia.com>
Signed-off-by: Wenqi Li <wenqil@nvidia.com>
/build |
Signed-off-by: Wenqi Li <wenqil@nvidia.com>
7804756
to
3ac8827
Compare
now simplified to the essential components
/black |
/build |
1 similar comment
/build |
Signed-off-by: Nic Ma <nma@nvidia.com>
6e53dcd
to
6d0aa5d
Compare
/build |
2 similar comments
/build |
/build |
Signed-off-by: Nic Ma <nma@nvidia.com>
/black |
/build |
1 similar comment
/build |
I have merged this PR into the whole draft PR: #3593 and verified the overall features. Thanks. |
Task step 2 of #3482 .
Description
This PR implemented the
ConfigComponent
feature for config parsing, it's for task 2 of #3482 .The whole proposal is in draft PR: #3593.
Status
Ready
Types of changes
./runtests.sh -f -u --net --coverage
../runtests.sh --quick --unittests --disttests
.make html
command in thedocs/
folder.