Conversation
|
This is not ready for official review, I"ll be breaking it into chunks for PRs. |
|
| # Small numeric differences select a different basin for the first molecule. | ||
| return molecules[1:] |
There was a problem hiding this comment.
This now drops the first UFF validation molecule for both precision modes and reduces the fixture from five molecules to four, although the stated basin instability only explains excluding it from SINGLE. This weakens the existing default FULL regression coverage while the force-field and minimizer implementations are being generalized. Please retain this molecule for FULL and apply any precision-specific expectation only to SINGLE. The same unconditional removal also appears in tests/test_uff.cu:301-302.
No description provided.