[EVO] Moved stored products in l1t:: to versioned namespace#50075
Conversation
|
cms-bot internal usage |
|
type evolution |
|
-code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-50075/47960 Code check has found code style and quality issues which could be resolved by applying following patch(s)
|
|
+code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-50075/47979 |
|
A new Pull Request was created by @Dr15Jones for CMSSW_16_1_EVOLUTION_X. It involves the following packages:
@BenjaminRS, @cmsbuild, @quinnanm can you please review it and eventually sign? Thanks. cms-bot commands are listed here |
|
please test |
|
assign core |
|
New categories assigned: core @Dr15Jones,@makortel,@smuzaffar you have been requested to review this Pull request/Issue and eventually sign? Thanks |
|
-1 Failed Tests: Build Failed BuildI found compilation error when building: >> Checking EDM Class Version for src/L1Trigger/VertexFinder/src/classes_def.xml in L1TriggerVertexFinder @@@@ ----> OK EDM Class Version L1TriggerPhase2L1GT @@@@ ----> OK EDM Class Version L1TriggerL1THGCalUtilities >> Checking EDM Class Transients in L1TriggerPhase2L1GT >> Checking EDM Class Transients in L1TriggerL1THGCalUtilities error: class 'trigger::TriggerRefsCollections' has a different checksum for ClassVersion 3. Increment ClassVersion to 4 and assign it to checksum 1210037489 error: class 'trigger::TriggerFilterObjectWithRefs' has a different checksum for ClassVersion 3. Increment ClassVersion to 4 and assign it to checksum 4124915994 error: class 'trigger::TriggerEventWithRefs' has a different checksum for ClassVersion 3. Increment ClassVersion to 4 and assign it to checksum 307844616 error: class 'L2MuonTrajectorySeed' has a different checksum for ClassVersion 13. Increment ClassVersion to 14 and assign it to checksum 846735298 error: class 'L3MuonTrajectorySeed' has a different checksum for ClassVersion 3. Increment ClassVersion to 4 and assign it to checksum 1398743106 error: class 'reco::IsolatedPixelTrackCandidate' has a different checksum for ClassVersion 3. Increment ClassVersion to 4 and assign it to checksum 412602791
|
|
+code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-50075/48039 |
|
Pull request #50075 was updated. @BenjaminRS, @Dr15Jones, @Martin-Grunewald, @Moanwar, @cmsbuild, @jfernan2, @makortel, @mandrenguyen, @mmusich, @quinnanm, @smuzaffar, @srimanob can you please check and sign again. |
|
please test |
|
-1 Failed Tests: RelVals AddOn The following merge commits were also included on top of IB + this PR after doing git cms-merge-topic:
You can see more details here:
Failed RelValsExpand to see more relval errors ...Failed AddOn Tests |
|
ignore tests-rejected with manual-override |
| <class name="l1t::HGCalTower" /> | ||
| <class name="std::vector<l1t::HGCalTower>" /> | ||
| <class name="std::unordered_map<int,l1t::HGCalTower>" /> | ||
| <class name="l1t::io_v1::HGCalTower" /> |
There was a problem hiding this comment.
This entry should have a class version (but we could add it later).
|
+core |
|
merge |
PR description:
Part of the versioned namespace migration.
PR validation:
Code compiles.
resolves cms-sw/framework-team#1873