Skip to content
Snippets Groups Projects

xAODMuonPrepData - Split up the sTgcPrepData

Merged Johannes Junggeburth requested to merge jojungge/athena:SplitSTGCMeas into main

Hi everybody,

this merge request splits up the sTgc uncalibrated measurement container into Strip, Wire and Pad hits because I think they're quite differnt

  • Strip hits are the only ones which are clustered. They're the only one where the information about the cluster constituents makes sense
  • Pad hits are intrinsically 2D measurements. Dumping their local coordinates is incompatible with the dump of strips and wires

Tagging: @emoyse, @pscholer

Edited by Johannes Junggeburth

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
  • Jenkins please retry a build

  • Johannes Junggeburth resolved all threads

    resolved all threads

  • added 1 commit

    • 81b0c849 - Apply 1 suggestion(s) to 1 file(s)

    Compare with previous version

  • This merge request affects 4 packages:

    • MuonSpectrometer/MuonPhaseII/Event/xAOD/xAODMuonPrepData
    • MuonSpectrometer/MuonPhaseII/Event/xAOD/xAODMuonPrepDataAthenaPool
    • MuonSpectrometer/MuonPhaseII/MuonCnv/xAODMuonSimHitCnv
    • MuonSpectrometer/MuonPhaseII/MuonCnv/xAODMuonTrkPrepDataCnv

    Affected files list will not be printed in this case

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

  • :x: CI Result FAILURE (hash 6f253a42)

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

    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-EL9 4088]

  • added 1 commit

    Compare with previous version

  • Johannes Junggeburth marked this merge request as ready

    marked this merge request as ready

  • Jenkins please retry a build

  • This merge request affects 4 packages:

    • MuonSpectrometer/MuonPhaseII/Event/xAOD/xAODMuonPrepData
    • MuonSpectrometer/MuonPhaseII/Event/xAOD/xAODMuonPrepDataAthenaPool
    • MuonSpectrometer/MuonPhaseII/MuonCnv/xAODMuonSimHitCnv
    • MuonSpectrometer/MuonPhaseII/MuonCnv/xAODMuonTrkPrepDataCnv

    Affected files list will not be printed in this case

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

  • Johannes Junggeburth changed the description

    changed the description

  • :white_check_mark: CI Result SUCCESS (hash 3a970fa0)

    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-EL9 4096]

  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Please register or sign in to reply
    Loading