[20_0_X] Move stored types in io_v1 namespace, reset class versions to 3#50968
Open
makortel wants to merge 1 commit into
Open
[20_0_X] Move stored types in io_v1 namespace, reset class versions to 3#50968makortel wants to merge 1 commit into
makortel wants to merge 1 commit into
Conversation
Contributor
Author
|
hold Until 20_0_X opens |
cmsbuild
added
reconstruction-pending
simulation-pending
dqm-pending
l1-pending
hlt-pending
core-pending
analysis-pending
alca-pending
daq-pending
fastsim-pending
generators-pending
visualization-pending
pending-signatures
tests-pending
orp-pending
hold
code-checks-pending
xpog-pending
tracking
trk
changes-dataformats
labels
Contributor
|
cms-bot internal usage |
Contributor
|
+code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-50968/49372
|
Contributor
|
Pull request has been put on hold by @makortel |
Contributor
|
A new Pull Request was created by @makortel for master. It involves the following packages:
@Alejandro1400, @BenjaminRS, @Dr15Jones, @JanChyczynski, @Martin-Grunewald, @Moanwar, @alja, @arunhep, @atpathak, @battibass, @civanch, @cmsbuild, @ctarricone, @emeschi, @ftorrresd, @gabrielmscampos, @jfernan2, @kpedro88, @lviliani, @makortel, @mandrenguyen, @mdhildreth, @mkirsano, @mmusich, @perrotta, @quinnanm, @rseidita, @sensrcn, @smorovic, @smuzaffar, @srimanob, @ssekmen, @theofil, @tvami can you please review it and eventually sign? Thanks. cms-bot commands are listed here |
Contributor
Author
Contributor
Author
|
This change was needed because #49932 was merged recently and that added a forward declaration instead of |
Contributor
|
+code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-50968/49434
|
Contributor
|
Pull request #50968 was updated. @Alejandro1400, @BenjaminRS, @Dr15Jones, @JanChyczynski, @Martin-Grunewald, @Moanwar, @arunhep, @atpathak, @battibass, @civanch, @cmsbuild, @ctarricone, @ftorrresd, @gabrielmscampos, @jfernan2, @kpedro88, @lviliani, @makortel, @mandrenguyen, @mdhildreth, @mkirsano, @mmusich, @perrotta, @quinnanm, @rseidita, @sensrcn, @smuzaffar, @srimanob, @ssekmen, @theofil can you please check and sign again. |
Contributor
Author
|
@cmsbuild, please test |
Contributor
|
-1 Failed Tests: ClangBuild Failed Clang BuildI found compilation error while trying to compile with clang. Command used: >> Entering Package TopQuarkAnalysis/TopSkimming >> Entering Package TopQuarkAnalysis/TopTools >> Entering Package TrackPropagation/Geant4e >> Entering Package Utilities/ReleaseScripts >> Compile sequence completed for CMSSW CMSSW_17_0_X_2026-05-21-1100 gmake: *** [There are compilation/build errors. Please see the detail log above.] Error 1 Command exited with non-zero status 1 Command being timed: "scram build -k -j 32 COMPILER=llvm compile BUILD_LOG=yes" User time (seconds): 8229.75 System time (seconds): 859.53 Percent of CPU this job got: 1086% |
cmsbuild
added
tests-pending
code-checks-pending
and removed
tests-rejected
code-checks-approved
labels
Contributor
|
-code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-50968/49446 ERROR: Build errors found during clang-tidy run. |
The io_v1 namespace allows schema evolution of complex cases (e.g. pointed-to container type of edm::Ref being changed) that ROOT's schema evolution can not support. Moving classes to io_v1 breaks backwards compatibility for these types that was agreed to be done in CMSSW_20_0_0 that serves a start of Phase 2 (only) targeting software. Taking advantage of the backwards compatiblity break, the class versions are reset to 3. In addition, include two complex schema evolution cases that were too difficult earlier - Make PFCandidate to no longer inherit from CompositeCandidate - Reduce use of RangeMap in muon code
makortel
force-pushed
the
evoIntegrationDataFormats
branch
from
b4d5481 to
909f324
Compare
Contributor
|
+code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-50968/49457
|
Contributor
Author
|
@cmsbuild, please test |
Contributor
|
Pull request #50968 was updated. @Alejandro1400, @BenjaminRS, @Dr15Jones, @JanChyczynski, @Martin-Grunewald, @Moanwar, @arunhep, @atpathak, @battibass, @civanch, @ctarricone, @ftorrresd, @gabrielmscampos, @jfernan2, @kpedro88, @lviliani, @makortel, @mandrenguyen, @mdhildreth, @mkirsano, @mmusich, @perrotta, @quinnanm, @rseidita, @sensrcn, @smuzaffar, @srimanob, @ssekmen, @theofil can you please check and sign again. |
Contributor
|
-1 Failed Tests: RelVals-INPUT Failed RelVals-INPUT
Expand to see more relval errors ...Comparison SummaryMax Memory Comparisons exceeding threshold@cms-sw/core-l2 , I found 55 workflow step(s) with memory usage exceeding the error threshold: Expand to see workflows ...
|
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
8 participants
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.
PR description:
This PR integrates the data format changes prototyped and tested in the EVOLUTION_X branch to the
masterbranch for 20_0_X. Following the strategy agreed in ORP 2026-05-05, it is opened already now to allow it to be reviewed such that it could be merged quickly when the master branch moves to 20_0_X.This PR includes
io_v1namespace to allow schema evolution for complex cases that ROOT's schema evolution can not support (e.g. pointed-to container type of edm::Ref being changed)For more information please see the ORP 2026-05-05 presentation and the earlier talks linked there.
This PR is unusual in several ways:
git show -b b104b18evolutionlabel https://github.com/cms-sw/cmssw/pulls?q=is%3Apr+label%3Aevolution+is%3AclosedResolves cms-sw/framework-team#2227
PR validation:
None beyond the tests in the EVOLUTION_X IBs. Checked on CMSSW_17_0_X_2026-05-18-1100 that cherry-picking #50953, #50954, and this PR shows no difference wrt. CMSSW_17_0_EVOLUTION_X branch.