Modify the pers->trans conversion for cool based metadata
This recovers the lost information when the array index was stored as short instead of int (ATR-22116)
RD, could you please take a close look to see if I have missed anything. I have implemented it in such a way that it would also work if it is reading from integer.
Merge request reports
Activity
added 21.0 label
added Database review-pending-level-1 labels
✅ CI Result SUCCESS (hash 9f1a55d0)Athena AthDataQuality AthSimulation externals ✅ ✅ ✅ cmake ✅ ✅ ✅ make ⚠ ⚠ ⚠ required tests ✅ ✅ ✅ optional tests ✅ ✅ ✅ Full details available on this CI monitor view
⚠ Athena: number of compilation errors 0, warnings 162
⚠ AthDataQuality: number of compilation errors 0, warnings 1
⚠ AthSimulation: number of compilation errors 0, warnings 6
📝 For experts only: Jenkins output [CI-MERGE-REQUEST-CC7 45040]Should be fine from MR point of view, the make warnings are not connected toi this MR.
Pinging @schaffer , I think the message in MR description didn't went to him.
Pavol [as L2 MR shifter]
added review-approved label and removed review-pending-level-1 label
Thank you @schaffer for the confirmation and the reviewers for their checks as well.
mentioned in commit 82513fc8
added sweep:done label
picked the changes into the branch
cherry-pick-82513fc8229-master
with commit d3fb4c83mentioned in commit d3fb4c83
Sweep summary
successful:- master
(see https://aiatlas154.cern.ch/jenkins/job/auto_git_sweeps/1317/console for details)
mentioned in merge request !37545 (merged)
mentioned in commit ef64992d
mentioned in merge request !37687 (merged)
picked the changes into the branch
cherry-pick-82513fc8
with commit 6374809bmentioned in commit 6374809b
mentioned in merge request !37725 (merged)
Hi @stelzer,
I cherry-picked the change to 21.2 in !37725 (merged). (Using the button above.)
Cheers,
John
Thank you @jchapman !
Cheers, Joerg
mentioned in commit a034b886
mentioned in merge request !42384 (merged)