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

Fix setup and support toluene in mdpow-solvationenergy #230

Merged
merged 22 commits into from
Jan 20, 2023
Merged

Conversation

VOD555
Copy link
Contributor

@VOD555 VOD555 commented Jan 20, 2023

Fix #222

  • add mdpow-ptw to setup.py
  • support toluene in mdpow-solvationenergy

iorga and others added 22 commits December 4, 2022 19:32
* add toluene option for AMBER and OPLS-AA in forcefields.py

* add Gtol and pTW for the solvation free energy of a solute in toluene.

* Add TolueneSimulation for running equilibrium simulation for a solute in toluene.

* add toluene as options for equilibrium and fep simulations

* "[atomtypes]" in 1tol.itp are moved to ffnonbonded.itp and renamed to avoid conflict with atomtypes in solute parameters generated by GAFF

* add tests for OPLS-AA and AMBER toluene solvent

Co-authored-by: Shujie Fan <[email protected]>
@codecov
Copy link

codecov bot commented Jan 20, 2023

Codecov Report

Merging #230 (9b45ed3) into develop (e5ce7af) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff            @@
##           develop     #230   +/-   ##
========================================
  Coverage    79.05%   79.05%           
========================================
  Files           12       12           
  Lines         1728     1728           
  Branches       271      271           
========================================
  Hits          1366     1366           
  Misses         277      277           
  Partials        85       85           

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@VOD555
Copy link
Contributor Author

VOD555 commented Jan 20, 2023

@iorga I tested the scripts locally with S01 data. Can you pull and install the toluene_box branch to see whether it works?

@VOD555 VOD555 self-assigned this Jan 20, 2023
@VOD555 VOD555 requested a review from iorga January 20, 2023 10:00
Copy link
Collaborator

@iorga iorga left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This looks good to me. Thanks @VOD555 !

@VOD555 VOD555 merged commit 36f66e9 into develop Jan 20, 2023
@VOD555 VOD555 deleted the toluene_box branch January 20, 2023 15:02
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 this pull request may close these issues.

add toluene as solvent
3 participants