-
Notifications
You must be signed in to change notification settings - Fork 3
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
Adding 6 new benchmark systems #3
base: master
Are you sure you want to change the base?
Conversation
@brycestx @jeremy2401 - I see some issues that it would be nice to have sorted out here. Starting with the big picture:
Minor quibbles:
Big picture: Make it so people can tell what you're adding, why you're adding it, and why it might be interesting to them without having to read all of the papers you're depositing. (Also take note you probably don't have the rights to deposit the papers, only link to them...) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Changes requested, see comments.
Other things that would be worth knowing:
|
@brycestx any chance of follow-up on this? |
@davidlmobley I never received any response from @jeremy2401 in regards to your comments. Let me assess bandwidth internally and figure out if we can reproduce and redocument from scratch Jeremy's work. |
This PR adding protein and ligand systems for 6 systems that come from JMedChem papers published recently. Readme and documentation is provided. Most of the work came from @jeremy2401 from an internship this summer.