Update transient HepMcParticleLink to use unsigned int for eventIndex
Once !31619 (merged) has been merged then this merge request will update the transient HepMcParticleLink
class and fix the event number aliasing issue.
Merge request reports
Activity
added full-build label
added 21.9 Generators review-pending-level-1 labels
CI Result FAILURE (hash b9640d74)Athena externals cmake make required tests optional tests Full details available on this CI monitor view
Athena: number of compilation errors 0, warnings 154
For experts only: Jenkins output [CI-MERGE-REQUEST 43283]added review-user-action-required label and removed review-pending-level-1 label
I think the test issue here is the same as elsewhere, that prior updates result in an (expected) change to the simulation output, which requires the reference test to be updated. We would like to get these changes in ASAP so it would be appreciated if this could be reviewed, with the test failure being ignored for the time being.
added review-pending-level-1 label and removed review-user-action-required label
Based on the information from @nstyles, failure of CI tests are not related to this MR.
The copyright could be updated to 2020, but it should further delay this MR.
WARNINGs are unrelated, approving.
Xiaozhong (L1)
added review-approved label and removed review-pending-level-1 label
added review-pending-level-1 label and removed review-approved label
CI Result SUCCESS (hash b9640d74)Athena externals cmake make required tests optional tests Full details available on this CI monitor view
Athena: number of compilation errors 0, warnings 154
For experts only: Jenkins output [CI-MERGE-REQUEST 43326]added review-approved label and removed review-pending-level-1 label
mentioned in commit 5fae6242
added sweep:ignore label