John Mayfield
John Mayfield
Javax is now jakarta, and looks like Java 21 removed some of the parser APIS. You can add in the missing dependency or use Java 8/11/17 (maybe). Curiously our Java...
Can you try it directly in Java (or Jshell). ``` $ /opt/jdk-21.jdk/Contents/Home/bin/jshell --class-path ~/Downloads/cdk-2.9.jar ``` The following (Java equivalent of your groovy script) works OK ``` import org.openscience.cdk.fingerprint.*; import org.openscience.cdk.smiles.*;...
Try adding it and see if it fixes it? JARs are just zip files. I don’t recall changing how we packaged it.- JohnOn 28 Feb 2024, at 14:17, John Sing...
I think if this works in Java/Jshell 21 this is a groovy config issue so not much we can do. We didn't change how things were packaged but did update...
Why not off by default?
``` [john@sentinel:cdk]% du -sch bundle/target/cdk-2.9-SNAPSHOT.jar 41M bundle/target/cdk-2.9-SNAPSHOT.jar 41M total ``` Note the RInChI inclusion will make this larger so it's a losing battle :-).
41 still down from the peak of 45 but still larger
Up to 42 again...
Presume still similar... not much we can do without sweeping changes. Leave open as just good to keep in mind.
Merged in the test case, still have to option revert though although I believe your view is hopefully going forwards we should be able to detect the change and warn...