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

Ep/ensemble calc #537

Open
wants to merge 61 commits into
base: master
Choose a base branch
from
Open

Ep/ensemble calc #537

wants to merge 61 commits into from

Conversation

jnsLs
Copy link
Collaborator

@jnsLs jnsLs commented May 16, 2023

replaced ethanol model:
- old model was not compatible with CastTo64 transform anymore
- new model is trained on rmd17 instead of md17

now implemented:
SpkCalculator
SpkEnsembleCalculator <- inherits from SpkCalculator
BatchwiseCalculator <- inherits from SpkCalculator
BatchwiseEnsembleCalculator <- inherits from SpkEnsembleCalculator
SimpleEnsembleAverage (example for a particular average strategy)

the calculators have two new methods (to allow for more modular classes):
_collect_results <-- calculates model outputs and converts property units
_ase_specific_output_format <-- for individual structures this converts energy output to scalar value, for batches of structures it keeps the output in array form

updated batchwise optimization howto jupyter notebook --> 01_howto_batchwise_relaxations.ipynb

added howto_ensemble_calculations jupyter notebook --> 02_howto_ensemble_calculator

mgastegger and others added 30 commits December 16, 2022 11:04
Batchwise optimizer and ensemble NN and calculator are implemented
Add ensemble average functionality
with user defined and provided uncertainty measurement

Test Ensemble Calculator with ASE Optimitation and
ASE MD code
Test Ensemble Calculator with Batchwise optimization

Results from Batchwise Optimization with Batchwise Calculator and
Batchwise Optimitation with Ensemble Calculator differ
on average 0.01 %
Comparision by RMSD of atomic positions
update testing batchwise opt with ensemble
ASE base calculator
with this the ase gui error does not occur
@jnsLs jnsLs requested a review from Stefaanhess May 22, 2023 11:10
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

Successfully merging this pull request may close these issues.

None yet

4 participants