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

AlkEthOH (rings) snapshots and energies #23

Draft
wants to merge 234 commits into
base: main
Choose a base branch
from
Draft

AlkEthOH (rings) snapshots and energies #23

wants to merge 234 commits into from

Conversation

maxentile
Copy link
Member

Continuing from #20 , adding configuration snapshots, MM energies, and "ab-initio-like" energies

Done:

  • Generate dataset of ~1million configuration snapshots (1000 per molecule, for the ~1000 molecules of AlkEthOH rings set, by 500K MD), with accompanying Parsley and ANI-1ccx potential energies, as well as parameterized system objects. (Tracked with Git LFS.)

Todo:

Followup:

  • Expand to other molecule sets as planned

maxentile and others added 30 commits May 27, 2020 18:06
note that OpenEye Omega conformer generation failed for 906 of 1156 of these (results are produced for 250 of 1156)

this is due to an issue that was addressed in openforcefield/openff-toolkit#517 -- will need to update to use latest development version of openforcefield toolkit instead of 0.6.0 ...
should revert this when 0.7.0 is released
Each of the 1156 molecules in the AlkEthOH rings set was parameterized using OpenFF 1.0.0 Parsley. Serialized OpenMM systems are saved for each parameterized molecule are saved.

Each molecule is simulated in vacuum at 500K for 1 nanosecond, using the script in mm.py. (Simulation details: BAOAB Langevin thermostat, 1fs timestep, collision rate of 1ps, configuration snapshot saved every 1ps). Snapshots are saved using mdtraj, which stores xyz coordinates in units of nanometer.

Molecular mechanics energies and ANI-1ccx energies for each of these ~1million snapshots are also saved in units of kJ/mol.
hmm, minimization problem for bond and angle terms, torsions are solved though...
also, adjust so that it's randomly initialized between 0 and 2 * true value
@lgtm-com
Copy link

lgtm-com bot commented Aug 24, 2020

This pull request introduces 20 alerts when merging ccab4cf into cb0805c - view on LGTM.com

new alerts:

  • 10 for Unused import
  • 4 for Unused local variable
  • 3 for Non-callable called
  • 2 for Variable defined multiple times
  • 1 for Module is imported with 'import' and 'import from'

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.

2 participants