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

Default solvent padding is often nsufficient for solute-only dodecahedron boxes. #1062

Open
IAlibay opened this issue Dec 18, 2024 · 1 comment

Comments

@IAlibay
Copy link
Member

IAlibay commented Dec 18, 2024

Our default 1.2 nm solvation usually works fine when working on cubic boxes, but can lead to the following error when using dodecahedron boxes:

openmm.OpenMMException: The periodic box size has decreased to less than twice the nonbonded cutoff.

The current solution to this problem is to use a larger cutoff for solute-only calculations with dodecahedrons - usually 1.5 nm works.

This may be related to #827, specifically @mattwthompson's comment regarding the bounding box issues with small molecules.

@IAlibay
Copy link
Member Author

IAlibay commented Dec 18, 2024

Potential solutions discussed at yesterday's protocol-devs meeting:

  1. We could have a validation step that attempts to estimate if the box will be too small. I believe (please correct me if I'm wrong!) @hannahbaumann suggested maybe generating multiple conformers and attempting to resolve the box vectors in the same way openmm does.
  2. We could have adaptive settings that dynamically change the cutoff as necessary during the Protocol - which would require us transparently exposing the new settings in some way.
  3. We could have context-specific defaults that change based on the ChemicalSystem composition.

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

1 participant