Skip to content

Conversation

@manuelschwarze
Copy link
Contributor

@manuelschwarze manuelschwarze commented Dec 1, 2023

  • Binaries update
  • Version change from 4.8.0 => 4.8.1 (only for KNIME 4.6 and lower)
  • Bugfix for rooted fingerprints
  • Test updates for tests related to rooted fingerprint, SVG, Optimize Geometry, and Substructure Counter with stereo-chemistry

Automated tests were running successfully on Windows, Linux and Mac. One Mac one of the SVG tests delivered slightly different results, but this had happened in the past and was accepted.

This code will get merged first to nightly_pre4.7, and from there I will cherry-pick all meaningful changes to get into the master (for KNIME 4.7+).

Tosco, Paolo and others added 8 commits November 21, 2023 18:47
…_RW4LHiDKIJVt8dK077EKF75taX4o8e1nri2I@github.com/ptosco/knime-rdkit into nightly_pre4.7_2023_09_2
Some plugins that the RDKit Nodes Wizard requires have a dependency on javax.annotation, which was fulfilled in lower Java versions automatically, but not anymore when using Java 17. Therefore, it is now referenced additionally.
New RDKit version understands correctly duplicates now
Copy link
Member

@greglandrum greglandrum left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I know you tested this, so I'm not doing a detailed review

@greglandrum greglandrum merged commit 1446440 into nightly_pre4.7 Dec 2, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

3 participants