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

Output (just) atom-typing results #233

Closed
mattwthompson opened this issue Apr 11, 2019 · 3 comments
Closed

Output (just) atom-typing results #233

mattwthompson opened this issue Apr 11, 2019 · 3 comments
Labels

Comments

@mattwthompson
Copy link
Member

Foyer kinda squishes two steps: determining appropriate atom types and parametrizing a system according to those types. Paul Saxe today suggested it would be useful for these two steps to be more distinct. I'm not quite sure how this would be implemented but I think it's a good idea and would be a step toward making foyer have a more distinct API from mbuild and other packages.

@ahy3nz
Copy link
Contributor

ahy3nz commented Apr 11, 2019

As it stands now, after atomtyping is run, the omm.Topology contains atoms whose properties are blacklist, bond_partners, element, id, index, name, residue, whitelist, where id (a string) is the decided-atomtype after resolving atomtypes.

As for implementation, I wonder if the most intuitive thing is to return a dictionary whose keys are either omm.Atom, pmd.Atom, mb.Compound/Particle, and values are strings corresponding to the atom type?

@ctk3b
Copy link
Member

ctk3b commented Apr 11, 2019

We could remove whitelist and blacklist from the atoms and just operate on a mapping:

{
  id: {
    whitelist: [],
    blacklist: [],
    atomtype: the_final_type,
  ...
}

Then that mapping IS the output of the atomtyping step and you can just slap it onto the actual atom models. This would also avoid polluting the topology models with atomtyping specific, arguably temporary fields

@justinGilmer
Copy link
Contributor

This has been taken care of by #249

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

4 participants