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

Finegraining united atom to all-atom #28

Open
jennyfothergill opened this issue Oct 25, 2021 · 3 comments · May be fixed by #44
Open

Finegraining united atom to all-atom #28

jennyfothergill opened this issue Oct 25, 2021 · 3 comments · May be fixed by #44
Assignees

Comments

@jennyfothergill
Copy link
Member

jennyfothergill commented Oct 25, 2021

Fine-graining is not implemented for system sized morphologies. It would be a good proof of concept to show that we can take a UA trajectory, add hydrogens, and run the relaxation workflow Matty described in Fig 8 of http://dx.doi.org/10.1080/08927022.2017.1296958

We could use a similar logic to the new add_hydrogens kwarg to CG_System.

@chrisjonesBSU
Copy link
Member

I just thought about this while helping Marjan with something. Does GRiTS and smiles string matching work on united atom trajectories?

@JimmyRushing
Copy link

JimmyRushing commented Jun 9, 2022

Is this helpful? This finegrains xml.

https://bitbucket.org/cmelab/morphct/src/master/morphct/code/fine_grainer.py

@jennyfothergill
Copy link
Member Author

jennyfothergill commented Jun 9, 2022

Grits can add hydrogens to a trajectory for coarse-graining https://grits.readthedocs.io/en/latest/grits.html#grits.coarsegrain.CG_Compound
but it's not implemented very well for fine graining purposes yet.

Does GRiTS and smiles string matching work on united atom trajectories?

yes, just use the add_hydrogens kwarg

@jennyfothergill jennyfothergill linked a pull request Jul 6, 2022 that will close this issue
2 tasks
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 a pull request may close this issue.

3 participants