Skip to content
Snippets Groups Projects

Call Allen from Moore

Merged Dorothea Vom Bruch requested to merge dovombru_Allen_Moore_integration into master

Adds functionality to call Allen from Moore, one event at a time. The conditions and geometry information are passed to Allen through the AllenUpdater. Raw banks are dumped to the TES in the format expected by Allen. Output from Allen is saved in the HostBuffers struct, which is also written to the TES. Then various converters convert Allen objects to baseline LHCb objects, such as tracks and vertices. These are then used for physics checks.

The following checks are possible on all tracks of all events:

  • Track efficiency (PrChecker) and PV efficiency & resolution check (PrimaryVertexChecker), using Hlt/RecoConf/options/hlt1_reco_baseline_and_Allen_with_mcchecking.py
  • Track resolution (TrackResChecker), using Hlt/RecoConf/python/RecoConf/options/hlt1_reco_allen_trackresolution.py
  • IP resolution (TrackIPResolutionCheckerNT), using Hlt/RecoConf/options/hlt1_reco_allen_IPresolution.py
  • Muon ID efficiency, using Hlt/RecoConf/options/hlt1_reco_allen_muonid_efficiency.py
  • PV efficiency & resolution will be added with !501 (closed)

Allen can also be called as a trigger. Similar to the run_moore call, run_allen is added for this purpose. The Allen updater is configured in run_allen or run_allen_reconstruction. Similarly, the HltANNSvc is configured in run_allen based on the line names stored in the Allen configuration file, created at Allen build time. Hlt/Hlt1Conf/options/allen_hlt1_pp_default.py is the options script to call the Allen trigger, which will run the full HLT1 including selections and return the HostBuffers content as well as the DecReports to TES.

Allen HLT1 trigger rates are printed in the form of counters like so:

RunAllen                            SUCCESS Number of counters : 22
 |    Counter                                      |     #     |    sum     | mean/eff^* | rms/err^*  |     min     |     max     |
 |*"Selected by Hlt1BeamOneDecision"               |        10 |          0 |( 0.000000 +- 0.000000)% |
 |*"Selected by Hlt1BeamTwoDecision"               |        10 |          0 |( 0.000000 +- 0.000000)% |
 |*"Selected by Hlt1BothBeamsDecision"             |        10 |          0 |( 0.000000 +- 0.000000)% |
 |*"Selected by Hlt1D2KKDecision"                  |        10 |          0 |( 0.000000 +- 0.000000)% |
 |*"Selected by Hlt1D2KPiDecision"                 |        10 |          0 |( 0.000000 +- 0.000000)% |
 |*"Selected by Hlt1D2PiPiDecision"                |        10 |          0 |( 0.000000 +- 0.000000)% |
 |*"Selected by Hlt1DiMuonHighMassDecision"        |        10 |          0 |( 0.000000 +- 0.000000)% |
 |*"Selected by Hlt1DiMuonLowMassDecision"         |        10 |          0 |( 0.000000 +- 0.000000)% |
 |*"Selected by Hlt1DiMuonSoftDecision"            |        10 |          0 |( 0.000000 +- 0.000000)% |
 |*"Selected by Hlt1ErrorEventDecision"            |        10 |          0 |( 0.000000 +- 0.000000)% |
 |*"Selected by Hlt1GECPassthroughDecision"        |        10 |          9 |( 90.00000 +- 9.486833)% |
 |*"Selected by Hlt1LowPtDiMuonDecision"           |        10 |          1 |( 10.00000 +- 9.486833)% |
 |*"Selected by Hlt1LowPtMuonDecision"             |        10 |          1 |( 10.00000 +- 9.486833)% |
 |*"Selected by Hlt1NoBeamsDecision"               |        10 |          0 |( 0.000000 +- 0.000000)% |
 |*"Selected by Hlt1ODINLumiDecision"              |        10 |          0 |( 0.000000 +- 0.000000)% |
 |*"Selected by Hlt1ODINNoBiasDecision"            |        10 |          0 |( 0.000000 +- 0.000000)% |
 |*"Selected by Hlt1PassThroughDecision"           |        10 |         10 |( 100.0000 +- 0.000000)% |
 |*"Selected by Hlt1SingleHighPtMuonDecision"      |        10 |          0 |( 0.000000 +- 0.000000)% |
 |*"Selected by Hlt1TrackMVADecision"              |        10 |          1 |( 10.00000 +- 9.486833)% |
 |*"Selected by Hlt1TrackMuonMVADecision"          |        10 |          0 |( 0.000000 +- 0.000000)% |
 |*"Selected by Hlt1TwoTrackMVADecision"           |        10 |          0 |( 0.000000 +- 0.000000)% |
 |*"Selected by Hlt1VeloMicroBiasDecision"         |        10 |          0 |( 0.000000 +- 0.000000)% |

Two tests are added with this MR: One for writing the Allen DecReports to an MDF file, the second to compare the DecReports from the MDF file with the counters in the log file of the test producing the MDF file.

Depends on Allen!281 (merged), Rec!1897 (merged), Allen!385 (merged), Allen!382 (merged), Allen!402 (merged), LHCb!2649 (merged), Online!388 (merged), Allen!418 (merged),

requires !574 (merged) for ref. updates.

Edited by Christopher Rob Jones

Merge request reports

Pipeline #1787068 failed

Pipeline failed for 91ac739a on dovombru_Allen_Moore_integration

Merged by Christopher Rob JonesChristopher Rob Jones 4 years ago (Jul 17, 2020 11:58am UTC)

Loading

Pipeline #1804185 failed

Pipeline failed for 540b85a8 on master

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Tomasz Wojton mentioned in merge request !388 (merged)

    mentioned in merge request !388 (merged)

  • Tomasz Wojton mentioned in merge request Rec!1925 (merged)

    mentioned in merge request Rec!1925 (merged)

  • Roel Aaij added 1 deleted label

    added 1 deleted label

  • Ross John Hunter added 32 commits

    added 32 commits

    • eaa41ce5...913fae8d - 29 commits from branch master
    • 6c8011a7 - Put Allen into a node like moore_control_flow
    • 890fb3bc - Merge branch 'master' into dovombru_Allen_Moore_integration
    • 0f4b5d8c - make_algorithm -> Algorithm

    Compare with previous version

  • added 1 commit

    • 3a663f61 - make Allen calling from Moore work after merge with Moore master and using...

    Compare with previous version

  • Dorothea Vom Bruch changed the description

    changed the description

  • added 2 commits

    • 4bd2db35 - cleanup
    • d62c2a61 - dump default raw banks, so that ODIN is dumped as well

    Compare with previous version

  • added 33 commits

    • f6f877d0 - add track and IP resolution checker
    • 750dc250 - Fixed formatting
    • 80a9a084 - modify the format
    • a05fdbe8 - remove unused lines
    • f91f4efc - add options and qmt test for track and IP resolution
    • 31c22649 - Fixed formatting
    • faf438f6 - add script for reading IPresolution tuples
    • 3942ae50 - Fixed formatting
    • 9dbb2855 - add script for reading the IPresolution tuples
    • 6326a2c8 - Apply suggestion to Hlt/RecoConf/tests/qmtest/hlt1_reco_IPresolution.qmt
    • db9c1ce4 - Apply suggestion to Hlt/RecoConf/tests/qmtest/hlt1_reco_trackresolution.qmt
    • 1059ea41 - add scripts to plot the tracking and IP resolution
    • a1a5322b - check copyright
    • bb2386bd - Fixed formatting
    • 067a8110 - add scripts for plotting tracking efficiency
    • 3bd2452d - add scripts for plotting tracking efficiency
    • af4d9e50 - Fixed formatting
    • 53010761 - fix copyrights
    • 7bbcef59 - qmtest for plotting
    • 6fd0a306 - fix a typo
    • ce57a02b - add MuonMatch category for muonID Efficiency check
    • 4f678aaf - Merge branch 'master' into peilian_Resolution_monitor
    • 513ee71f - Merge branch 'master' into peilian_Resolution_monitor
    • 7deb1cc2 - add option files for muonID efficiency check
    • fbf1554a - add MC file loction
    • a606338a - Fixed formatting
    • 54fa58c8 - add missing algorithm
    • 41911f77 - Merge branch 'peilian_muonID_efficiency' into peilian_Resolution_monitor
    • 999911ef - Fixed formatting
    • 5be41a49 - fix bug
    • c6772870 - add defaul path for utils
    • f1bf0c18 - merge with peilian_Resolution_monitor
    • 755d8aef - fix cleanup

    Compare with previous version

  • added 13 commits

    • 755d8aef...27f51d84 - 8 commits from branch master
    • aff88189 - modify muonID checker
    • 00626210 - Fixed formatting
    • 61c2ad0e - Merge branch 'peilian_Resolution_monitor' into dovombru_Allen_Moore_integration
    • 7543bc13 - Merge branch 'master' into dovombru_Allen_Moore_integration
    • f9988186 - add RecoConf options script for muonid checking of Allen tracks

    Compare with previous version

  • Roel Aaij added 1 commit

    added 1 commit

    • 9408546a - Decode only HLT1 DecReports.

    Compare with previous version

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