Skip to content
Snippets Groups Projects

Detailed debug printout for ActsTrkFinding

Merged Tim Adye requested to merge adye/athena:adye-ckf4 into master
  • If Acts.doPrintTrackStates, ActsTrkFinding prints tables of input measurements/seeds and output track parameters/states.
  • Specify Acts::Logger for CKF
  • Harmonise ActsTrkFinding formatting (basic clang-format)
  • Split ActsMeasurementCheck from ActsATLASConverterTool::ATLASMeasurementToSourceLink
  • Split ActsTrackParameterCheck from ActsATLASConverterTool::ActsTrackParameterToATLAS

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
  • Carlo Varni
  • Waiting for !60352 (merged) to be merged. Otherwise looks good. Karolos (L2)

  • !60352 (merged) has been merged. @adye maybe resolve the thread and rerun the CI ? David as L2

  • Tim Adye
  • Tim Adye resolved all threads

    resolved all threads

  • Author Developer

    Jenkins please retry a build

  • This merge request affects 4 packages:

    • Tracking/Acts/ActsGeometry
    • Tracking/Acts/ActsInterop
    • Tracking/Acts/ActsTrkAlgs/ActsTrkFinding
    • Tracking/Acts/ActsTrkTools/ActsTrkFindingTool

    Affected files list will not be printed in this case

    Adding @tbold ,@amorley ,@toyamaza ,@cvarni ,@pagessin as watchers

  • Carlo Varni resolved all threads

    resolved all threads

  • CI Result SUCCESS (hash f4075a80)

    Athena
    externals
    cmake
    make
    tests

    Full details available on this CI monitor view. Check the JIRA CI status board for known problems
    Athena: number of compilation errors 0, warnings 0
    📝 For experts only: Jenkins output [CI-MERGE-REQUEST-CC7 65882]

  • Looks good, approving (L1)

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