-
Notifications
You must be signed in to change notification settings - Fork 85
Update deps for monai bundle download #762
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
base: dev
Are you sure you want to change the base?
Conversation
Signed-off-by: Suraj Pai <[email protected]>
models/vista3d/docs/README.md
Outdated
| # Installation Guide | ||
| ``` | ||
| pip install "monai[fire]" | ||
| pip install "monai[fire]" requests huggingface_hub |
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.
Hi @surajpaib , thanks for the contribution.
Since the bundle content is changed, can you also update the metadata.json with a new version and changelog?
https://github.com/Project-MONAI/model-zoo/blob/dev/models/vista3d/configs/metadata.json#L5
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.
This is done now!
Signed-off-by: Suraj Pai <[email protected]>
Signed-off-by: Suraj Pai <[email protected]>
models/vista3d/configs/metadata.json
Outdated
| @@ -1,7 +1,8 @@ | |||
| { | |||
| "schema": "https://github.com/Project-MONAI/MONAI-extra-test-data/releases/download/0.8.1/meta_schema_20240725.json", | |||
| "version": "0.5.10", | |||
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.
HI @surajpaib here also need to be changed to 0.5.11
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.
@yiheng-wang-nv Done!
Signed-off-by: Suraj Pai <[email protected]>
|
/build |
| # Installation Guide | ||
| ``` | ||
| pip install "monai[fire]" | ||
| pip install "monai[fire,ignite]" requests huggingface_hub |
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.
These added requirements can be mentioned in required_packages_version in the metadata file as well.
|
@yiheng-wang-nv please review again when you can and approve if we're good, @KumoLiu may be needed to trigger blossom if it's not working for you. Thanks! |
Description
Update deps for monai bundle download
monai bundle download commands fails without requests and huggingface_hub