Skip to content
Snippets Groups Projects

Connect Flow Elements to Muon container

Hi all (cc @mhodgkin )

This merge request covers new particle flow objects known as Flow Elements (in the updated EDM) and the linkage of these objects to Muons. This MR only couples charged flow elements to muons via track index matching.

Thanks

Matt

Edited by Matthew Thomas Anthony

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
  • added 1 commit

    • 6efa9030 - add handles for neutral muons

    Compare with previous version

  • added 1 commit

    Compare with previous version

  • added 1 commit

    Compare with previous version

  • added 1382 commits

    Compare with previous version

  • added 1 commit

    • 7e5cc31a - Made sure Gaudi Bool works for neutral flowelems works properly

    Compare with previous version

  • added 1 commit

    • 44a32f83 - Added Readhandle configuration to PFMuonFlowElementAssoc

    Compare with previous version

  • added 1 commit

    Compare with previous version

  • Matthew Thomas Anthony unmarked as a Work In Progress

    unmarked as a Work In Progress

  • This merge request affects 1 package:

    • Reconstruction/eflowRec

    Adding @goetz ,@mhodgkin as watchers

  • :negative_squared_cross_mark: CI Result FAILURE (hash bb57c065)

    Athena AthSimulation AthGeneration AnalysisBase
    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 :white_check_mark: :white_check_mark: :white_check_mark: :white_check_mark:
    required tests :o: :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
    :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
    :pencil: For experts only: Jenkins output [CI-MERGE-REQUEST-CC7 20959]

  • The tests are failing due to errors in the code:

    RAWtoESD 18:29:07 
    RAWtoESD 18:29:07 AttributeError: '<class 'eflowRec.eflowRecConf.PFMuonFlowElementAssoc'>' object has no attribute 'm_LinkNeutralFEClusters'
    RAWtoESD 18:29:07 

    There are also unresolved threads. Passing back to the user.

    Cheers,

    Michele (L1 reviewer)

  • added 2 commits

    • cccf1c55 - added a topocluster link to muons
    • b9a51a68 - add Gaudi switch for the topocluster matching

    Compare with previous version

  • Matthew Thomas Anthony resolved all threads

    resolved all threads

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