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

Remove previous instance of trained model when re-training #32

Open
TarekHC opened this issue May 4, 2021 · 2 comments
Open

Remove previous instance of trained model when re-training #32

TarekHC opened this issue May 4, 2021 · 2 comments

Comments

@TarekHC
Copy link
Collaborator

TarekHC commented May 4, 2021

I discovered today that when you train a model that has been already trained with a different input datasets (which may have a different energy binning) the previous instance of the trained model is not deleted. This gives errors in the later stages of the analysis (e.g. will extract all energy bins, from both instances).

@orelgueta
Copy link
Collaborator

Yes, I am aware of this and it annoyed me to no end until now. Never got around to fixing it =)
Will try to do it soon.

@TarekHC
Copy link
Collaborator Author

TarekHC commented May 4, 2021

Hi Orel

No rush at all! I just stumbled with this problem and lost some time until I realized about it... If you are aware of it, then another thing in our to-do list...

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