-
Notifications
You must be signed in to change notification settings - Fork 51
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
AMBER parsers #10
Comments
I'm checking with Ross Walker to see if he or someone in his group would want to be involved with this. |
@halx may also be interested. |
I have some response from Ross Walker and collaborators, including some sample files/scripts and potential future interest. I'm checking to see if I can post the info. |
Excellent, thanks @davidlmobley! If you can post these components the best way might be to throw it in a repo so we have a place interested devs can pull from. I can also start hacking on it to see if we run into issues with the data structures we want all parsers to produce, which as you know is based heavily on GROMACS experience. |
From @danmermelstein, here is some AMBER material. Here's what he wrote to me:
I'm attaching his zip file. I'm a bit swamped today/tomorrow so I'm not dumping it into a repo right now, but feel free to put it wherever you want, @dotsdl . |
Closed with PR #36. |
Congratulations @shuail ! Excellent work and a major milestone done. |
In line with the overall API proposal, we want to have parsers for each of the major MD engines, and eventually have coverage for all of those in use. Since there are essentially two types of estimators (TI and FEP), each packages needs a parser for:
Extracting reduced potentials(EDIT: This objective is postponed, emphasis right now on TI. Open issue foru_nk
from output files (for FEP).u_nk
when needed. — @orbeckst 2017-11-03)DHdl
from output files (for TI).This issue is the nexus for discussion for such parsers for the AMBER package. If you have existing parsing code for this package, comment below and we can begin adapting it into the parsers outlined above in a PR.
The text was updated successfully, but these errors were encountered: