Dev/add calib hits flow elements

Hi all (cc @mhodgkin)

Adds the truth calibration Hits map to the Flow Elements following the same logic as that for PFOs.

Since we needed also to ensure that the flow elements are actually filled/available when we do this linkage, I've included a Gaudi option to enable the calibhits mapping to flow elements. This option is only set if the job configuration includes FlowElements and by default is set to false.

Thanks

Matt

added 1 commit

• bede2c5a - update copyrights to date

Compare with previous version

This merge request affects 1 package:

• Reconstruction/PFlow/PFlowUtils

This merge request affects 3 files:

• Reconstruction/PFlow/PFlowUtils/share/PFlowCalibHitDecorator_algorithms.py
• Reconstruction/PFlow/PFlowUtils/src/PFlowCalibPFODecoratorAlgorithm.cxx
• Reconstruction/PFlow/PFlowUtils/src/PFlowCalibPFODecoratorAlgorithm.h

Adding @goetz ,@mhodgkin as watchers

added JetEtmiss Reconstruction master review-pending-level-1 labels

This merge request affects 1 package:

• Reconstruction/PFlow/PFlowUtils

This merge request affects 3 files:

• Reconstruction/PFlow/PFlowUtils/share/PFlowCalibHitDecorator_algorithms.py
• Reconstruction/PFlow/PFlowUtils/src/PFlowCalibPFODecoratorAlgorithm.cxx
• Reconstruction/PFlow/PFlowUtils/src/PFlowCalibPFODecoratorAlgorithm.h

Adding @goetz ,@mhodgkin as watchers

CI Result SUCCESS (hash 2a9a794b)

	Athena	AthSimulation	AthGeneration	AnalysisBase
externals
cmake
make
required tests
optional tests

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

CI Result SUCCESS (hash bede2c5a)

	Athena	AthSimulation	AthGeneration	AnalysisBase
externals
cmake
make
required tests
optional tests

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

mentioned in merge request !39102 (merged)

Looks fine to me but a bit of formatting to comply with the c++ style guide would be appreciated if possible. Tom - L1 shifter

added review-user-action-required label and removed review-pending-level-1 label

added 1 commit

• 0b73f006 - reduce character length of long arguments

Compare with previous version

This merge request affects 1 package:

• Reconstruction/PFlow/PFlowUtils

This merge request affects 3 files:

• Reconstruction/PFlow/PFlowUtils/share/PFlowCalibHitDecorator_algorithms.py
• Reconstruction/PFlow/PFlowUtils/src/PFlowCalibPFODecoratorAlgorithm.cxx
• Reconstruction/PFlow/PFlowUtils/src/PFlowCalibPFODecoratorAlgorithm.h

Adding @goetz ,@mhodgkin as watchers

added review-pending-level-1 label and removed review-user-action-required label

added 1 commit

• d096764f - fix alias

Compare with previous version

This merge request affects 1 package:

• Reconstruction/PFlow/PFlowUtils

This merge request affects 3 files:

• Reconstruction/PFlow/PFlowUtils/share/PFlowCalibHitDecorator_algorithms.py
• Reconstruction/PFlow/PFlowUtils/src/PFlowCalibPFODecoratorAlgorithm.cxx
• Reconstruction/PFlow/PFlowUtils/src/PFlowCalibPFODecoratorAlgorithm.h

Adding @goetz ,@mhodgkin as watchers

CI Result SUCCESS (hash 0b73f006)

	Athena	AthSimulation	AthGeneration	AnalysisBase
externals
cmake
make
required tests
optional tests

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

CI Result SUCCESS (hash d096764f)

	Athena	AthSimulation	AthGeneration	AnalysisBase
externals
cmake
make
required tests
optional tests

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

resolved all threads

added review-approved label and removed review-pending-level-1 label