Skip to content
Snippets Groups Projects

Add B Hadron Plots to IDPVM

Merged Samuel Van Stroud requested to merge (removed):master-idvpm-truth-origin into master
All threads resolved!

Make and fill plots for tracks matched to B hadron decay products.

  • A flag doTruthOriginPlots is added to turn off/on the changes (default is off). Please note currently plots are booked even with the flag unset.
  • B track definition comes from the InDetTrackTruthOriginTool (a small change was maade to the tool to make the isFrom method public.

Plots presented in CTIDE: https://indico.cern.ch/event/1001254/#3-indetphysval-bs

@goblirsc @vcairo @gfacini @battagl

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
  • Samuel Van Stroud added 3780 commits

    added 3780 commits

    • 0ab0ebbe...950e0308 - 3779 commits from branch atlas:master
    • 11f971b8 - Merge remote-tracking branch 'upstream/master' into master-idvpm-truth-origin

    Compare with previous version

  • This merge request affects 2 packages:

    • InnerDetector/InDetValidation/InDetPhysValMonitoring
    • PhysicsAnalysis/TrackingID/InDetTrackSystematicsTools

    Affected files list will not be printed in this case

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

  • :white_check_mark: CI Result SUCCESS (hash 11f971b8)

    Athena AthSimulation AthGeneration AnalysisBase AthAnalysis
    externals :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:
    make :white_check_mark: :white_check_mark: :white_check_mark: :white_check_mark: :white_check_mark:
    required tests :white_check_mark: :white_check_mark: :white_check_mark: :white_check_mark: :white_check_mark:
    optional tests :cloud: :white_check_mark: :white_check_mark: :white_check_mark: :white_check_mark:

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

  • added 1 commit

    • c242a107 - imporve setup of InDetTrackTruthOriginTool

    Compare with previous version

  • This merge request affects 2 packages:

    • InnerDetector/InDetValidation/InDetPhysValMonitoring
    • PhysicsAnalysis/TrackingID/InDetTrackSystematicsTools

    Affected files list will not be printed in this case

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

  • added 1 commit

    Compare with previous version

  • This merge request affects 2 packages:

    • InnerDetector/InDetValidation/InDetPhysValMonitoring
    • PhysicsAnalysis/TrackingID/InDetTrackSystematicsTools

    Affected files list will not be printed in this case

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

  • :white_check_mark: CI Result SUCCESS (hash c69d48a0)

    Athena AthSimulation AthGeneration AnalysisBase AthAnalysis
    externals :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:
    make :white_check_mark: :white_check_mark: :white_check_mark: :white_check_mark: :white_check_mark:
    required tests :white_check_mark: :white_check_mark: :white_check_mark: :white_check_mark: :white_check_mark:
    optional tests :cloud: :white_check_mark: :white_check_mark: :white_check_mark: :white_check_mark:

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

  • :white_check_mark: CI Result SUCCESS (hash c242a107)

    Athena AthSimulation AthGeneration AnalysisBase AthAnalysis
    externals :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:
    make :white_check_mark: :white_check_mark: :white_check_mark: :white_check_mark: :white_check_mark:
    required tests :white_check_mark: :white_check_mark: :white_check_mark: :white_check_mark: :white_check_mark:
    optional tests :cloud: :white_check_mark: :white_check_mark: :white_check_mark: :white_check_mark:

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

  • Hi @svanstro, it looks like you've addressed the comments, but make sure you click on "Resolve thread" for each of them so we know this is ready to be reviewed. Tom - L1 shifter

  • Samuel Van Stroud resolved all threads

    resolved all threads

  • Thanks @tneep, I wasn't sure whether to leave that to Max, but I have resolved these now.

  • mentioned in commit 14b8540e

  • Please register or sign in to reply
    Loading