Skip to content
Snippets Groups Projects

22.0-migration_gavrilenko-SiSPSeededTrackFinder

All threads resolved!

This is a migration of work undertaken by @gavrilen to attempt to speed up the SiSPSeededTrackFinder. Pinging also @ncalace. ATLIDTRKCP-250 Created to follow discussion and track supporting documents; discussion there indicates some ~20% speedup with little (0.13%, with pileup 40, 0.26% with pileup 60) efficiency loss. We will watch the SPOT plots, assuming this goes in, to track the CPU changes.

Edited by Shaun Roe

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
  • Shaun Roe changed the description

    changed the description

  • Shaun Roe resolved all threads

    resolved all threads

  • Shaun Roe changed the description

    changed the description

  • Shaun Roe unmarked as a Work In Progress

    unmarked as a Work In Progress

  • This merge request affects 3 packages:

    • InnerDetector/InDetRecAlgs/SiSPSeededTrackFinder
    • InnerDetector/InDetRecTools/SiSpacePointsSeedTool_xk
    • InnerDetector/InDetRecTools/SiTrackMakerTool_xk

    Adding @goetz ,@amorley ,@gavrilen ,@sroe as watchers

  • :white_check_mark: CI Result SUCCESS (hash 619b7d26)

    Athena AthSimulation AnalysisBase AthGeneration
    externals :white_check_mark: :white_check_mark: :white_check_mark: :white_check_mark:
    cmake :white_check_mark: :white_check_mark: :white_check_mark: :white_check_mark:
    make :warning: :white_check_mark: :white_check_mark: :white_check_mark:
    required tests :white_check_mark: :white_check_mark: :white_check_mark: :white_check_mark:
    optional tests :white_check_mark: :white_check_mark: :white_check_mark: :white_check_mark:

    Full details available on this CI monitor view
    :warning: Athena: number of compilation errors 0, warnings 5
    :white_check_mark: AthSimulation: number of compilation errors 0, warnings 0
    :white_check_mark: AnalysisBase: number of compilation errors 0, warnings 0
    :white_check_mark: AthGeneration: number of compilation errors 0, warnings 0
    :pencil: For experts only: Jenkins output [CI-MERGE-REQUEST-CC7 11942]

  • Author Developer

    warnings are unrelated to this package

  • I agree, warnings are unrelated, changes looks fine, so approving.

    Pavol [as L2 MR shifter]

  • merged

  • Walter Lampl mentioned in commit 2bae2c94

    mentioned in commit 2bae2c94

  • mentioned in merge request !33435 (merged)

  • Please register or sign in to reply
    Loading