Skip to content
Snippets Groups Projects

23.0-debug-TrkTrack

Merged Shaun Roe requested to merge sroe/athena:23.0-debug-TrkTrack into master

A step in debugging ATLASRECTS-7335

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
  • Christos Anastopoulos
    • I think is good that GXFTrackState are moved next to the TSOS.

    • Rather than running the danger to x-include and tie in the future our selves in knots. I would have had a factory method in GXFTrackState producing TSOS. Rather that an uber special converting ctor inside the TSOS.

    • We do not want someone to wonder if the inheriting AlignTSOS or MultiComponentTSOS should implement it etc ... a GXF method returning TSOS ala a factory seemes neater .

    • I would assume a factory can return something meaningful if insane is to be created

    • For the "knots" we had the issue (if you recall) of the TrkParametersBase and Surfaces almost being cross inluded. Things depend on exact fwd declaration and re-factoring can be hard ...

    • You add a couple of utilities that I would prb have moved either under TrkEventPrimitives or TrkEventUtils . Plus their test.

    • The GXFMaterial not sure if they should go where other MaterialEffects classes are.

    Edited by Christos Anastopoulos
  • Shaun Roe added 5 commits

    added 5 commits

    Compare with previous version

  • Author Developer

    Jenkins please retry a build

  • This merge request affects 3 packages:

    • Tracking/TrkEvent/TrkEventPrimitives
    • Tracking/TrkEvent/TrkTrack
    • Tracking/TrkFitter/TrkGlobalChi2Fitter

    Affected files list will not be printed in this case

    Adding @amorley as watcher

  • :warning: WARNING: big files (>100K) are found in the changeset

    :pencil: 296K in file Tracking/TrkFitter/TrkGlobalChi2Fitter/src/GlobalChi2Fitter.cxx

  • Christos Anastopoulos resolved all threads

    resolved all threads

  • :pencil: :scissors: CI integration tests for projects AthGeneration are cancelled because of compilation error(s)

  • :x: CI Result FAILURE (hash 9fe5305f)

    Athena AthSimulation AthGeneration AnalysisBase AthAnalysis DetCommon
    externals :white_check_mark: :white_check_mark: :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: :white_check_mark: :white_check_mark:
    make :white_check_mark: :white_check_mark: :o: :white_check_mark: :white_check_mark: :white_check_mark:
    tests :white_check_mark: :white_check_mark: :white_check_mark: :white_check_mark: :white_check_mark: :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
    :white_check_mark: AthSimulation: number of compilation errors 0, warnings 0
    :o: AthGeneration: number of compilation errors 1, warnings 7
    :white_check_mark: AnalysisBase: number of compilation errors 0, warnings 0
    :white_check_mark: AthAnalysis: number of compilation errors 0, warnings 0
    :white_check_mark: DetCommon: number of compilation errors 0, warnings 0
    :pencil: For experts only: Jenkins output [CI-MERGE-REQUEST-CC7 61663]

  • Author Developer

    I set up AthGeneration; I cannot reproduce that compilation failure.

  • Shaun Roe marked this merge request as ready

    marked this merge request as ready

  • Author Developer

    Jenkins please retry a build

  • This merge request affects 3 packages:

    • Tracking/TrkEvent/TrkEventPrimitives
    • Tracking/TrkEvent/TrkTrack
    • Tracking/TrkFitter/TrkGlobalChi2Fitter

    Affected files list will not be printed in this case

    Adding @amorley as watcher

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