Skip to content
Snippets Groups Projects

21.9 Further updates from 20.20

Merged Noemi Calace requested to merge ncalace/athena:21.9-CheckReconstruction into 21.9
All threads resolved!

This MR collects further updates to be ported from 20.20 to 21.9. I have tested the following on top of !27922 (merged). Therefore, this stays WIP until !27922 (merged) is not merged.

Here is the efficiency tables for SiSPSeededSLHCTracks and ResolvedSLHCTracks for 10k single muons with pT=1GeV.

@nstyles @gavrilen @asalzbur @npetters @tstreble @zschilla

22:56:31 |-----------------------------------------------------------------------------------|
22:56:31 |                      Statistic for SiSPSeededSLHCTracks                           |
22:56:31 |-----------------------------------------------------------------------------------|
22:56:31 |-----------------------------------------------------------------------------------|
22:56:31 | Efficiency reconstruction (number lose+wrong < 3) =   0.98160 (  0.98047 )        |
22:56:31 |                             For barrel     region =   0.99269 (  0.99269 )        |
22:56:31 |                             For transition region =   0.99328 (  0.99262 )        |
22:56:31 |                             For endcap     region =   0.98716 (  0.98716 )        |
22:56:31 |                             For forward    region =   0.96586 (  0.96312 )        |
22:56:31 |-----------------------------------------------------------------------------------|

22:56:31 |-----------------------------------------------------------------------------------|
22:56:31 |                      Statistic for ResolvedSLHCTracks                             |
22:56:31 |-----------------------------------------------------------------------------------|
22:56:31 |-----------------------------------------------------------------------------------|
22:56:31 | Efficiency reconstruction (number lose+wrong < 3) =   0.97319 (  0.97206 )        |
22:56:31 |                             For barrel     region =   0.99011 (  0.99011 )        |
22:56:31 |                             For transition region =   0.99060 (  0.98993 )        |
22:56:31 |                             For endcap     region =   0.98212 (  0.98212 )        |
22:56:31 |                             For forward    region =   0.94908 (  0.94638 )        |
22:56:31 |-----------------------------------------------------------------------------------|

This is without !27922 (merged) and this MR:

17:41:57 |-----------------------------------------------------------------------------------|
17:41:57 |                      Statistic for SiSPSeededSLHCTracks                           |
17:41:57 |-----------------------------------------------------------------------------------|
17:41:57 |-----------------------------------------------------------------------------------|
17:41:57 | Efficiency reconstruction (number lose+wrong < 3) =   0.94943 (  0.94833 )        |
17:41:57 |                             For barrel     region =   0.98925 (  0.98925 )        |
17:41:57 |                             For transition region =   0.98590 (  0.98523 )        |
17:41:57 |                             For endcap     region =   0.95140 (  0.95140 )        |
17:41:57 |                             For forward    region =   0.90640 (  0.90383 )        |
17:41:57 |-----------------------------------------------------------------------------------|

17:41:57 |-----------------------------------------------------------------------------------|
17:41:57 |                      Statistic for ResolvedSLHCTracks                             |
17:41:57 |-----------------------------------------------------------------------------------|
17:41:57 |-----------------------------------------------------------------------------------|
17:41:57 | Efficiency reconstruction (number lose+wrong < 3) =   0.93734 (  0.93625 )        |
17:41:57 |                             For barrel     region =   0.98667 (  0.98667 )        |
17:41:57 |                             For transition region =   0.98187 (  0.98121 )        |
17:41:57 |                             For endcap     region =   0.94223 (  0.94223 )        |
17:41:57 |                             For forward    region =   0.88279 (  0.88028 )        |
17:41:57 |-----------------------------------------------------------------------------------|
Edited by Noemi Calace

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
  • Thomas Strebler
  • Thomas Strebler
  • Noemi Calace added 1 commit

    added 1 commit

    • 888bddf0 - Avoiding temporary setting of private member

    Compare with previous version

  • Noemi Calace resolved all threads

    resolved all threads

  • Noemi Calace added 1 commit

    added 1 commit

    Compare with previous version

  • Noemi Calace unmarked as a Work In Progress

    unmarked as a Work In Progress

  • This merge request affects 2 packages:

    • InnerDetector/InDetRecTools/SiCombinatorialTrackFinderTool_xk
    • InnerDetector/InDetRecTools/SiSpacePointsSeedTool_xk

    Adding @goetz ,@amorley ,@sroe as watchers

  • Nicholas Styles mentioned in merge request !27922 (merged)

    mentioned in merge request !27922 (merged)

  • :white_check_mark: CI Result SUCCESS (hash 5bc68ba5)

    Athena
    externals :white_check_mark:
    cmake :white_check_mark:
    make :white_check_mark:
    required tests :white_check_mark:
    optional tests :white_check_mark:

    Full details available on this CI monitor view
    :white_check_mark: Athena: number of compilation errors 0, warnings 0
    :pencil: For experts only: Jenkins output [CI-MERGE-REQUEST 41747]

  • Please resolve the open threads and then re-add the label. Thanks, Pienpen (L1l

  • Noemi Calace resolved all threads

    resolved all threads

  • Noemi Calace added 1 commit

    added 1 commit

    Compare with previous version

  • Author Developer

    The copyright statement has been updated.

  • This merge request affects 2 packages:

    • InnerDetector/InDetRecTools/SiCombinatorialTrackFinderTool_xk
    • InnerDetector/InDetRecTools/SiSpacePointsSeedTool_xk

    Adding @goetz ,@amorley ,@sroe as watchers

  • :white_check_mark: CI Result SUCCESS (hash 01fc5ea6)

    Athena
    externals :white_check_mark:
    cmake :white_check_mark:
    make :white_check_mark:
    required tests :white_check_mark:
    optional tests :white_check_mark:

    Full details available on this CI monitor view
    :white_check_mark: Athena: number of compilation errors 0, warnings 0
    :pencil: For experts only: Jenkins output [CI-MERGE-REQUEST 41794]

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

  • Looks OK now. Approving.

    Tadej (L2)

  • Nicholas Styles mentioned in commit 5902ace4

    mentioned in commit 5902ace4

  • Please register or sign in to reply
    Loading