[baltzell]

Requires #1491 and #1541.

[c-dilks]

updated kpp-ai and rebased this j4ml branch onto it

[c-dilks]

Looking great so far!

[c-dilks]

The above CI failures indicate that slf4j is no longer found at runtime; that is because it was being resolved through some j4ml packages, transitively, e.g.:

org.jlab.clas12.detector:clas12detector-mltn:jar:14.2.3-SNAPSHOT
+- j4ml:j4ml-neuroph:jar:0.9-SNAPSHOT:compile
   +- com.github.neuroph:neuroph-core:jar:2.98:compile
      +- org.slf4j:slf4j-api:jar:1.7.6:compile

However, djl also needs it at runtime, so it needs to be added to our installation classpath.

The reconstruction/alert module copies some of djl's runtime dependencies, which ultimately are copied into our installation prefix class path (/coatjava/lib/services); slf4j, among others, are not included.

Instead of leaving it up to the alert module's POM, I decided to pull this stuff out into a new dedicated module, common-tools/djl-runtime. Instead of including just the artifacts we need, I use the djl runtime scope, so there are a few more JARs in the prefix, but it reduces the maintenance burden here (if djl needs a new runtime dependency, we won't be "surprised").

Here's the diff of the prefix classpath file tree:

[c-dilks]

Sorry for hijacking this PR, to see what I did,

git diff fc8060635..HEAD

[raffaelladevita]

don't have the expertise to review the new djl-runtime module but can approve the rest

[c-dilks]

don't have the expertise to review the new djl-runtime module but can approve the rest

@baltzell are you okay with how djl is handled here?