Small performance opportunity in new Conjugation class. #395
Labels
enhancement
New feature or request.
help wanted
Extra attention is requested.
low priority
Not a show stopper, but should be addressed.
Originally posted by @primaryodors in #388 (comment)
The algorithm finds the two mutually closest atoms, saving the result in case the same call is made afterward from the opposite direction. However, the returned atom from the second call is always incorrect, which is why the feature was commented. As a result, the algorithm is run twice by the code that calls this function, when once should be sufficient.
The text was updated successfully, but these errors were encountered: