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

Cg haddock cg2aa #1193

Open
wants to merge 23 commits into
base: CG-HADDOCK
Choose a base branch
from
Open

Cg haddock cg2aa #1193

wants to merge 23 commits into from

Conversation

rversin
Copy link
Member

@rversin rversin commented Jan 24, 2025

You are about to submit a new Pull Request. Before continuing make sure you read the contributing guidelines.

Checklist

  • Tests added for the new code
  • Documentation added for the code changes
  • Does not break licensing
  • Does not add any dependencies, if it does please add a thorough explanation

Summary of the Pull Request

The aatocg module was added. In order to have the aatocg module fully functional, modifications had to be made to the libs library. The topocg and rigisbody modules had to be adapted as well.

Related Issue

Additional Info

Copy link
Contributor

Choose a reason for hiding this comment

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

Are all those parameters required to the cgtoaa module ?
I think most of them could be removed (e.g: ligand_param_fname, maxmodels, etc...)

Copy link
Member

Choose a reason for hiding this comment

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

Good point - I did already remove quite a bit - will check the remaining ones.

src/haddock/libs/libcns.py Outdated Show resolved Hide resolved
src/haddock/libs/libcns.py Outdated Show resolved Hide resolved
amjjbonvin and others added 2 commits January 27, 2025 17:45
Co-authored-by: Victor Reys <132575181+VGPReys@users.noreply.github.com>
@rvhonorato rvhonorato removed their request for review January 28, 2025 09:55
rversin and others added 3 commits January 29, 2025 14:14
Co-authored-by: Victor Reys <132575181+VGPReys@users.noreply.github.com>
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.

3 participants