Conversation
Member
Author
|
sorta conflicts with #1449, but I wanted to test this independently of that PR |
org.jlab.coda:jclara to 6.1jclara to 6.2 and xmsg to 2.5
c-dilks
marked this pull request as draft
September 28, 2026 20:57
c-dilks
marked this pull request as ready for review
September 29, 2026 12:36
jclara to 6.2 and xmsg to 2.5jclara, xmsg, ccdbrcdb
jclara, xmsg, ccdbrcdb
c-dilks
marked this pull request as draft
September 29, 2026 21:28
c-dilks
commented
Oct 2, 2026
| <artifactId>jevio</artifactId> | ||
| <classifier>jar-with-dependencies</classifier> | ||
| <version>6.2-SNAPSHOT</version> | ||
| <version>6.2.1</version> |
Member
Author
There was a problem hiding this comment.
we need to verify this new version of jevio is consistent with the version we've been using... it's our data format
c-dilks
commented
Oct 2, 2026
clasweb/clas12maven dependencies to code.jlab.org
clasweb/clas12maven dependencies to code.jlab.orgclasweb/clas12maven dependencies to code.jlab.org
This branch has not been deployed
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
The primary goal here is to migrate away from
clasweb's Maven repo, in favor ofcode.jlab.orgpackage registries; reasons:claswebaccess denial by CloudflareIt only addresses those on
clas12maven; those onjhep/mavenare out of this PR's scope.For many dependencies, the migration was straightforward; in some cases, the dependency was removed entirely. The following sections describe each.
Straightforward Dependency Migrations
Most of the changes in these dependencies are for deploying them to their package registries; here are their release notes:
ccdbrcdbetHeld back on the
claswebversion for now, but a new version is available. Since this is risky, it'll be bumped in its own PR.grootjclarajevioHeld back on the
claswebversion for now, but a new version is available. Since this is risky, it'll be bumped in its own PR.xmsgDependency Removals
The following dependencies are no longer needed and therefore have been removed completely:
jtoolsWe only use
org.jlab.clas12.tools.MimeTypeandorg.jlab.clas12.tools.property.JPropertyList, in one of the twoClas12Typesclasses; I updated that class to be more like the other one, removing the need forjtoolscompletely.trackfitterWe are only using its
LineFitter,LineFitPars, andProbChi2perNDF.Here is a repository showing the decompiled JAR file (you must be signed in):
LineFitterandLineFitParsare only consumed bydcmodule, which has its own versions of these classes already.trackfitterversion is practically the same as ourdcmodule's versiontrackfitter'sLineFitter.fitStatusmethod hasdouble[]parameters, whereas thedcmodule's version hasList<double>parametersdouble[]version is more commonly used in the source code, whereas theList<double>version is only used in one classdcmodule'sfitStatusmethod to takedouble[]parameters, and added an overload takingList<double>parametersOn the other hand,
ProbChi2perNDFdoes not appear to be implemented anywhere incoatjava. I decompiledtrackfitter's JAR file, and rewrote this function usingapache'scommon-math3. The edge-case handling remains the same, and is actually consistent with ROOT'sTMath::Probfunction.Special Cases
jminuitWe currently consume
org.freehep:jminuit:1.0fromclasweb, and only therichmodule uses it. This version is binary equivalent toorg.freehep:freehep-jminuit:1.0.1hosted at SLAC: https://srs.slac.stanford.edu/nexus/content/groups/freehep-maven2-public/I decided to bump this to the
1.0.2version from SLACgrootuses this repository and this versionCleaning Shaded JAR (a.k.a.
coat-libs-*.jaruber JAR)Some of these
claswebdependencies are also being shaded (included in our uber JAR), mostly for downstream consumers such asmon12andcalcode. Many of them are unused. The following are removed:org.freehep:jminuit(actually,calcodemay be consuming this transitively, but we should havecalcodeconsume this directly instead)org.jama:jamapackorg.jlab.plugins:jMathde.erichseifert:vectorgraphics2dnet.objecthunter:exp4j