Skip to content

Address primary-ligand scoring bug#22

Merged
amorehead merged 26 commits intomainfrom
dev/20
Mar 21, 2026
Merged

Address primary-ligand scoring bug#22
amorehead merged 26 commits intomainfrom
dev/20

Conversation

@amorehead
Copy link
Collaborator

What does this PR do?

Addresses a primary-ligand scoring bug discovered by @95028. Astex Diverse, DockGen-E, and PoseBusters Benchmark results have been updated accordingly, and an updated arXiv manuscript should be online soon.

Fixes #20.

Before submitting

  • Did you make sure title is self-explanatory and the description concisely explains the PR?
  • Did you make sure your PR does only one thing, instead of bundling different changes together?
  • Did you list all the breaking changes introduced by this pull request?
  • Did you run pre-commit hooks with pre-commit run -a command?

Did you have fun?

Make sure you had fun coding 🙃

…molecule objects in select_primary_ligands_in_df
@amorehead amorehead merged commit afa3d52 into main Mar 21, 2026
3 of 4 checks passed
@amorehead amorehead deleted the dev/20 branch March 21, 2026 00:34
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.

Bug in select_primary_ligands_in_df: true ligand coordinates are modified during fragment selection, causing artificially low RMSD

1 participant