Skip to content
Snippets Groups Projects

MuonSimHit - Add prefixes to the sim hit Identifiers

Merged Johannes Junggeburth requested to merge jojungge/athena:SimHitDecors into main
All threads resolved!

Avoids type race conditions in the xAOD.

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Johannes Junggeburth resolved all threads

    resolved all threads

  • added 1 commit

    • f9a1d469 - Apply 1 suggestion(s) to 1 file(s)

    Compare with previous version

  • This merge request affects 3 packages:

    • MuonSpectrometer/MuonPhaseII/Event/xAOD/xAODMuonSimHit
    • MuonSpectrometer/MuonPhaseII/MuonCnv/xAODMuonSimHitCnv
    • MuonSpectrometer/MuonPhaseII/MuonDetDescr/MuonGeoModelTestR4

    Affected files list will not be printed in this case

    Adding @rosati ,@stavrop ,@kluit ,@jojungge ,@apsallid ,@pscholer as watchers

  • :pencil: :scissors: The system selected 51 tests to probe the Athena changeset (out of 65 available tests). Link to tests selection rules

  • :white_check_mark: CI Result SUCCESS (hash f9a1d469)

    Athena
    externals :white_check_mark:
    cmake :white_check_mark:
    make :white_check_mark:
    tests :white_check_mark:

    Full details available on this CI monitor view. Check the JIRA CI status board for known problems
    :white_check_mark: Athena: number of compilation errors 0, warnings 0
    :pencil: For experts only: Jenkins output [CI-MERGE-REQUEST-CC7 79283]

  • The changes look okay and there are no issues with the CI. Approving from L1.

  • added review-approved label and removed review-pending-level-1 label

  • merged

  • Edward Moyse mentioned in commit e8244ad6

    mentioned in commit e8244ad6

  • Please register or sign in to reply
    Loading