rinikerlab / reeds

This pipeline is executing a RE-EDS run for relative free energy calculations. It can be used for example for calculation of hydration free energies or ligand binding free energies.
https://rinikerlab.github.io/reeds/
MIT License
30 stars 8 forks source link

Rework of initialization options #66

Closed eruijsena closed 2 years ago

eruijsena commented 2 years ago

Description

This branch contains a rework of the imd manipulation / adaptation for steps a-c as well as some minor tweaks.

Changes to the imd:

Other changes:

Todos

Notable points that this PR has either accomplished or will accomplish.

lgtm-com[bot] commented 2 years ago

This pull request introduces 7 alerts and fixes 1 when merging ee497357b388a9d4b344fa050f1c871abf8908bb into 84ac08d1b857c371106056110e8562274a7b9828 - view on LGTM.com

new alerts:

fixed alerts:

lgtm-com[bot] commented 2 years ago

This pull request introduces 5 alerts and fixes 1 when merging f5aae72cfe0b61501f88f2ec1230ea9f16d51581 into 84ac08d1b857c371106056110e8562274a7b9828 - view on LGTM.com

new alerts:

fixed alerts:

candidechamp commented 2 years ago

I just found a small bug in the import statements of reeds/function_libs/pipeline/module_functions.py

the line from pygromos.utils import amino acids as bash should read from pygromos.utils import bash

Apart form that everything else looks very good.

eruijsena commented 2 years ago

Voilà!

lgtm-com[bot] commented 2 years ago

This pull request introduces 5 alerts and fixes 1 when merging d2d5838b781f483bd0294737e9c94b7f4d4b4ea6 into 84ac08d1b857c371106056110e8562274a7b9828 - view on LGTM.com

new alerts:

fixed alerts: