Skip to content

17 tiny molecules cause issues#18

Merged
ntampellini merged 3 commits intomasterfrom
17-tiny-molecules-cause-issues
Mar 17, 2026
Merged

17 tiny molecules cause issues#18
ntampellini merged 3 commits intomasterfrom
17-tiny-molecules-cause-issues

Conversation

@ntampellini
Copy link
Owner

  • Added checks for linear molecules, single atoms, and non-heavy molecules (i.e. only made up of "H" atoms, for now).
  • Avoids RuntimeWarnings and appears to have the desired behavior.
  • Does not seem too detrimental to the performance of large ensembles.

@corinwagen Do you want to give this a try and see if performance is not impacted on your end as well?

@ntampellini ntampellini linked an issue Mar 17, 2026 that may be closed by this pull request
@ntampellini ntampellini self-assigned this Mar 17, 2026
@corinwagen
Copy link
Contributor

seems good to me!

@ntampellini ntampellini merged commit e46f843 into master Mar 17, 2026
1 check failed
@ntampellini ntampellini deleted the 17-tiny-molecules-cause-issues branch March 17, 2026 20:55
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.

Tiny Molecules Cause Issues

2 participants