Skip to content
Snippets Groups Projects

Include TrackExtension in FPGA sim tests

Merged Tomasz Bold requested to merge tbold/athena:track-ext-test into main
All threads resolved!

This MR makes use of the TrackExtension alg on the output on FPGA sim output. At this stage the algorithm is correctly scheduled and loads the prototracks and sends them to CKF. No output tracks are produced yet (to be debugged in followup MRs).

Tagging @piazza @jahreda @imaznas @sabidi

Edited by Tomasz Bold

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
  • Ioannis Maznas
  • Tomasz Bold added 1 commit

    added 1 commit

    Compare with previous version

  • Hi @tbold, I see a conflict in Tracking/Acts/ActsTrackReconstruction/src/TrackExtensionAlg.cxx file.

    Please fix this.

    L1.

  • Tomasz Bold resolved all threads

    resolved all threads

  • Tomasz Bold added 47 commits

    added 47 commits

    Compare with previous version

  • This merge request affects 3 packages:

    • Tracking/Acts/ActsTrackReconstruction
    • Trigger/EFTracking/FPGATrackSim/FPGATrackSimConfTools
    • Trigger/EFTracking/FPGATrackSim/FPGATrackSimPrototrackFitter

    Affected files list will not be printed in this case

    Adding @cvarni ,@jojungge ,@tbold ,@pagessin ,@goetz ,@toyamaza ,@adye as watchers

  • :white_check_mark: CI Result SUCCESS (hash dd8c27b0)

    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 13574] (remote access info)

  • Changes look fine to me, approving.

    Cheers, Philipp (L1)

  • Edward Moyse mentioned in commit d8c35966

    mentioned in commit d8c35966

  • merged

  • Please register or sign in to reply
    Loading