Skip to content

Switch to Using Heavy Matches Heavy Atom mappings#10

Merged
badisa merged 4 commits into
mainfrom
task/heavy_matches_heavy_maps
Feb 12, 2026
Merged

Switch to Using Heavy Matches Heavy Atom mappings#10
badisa merged 4 commits into
mainfrom
task/heavy_matches_heavy_maps

Conversation

@badisa

@badisa badisa commented Feb 12, 2026

Copy link
Copy Markdown
Contributor

Warning

Note that in #9 there was an issue with the OpenFE preparation, so it is different in notable ways to the previous sets.

Summary

Overall the total nanoseconds simulated changed from +2% nanoseconds to -30% nanoseconds simulated. The average reduction in nanoseconds appears to be about 10%.
image

The overall rank correlation held out for larger datasets, but suffered for the smaller sets which are more sensitive to run to run variability
image

One such example is Factor Xa, where a minor change in the prediction swaps the ranking of two compounds and produces a -1 correlation
image

The one set that stood out from this was BACE, which is covered in more detail below.

BACE

Looking through the results the BACE dataset stood out as a set that got worse. To investigate, multiple replicates were run with and without the new maps to see how the performance changes. Despite a significant change to the previous BACE results, the replicates with the old and new maps produced similar results.

Reference (5 replicates) Heavy matches Heavy (6 replicates)

Looking closer it became apparent that the preparation of the datasets from OpenFE was wrong and tearing out more than intended. In the case of BACE there are crystal waters that were removed. Re-running the performance looks more in line with previous references

Reference (5 replicates) Heavy matches Heavy (3 replicates)

The performance of the heavy matches heavy is still slightly worse than the previous atom mapping, but for consistency will stay with the new map. The atom mapping protocol may be useful to tweak for different systems, which is already an expectation.

Next Steps

  • Fix the preparation issues and re-run the benchmarks.

* Things are faster, but needs a bit more analysis to be sure that this
  is a net improvement
@badisa badisa merged commit d356f06 into main Feb 12, 2026
1 check passed
@badisa badisa deleted the task/heavy_matches_heavy_maps branch February 13, 2026 00:01
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.

1 participant