-
Notifications
You must be signed in to change notification settings - Fork 9
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
Add MD progress bar #430
Add MD progress bar #430
Conversation
Thanks for this! When you say does not discriminate, do you mean it has a single progress bar for a combined heating + MD simulation, with the steps for both summed? |
I'm not sure the heating is currently filling up completely (with or without MD after)?
|
I noticed this too when the CI tests ran. I can fix it, but I'm trying to see if there's a better way to approach this since this method seems really fragile at the moment. |
Superseded by #444 |
Resolves #405
I think there are a few ways that this could be done - this one (attaching the progress bar as an observer function) does not discriminate between heating and production MD steps. I tried using
irun
as had been suggested, but since half of the ASE ensembles don't supportirun
right now, this seemed like the best way.There's a hack at md.py lines 1060/1061 to get around ASE's
NPT
ensemble calling observers 1 less time than expected. Once the fix is incorporated (https://gitlab.com/ase/ase/-/merge_requests/3598), those lines will need to be deleted.There is also a test that should check each ensemble fills the progress bar completely by the end of the run.