Skip to content
Snippets Groups Projects

Reco sequence without UT

Merged Michel De Cian requested to merge decianm-sequenceWithoutUT into master
All threads resolved!

Reconstruction sequence without UT information.

  • Only produce Velo, Long and Seed tracks.
  • Do not add UT information to long tracks.
  • The HLT1 CPU sequence only produces Velo tracks.
  • Add two tests: hlt2_light_reco_pr_kf_without_UT_with_mcchecking (checks performance with the default ghost prob cut of 0.5) and hlt2_fastest_reco_without_UT_with_mcchecking.py (checks performance with a modified ghost prob cut of 0.8)`
Edited by Michel De Cian

Merge request reports

Merge request pipeline #3308098 passed

Merge request pipeline passed for e88fc665

Approved by

Merged by Rosen MatevRosen Matev 3 years ago (Dec 1, 2021 9:50pm UTC)

Merge details

  • Changes merged into master with ab498596.
  • Deleted the source branch.

Pipeline #3308242 passed

Pipeline passed for ab498596 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
  • Michel De Cian added 2 commits

    added 2 commits

    • c417021f - add possibility to skip UT in the fastest global reco
    • 3325153f - fix wrong includes, format

    Compare with previous version

  • Michel De Cian added 1 commit

    added 1 commit

    Compare with previous version

  • Michel De Cian added 1 commit

    added 1 commit

    • f9e8d23a - update the definitions for the HLT2 histograms

    Compare with previous version

  • Carla Marin Benito
  • Carla Marin Benito
  • Michel De Cian added 17 commits

    added 17 commits

    Compare with previous version

  • Michel De Cian added 1 commit

    added 1 commit

    • 12e63296 - unify HLT1 and HLT2 checker scripts, add a flag to distinguish

    Compare with previous version

    • Author Developer
      Resolved by Michel De Cian

      @peilian I unified the HLT1 and HLT2 PrCheckerEfficiency scripts, can you have a look and tell me if it makes sense? There is now a flag which allows to run over the HLT1 or HLT2 track types. There is a small change of functionality, but it should be an upgrade: The ghost plots are now made similarly to the efficiency plots, which made them easier to be retrieved and overlaid after saving in a file.

      Thanks!

  • Michel De Cian added 1 commit

    added 1 commit

    • 18aa9252 - remove dedicated HLT1 efficiency checker. functionality is now in 'PrCheckerEfficiency'

    Compare with previous version

  • Michel De Cian added 3 commits

    added 3 commits

    Compare with previous version

  • Michel De Cian resolved all threads

    resolved all threads

  • assigned to @lpica

  • Michel De Cian marked this merge request as ready

    marked this merge request as ready

  • assigned to @rmatev

  • Rosen Matev added 26 commits

    added 26 commits

    • 7520e14a...afa04d21 - 8 commits from branch master
    • 0f7a4d4b - add files for running without UT
    • 0d56428c - one more options file
    • 11985d0c - add test
    • 729edbb2 - add fastest test without UT, make ghost prob cut configurable
    • cf5a532a - formatting
    • 79ac8133 - add possibility to skip UT in the fastest global reco
    • 01ec771c - fix wrong includes, format
    • a0c12581 - remove unused lines
    • 75eb1f29 - update the definitions for the HLT2 histograms
    • bf3e5a65 - fix wrong TM configuration
    • f9352c9e - add fastest reco test without UT
    • efdc69d6 - unify HLT1 and HLT2 checker scripts, add a flag to distinguish
    • bd796675 - remove dedicated HLT1 efficiency checker. functionality is now in 'PrCheckerEfficiency'
    • d3aab2f2 - undo changes for test
    • 883cb27f - adapt test for new PrCheckerEfficiency script
    • c556d8f9 - fix test
    • 07ac1e73 - Update refs from lhcb-master-mr/3229
    • e88fc665 - Manually apply change like in !1191 (merged)

    Compare with previous version

  • Rosen Matev resolved all threads

    resolved all threads

  • Rosen Matev approved this merge request

    approved this merge request

  • merged

  • Rosen Matev mentioned in commit ab498596

    mentioned in commit ab498596

  • Rosen Matev mentioned in commit 1af6dfe3

    mentioned in commit 1af6dfe3

  • Rosen Matev mentioned in merge request !1198 (merged)

    mentioned in merge request !1198 (merged)

  • mentioned in issue #356 (closed)

  • Rosen Matev mentioned in commit 9bf42b7b

    mentioned in commit 9bf42b7b

  • Rosen Matev mentioned in commit 5f9c3e6d

    mentioned in commit 5f9c3e6d

  • Rosen Matev mentioned in commit 4ec491c1

    mentioned in commit 4ec491c1

  • Rosen Matev mentioned in commit d6c06040

    mentioned in commit d6c06040

  • Please register or sign in to reply
    Loading