Skip to content
Snippets Groups Projects

RPC Phi fast digitisation

Merged Paul Nathaniel Swallow requested to merge pswallow/athena:ps-Digitisation into main
All threads resolved!

In the Phase II RPCs for the readout instead of two orthogonal planes on either side of the gasGap to measure (Eta,Phi), they have two parallel planes measuring (Eta) but both sides of the plane are read out, such that the time difference can be used to determine Phi.

This MR adds in fast digitisation in this case, and a smearing for the Phi coordinate for typical propagation velocities and RPC time resolutions. Then assigns it to an RpcStrip2D container.

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
  • added 1 commit

    • 7429f284 - Removing a validation TTree for testing.

    Compare with previous version

  • Paul Nathaniel Swallow resolved all threads

    resolved all threads

  • This merge request affects 2 packages:

    • MuonSpectrometer/MuonPhaseII/MuonCnv/xAODMuonSimHitCnv
    • MuonSpectrometer/MuonPhaseII/MuonValidation/MuonPatternRecognitionTest

    Affected files list will not be printed in this case

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

  • Removed a Validation TTree that was for testing purposes - at Johannes' request will create a separate algorithm for validation testing later.

  • CI Result SUCCESS (hash 8cc66856)

    Athena
    externals
    cmake
    make
    tests

    Full details available on this CI monitor view. Check the JIRA CI status board for known problems
    Athena: number of compilation errors 0, warnings 1
    📝 For experts only: Jenkins output [CI-MERGE-REQUEST-EL9 11711] (remote access info)

  • CI Result SUCCESS (hash 7429f284)

    Athena
    externals
    cmake
    make
    tests

    Full details available on this CI monitor view. Check the JIRA CI status board for known problems
    Athena: number of compilation errors 0, warnings 0
    📝 For experts only: Jenkins output [CI-MERGE-REQUEST-EL9 11714] (remote access info)

  • All OK - Joany (L1)

  • merged

  • Edward Moyse mentioned in commit 83c95ae8

    mentioned in commit 83c95ae8

  • Please register or sign in to reply
    Loading