Update Vista3d eval and add zero shot descriptions
Fixes # .
Description
A few sentences describing the changes proposed in this pull request.
Status
Ready/Work in progress/Hold
Please ensure all the checkboxes:
- [x] Codeformat tests passed locally by running
./runtests.sh --codeformat. - [ ] In-line docstrings updated.
- [ ] Update
versionandchangeloginmetadata.jsonif changing an existing bundle. - [ ] Please ensure the naming rules in config files meet our requirements (please refer to:
CONTRIBUTING.md). - [ ] Ensure versions of packages such as
monai,pytorchandnumpyare correct inmetadata.json. - [ ] Descriptions should be consistent with the content, such as
eval_metricsof the provided weights and TorchScript modules. - [ ] Files larger than 25MB are excluded and replaced by providing download links in
large_file.yml. - [ ] Avoid using path that contains personal information within config files (such as use
/home/your_name/for"bundle_root").
Thanks for the update, looks good to me. I have another question regarding the extra metric in the training config, seems it'n not used, do we need include it here? Thanks. https://github.com/Project-MONAI/model-zoo/blob/dev/models/vista3d/configs/train.json#L355C10-L355C28 https://github.com/Project-MONAI/model-zoo/blob/dev/models/vista3d/configs/train.json#L369
Hi @KumoLiu , thanks for the question. There are some configs that are not used, like additional_metrics as you mentioned. I'm not sure why we include it in the config file. Hi @heyufan1995 , could you provide some information here? Thanks!
Thanks for the update, looks good to me. I have another question regarding the extra metric in the training config, seems it'n not used, do we need include it here? Thanks. https://github.com/Project-MONAI/model-zoo/blob/dev/models/vista3d/configs/train.json#L355C10-L355C28 https://github.com/Project-MONAI/model-zoo/blob/dev/models/vista3d/configs/train.json#L369
Hi @KumoLiu , thanks for the question. There are some configs that are not used, like
additional_metricsas you mentioned. I'm not sure why we include it in the config file. Hi @heyufan1995 , could you provide some information here? Thanks!
Seems like it was not used. It was created by Wenqi @wyli
/build