-
Notifications
You must be signed in to change notification settings - Fork 18
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
Specify molecules names in the input file #137
Comments
I really like your proposal.
Actually, we don't need the positions. We can compute them based on the insertion scheme, i.e using configurational bias using the intramolecular potentials. |
Yep, that is true! We will need support for rigid molecules in CBMC too then. Do you know if this is possible? |
Yes it's quite easy to implement. I'd add fixed bonds/angles as soon as we decide on how to communicate constraints. |
Would you also use this declaration of molecules to create or check bonds/angles/dihedrals from the positions in the start configuration? |
Creating bonds will be hard: how could we know which atoms are in the same molecules if we don't have the bonds between them? Checking bonds might be hard too: how do we know that two molecules are equivalent, and that they should have the same bonds? Either they have the same bonds, or we need to check for graph equivalence. Actually, the way I though of this was to assign molecule types by checking if the definition in the input file and in the initial configuration did match. On the 'check' side, we could have a flag to ensure that all the molecules in the initial configuration are defined in the input file. |
Apologies, I did not express myself very well here.
Ok. I thought we'd supply |
We don't need to, as we can determine them from the list of bonds only. And yes, there would still be ambiguities in the generic case. This is mainly due to using chemfiles as input library, and accepting any file format. I feel like this is still a nice feature to have, but we could pick a standard format and add specific features to this format. For example automatic molecules detections ! |
This is an user interface improvement proposed by @g-bauer.
The idea is to specify molecules in a section of the input, and then use the molecules by names while specifying moves.
The current input look like this:
The idea is to add a 'molecule' section:
Proposed syntax
We only need the molecule type for most Monte-Carlo moves, which is derived from the atoms names and the bonding information. We can get it the same way as today, from a file:
Or explicitly specify it:
For GCMC moves, we also need the atomic positions:
The text was updated successfully, but these errors were encountered: