-
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
Update SolventComponent properties to include those that define the composition of the solvent #261
Comments
I've put more detail in the PR, but I don't think I agree that |
I've responded in the PR - I think there's a fundamental issue where we're trying to be too conservative in where these things will be applied. Defining N is important and really should be a Component thing in my opinion - especially because it's crucial to things like binding enthalpy calculations. |
Currently we have a mismatch where we have some details about how solvation happens in the SolvationSettings and some in SolventComponent.
My proposal is that anything that defines the molecular composition of the system should be defined in the the SolventComponent.
This includes:
padding
&mass density
used to add the watersBox vectors
Box shape
ions
&concentration
that will be added (done)smiles
(done)By comparison, anything that defines the implementation details should be in the
SolvationSettings
:water model
tolerance
for any heuristic in water addition (see packmol)The text was updated successfully, but these errors were encountered: