schrodinger / coordgenlibs

Schrodinger-developed 2D Coordinate Generation
BSD 3-Clause "New" or "Revised" License
42 stars 28 forks source link

Setup test infrastructure and add a simple test #35

Closed ricrogz closed 5 years ago

ricrogz commented 5 years ago

I have set up the infrastructure to build a test executable, and added a simple test as example.

Also, to make it easier to add tests, I have extracted and exported the simple maeparser Block to sketcherMinimizerMolecule parser that is used to read the templates used the minimizer.

When I added the test, I found a precision/rounding issue that caused some setups to produce very different results than others. It all started with a minimal difference in the score produced by sketcherMinimizer::testAlignment(), which caused an extra rotation inversion. I patched this adding some rounding to improve reproducibility of the results.

As a side node, I have noticed a great deal of arguments passed by value; it might be worth taking a look at them and turn them into references.