-
Notifications
You must be signed in to change notification settings - Fork 46
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
ValueError: Key "model_builders" is different in config and the result trainer.pth file. Please double check #111
Comments
Dear Giuseppe, Thank you for your interest in our code. Apologies for the delayed response, but the If you really need to get things to work with the current public code and want to try to debug this issue, some comments. The error thrown comes from I would advise trying to migrate to the new infrastructure if possible, but it's understandable if it's more favorable to continue using the current public infrastructure if you're in the middle of a project and have various models trained in that framework. Happy to give further advice. Chuin Wei |
Dear Wei, thanks a lot for your reply! I've a couple of questions. I've installed both develop branches of nequip and File "/home/cassone/anaconda3/lib/python3.12/site-packages/hydra/core/override_parser/overrides_parser.py", line 96, in parse_overrides Also, if I run "pytest tests/" to make an extensive testing of the nequip installation I get this error: ERROR tests/integration/test_deploy.py Could you please tell me why and how to fix these errors related to the develop branches of nequip and allegro? Thanks a lot in advance. One final thing related to my previous issue: how could I inspect the best_model.pth file? Many thanks in advance and best wishes, |
with several caveats depending on where you're running it from, etc.
|
Dear Chuin Wei, thank you for the prompt reply. Additionally to the tutorial.yaml file, are there some other templates to be used for training/testing, specifically adapted for water systems? Thank you again and best wishes, |
Hi Giuseppe, For learning about the training infrastructure, This paper should have the relevant Allegro details for water systems. https://pubs.acs.org/doi/10.1021/acs.jpclett.4c00605. The SI has an allegro config for the old infrastructure, you'd have to translate it to the new infrastructure (carefully separating what are training hyperparameters and model architecture hyperparamaters, since the old infrastructure uses a flat list of configuration arguments while the new infrastructure has them separated into sections). Chuin Wei |
Dear Chuin Wei, thanks a lot for your important suggestions. Since the documentation on the development version of allegro is not yet available (isn't it?), could you please tell me how, e.g., to exploit either TensorBoard or WandB in conjunction with the new development branches? Is there any blog on these new development versions? Thanks a lot in advance and best wishes, |
Hi Giuseppe,
You can find the relevant line here. https://github.com/mir-group/nequip/blob/ece09b587ab1082c2c806a094fb5cc1dc5489b60/configs/tutorial.yaml#L125 We've migrated to using Last bit of warning -- I think I was over-optimistic in my suggestions to use the new developments and wish to now backtrack (sorry!). To put it bluntly, it's not stable enough for me to recommend migrating over for production use at this point in time (but definitely fine if you wanna test it, with the expectation that things will change in breaking ways in the coming months, such that you might have to reinstall everything/retrain all your models, etc). That being said, if you want to use the new developments and face problems, we can be reached at [email protected]. |
Dear developers,
I'm trying to restart a train that crashed by means of inserting the following lines in the input:
initial_model_state: results/example/best_model.pth
However, it systematically gives the following error:
ValueError: Key "model_builders" is different in config and the result trainer.pth file. Please double check
Does anyone knows why?
Thanks in advance and best wishes,
Giuseppe
The text was updated successfully, but these errors were encountered: