-
Notifications
You must be signed in to change notification settings - Fork 5
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
Duplicate model entries #47
Comments
@lmoyasans @esgomezm we should delete the models from here when possible and upload them through the bioimage.io uploader. |
Hi there, this has been done already as soon as the models were correctly uploaded (there are many open pull requests) such as the cases of FRUNEt, ZCDL4M unets, pancreatic cells etc. which have already disappeared from the main manifest yaml of deepImageJ. Still when removing those models from the yaml, please, for our internal record, keep them updated here https://github.com/deepimagej/models/blob/master/manifest.bioimageio_dij_install_model.yaml. |
I have created a list with the errors that I have seen in each of the models. A couple of the DIJ ones have errors coming from the zipped model, as it is not up to date with the model defined in the model card. If we keep the redundancy of the zipped model, it is important to make sure they are fine as it could be a source of errors |
Some models, e.g.
are linked to the bioimage.io collection via https://github.com/deepimagej/models/blob/master/manifest.bioimage.io.yaml
and a zenodo record.
As discussed previously we should prefer the zenodo records to host models and remove the entries in this manifest here.
The text was updated successfully, but these errors were encountered: