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

🔊 Warn user when a model is reused for the the latent task #93

Open
enryH opened this issue May 16, 2024 · 1 comment
Open

🔊 Warn user when a model is reused for the the latent task #93

enryH opened this issue May 16, 2024 · 1 comment

Comments

@enryH
Copy link
Member

enryH commented May 16, 2024

In case of typos in configurations, the analyze latent reloads a model and finds the wrong dimensions. Should it be possible to always retrain?

Definitely log that a pre-trained model was found.

@ri-heme
Copy link
Collaborator

ri-heme commented May 29, 2024

Definitely log that a pre-trained model was found.

If I'm not mistaken, this already happens. In the log file, the message: "Re-loading model" changes to "Training model" depending on whether a pre-trained model is being used. But agree that this should be explained more clearly. Perhaps also adding it as a warning? Or checking that the weights/dimensions match and the model can be ran without issue?

Should it be possible to always retrain?

The idea behind not doing this is to prevent an old model from being erased/overwritten, and also save time re-running the task if you want to just (re)generate the plots.

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