Skip to content
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

How to avoid inference script duplication #36

Open
hermancollin opened this issue Sep 20, 2023 · 1 comment
Open

How to avoid inference script duplication #36

hermancollin opened this issue Sep 20, 2023 · 1 comment

Comments

@hermancollin
Copy link
Contributor

hermancollin commented Sep 20, 2023

This was briefly mentioned during the last ivadomed meeting.

The inference script is a nice wrapper for nnUNet. It's hard to see how we could avoid duplicating it in model repositories, but it would be nice to include it because it makes these repos usable in a standalone fashion. 3rd parties can still access the model checkpoint by simply downloading the release asset, but providing an inference script is great for external users. Also, nnUNet is not yet supported in ADS but in the meantime I need a way to share models with collaborators. What happens it that we get variations of the script everywhere, i.e.

A way to centralize everything and to have only 1 script to maintain would be to actually package this small wrapper and list it as a requirement in model repositories. There's even already a requirements file!

@valosekj
Copy link
Member

valosekj commented Oct 3, 2023

Relevant: #34

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

No branches or pull requests

2 participants