Skip to content
Snippets Groups Projects

Reducing MinHits for FTF trigger ITk implementation

Merged Elliot Watton requested to merge ewatton/athena:ewatton_TrigITkFTF_MinHits into 24.0
All threads resolved!

This changes FTF to process RoIs which have at least 3 spacepoints for the trigger implementation for running on an ITk dataset. This is to match that for running FTF as an offline algorithm in TrigFastTrackFinder/ITkFastTrackFinderStandaloneConfig. This is seen to improve the efficiency in the barrel region for muon RoIs when running locally on a smaller number of events compared to the ITk tests in TrigInDetValidation. This should not affect Run 3 due to the condition based on ITk geometry.

Targetting 24.0 to ensure that trigger packages don't get out of step. This change will eventually be swept to main.

Tagging @baines, @sutt, @jmasik

Edited by Elliot Watton

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
  • Mark Sutton resolved all threads

    resolved all threads

  • :white_check_mark: CI Result SUCCESS (hash d29292e5)

    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 5147]

  • Looks good to me :smile:. L1

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

  • mentioned in commit a66c4fe2

  • Tadej Novak mentioned in merge request !69214 (merged)

    mentioned in merge request !69214 (merged)

  • Please register or sign in to reply
    Loading