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

Update main.cpp . CLASSIFY LIGANDS WITH 160-300 ATOMS IN MaxGroup #150

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

PabloPauling
Copy link

Added else if to classify ligands wwith number of atoms between 160 and 300. Actual version missed the ligands that should be added to maxGroup.

Added else if to classify ligands wwith number of atoms between 160 and 300.
Actual version missed the ligands that should be added to maxGroup.
@OleksandrSavytskyi
Copy link

I have another question: Will increasing the number of atoms in ligands (> 160 atoms for unidock) have any negative effects on accuracy? For instance, might the grid size or the number of cells in the grid need to be limited? Or does this depend on the scoring function we use, such as AD4 or Vina/Vinardo (there are no electrostatic calculations)? Maybe it was the reason why you locked unidock for ligands up to 160 atoms. It will be useful to have feedback from your side. Thank you in advance.

@zhengh96 zhengh96 requested a review from kongexp February 27, 2025 03: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.

2 participants