Add hooks for DQ histogram creation to CA Reco_tf configuration

Run DQ in CA Reco_tf if outputHISTFile is specified

Touches a number of packages, comments welcome

This is for ATLASRECTS-7068

resolved all threads

marked this merge request as ready

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

140K in file InnerDetector/InDetMonitoring/TRTMonitoringRun3/src/TRTMonitoringRun3RAW_Alg.cxx

This merge request affects 10 packages:

• Control/AthenaMonitoring
• InnerDetector/InDetMonitoring/InDetAlignmentMonitoringRun3
• InnerDetector/InDetMonitoring/InDetGlobalMonitoringRun3Test
• InnerDetector/InDetMonitoring/PixelMonitoring
• InnerDetector/InDetMonitoring/SCT_Monitoring
• InnerDetector/InDetMonitoring/TRTMonitoringRun3
• LArCalorimeter/LArConfiguration
• Reconstruction/RecJobTransforms
• TileCalorimeter/TileMonitoring
• Trigger/TrigT1/TrigT1CaloMonitoring

Affected files list will not be printed in this case

Adding @goetz ,@cgrefe ,@kzoch ,@ibragimo ,@calfayan ,@pavol ,@battagl ,@sroe ,@harkusha ,@solodkov ,@ssnyder ,@jajimene ,@stsuno ,@thompson ,@lshan ,@martis ,@afaulkne as watchers

added Core DQ InnerDetector JetEtmiss LAr Reconstruction Tile Trigger master review-pending-level-1 labels

CI Result FAILURE

	Athena	AthSimulation	AthGeneration	AnalysisBase	AthAnalysis	DetCommon
externals
cmake	N/A	N/A	N/A	N/A	N/A	N/A
make	N/A	N/A	N/A	N/A	N/A	N/A
required tests	N/A	N/A	N/A	N/A	N/A	N/A
optional tests	N/A	N/A	N/A	N/A	N/A	N/A

Due to problems in externals build or cmake configuration the job is stopped, results are not available on the ATLAS CI monitor 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
AthAnalysis: number of compilation errors 0, warnings 0
DetCommon: number of compilation errors 0, warnings 0
For experts only: Jenkins output [CI-MERGE-REQUEST-CC7 53398]

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

I doubt that the CI failure is related to the changes here, so I will restart it. However, I'm actually wondering if you are targeting the correct branch. If this MR updates the Run3 reconstruction without changing the output it should go into 22.0 .So maybe it's best I let you comment and restart the CI after this is resolved to not waste CI power.

Kira (L1)

resolved all threads

added alsoTargeting:22.0 label

added 494 commits

• 7da1a630...41f0f265 - 490 commits from branch atlas:master
• ac1d8ccf - Add hooks for DQ histogram creation to CA Reco_tf configuration
• d9fca293 - Trailing whitespace fix
• da0d1c11 - Add perfmon domain
• fb468882 - Merge branch 'master-dq-ca-config' of...

Compare with previous version

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

140K in file InnerDetector/InDetMonitoring/TRTMonitoringRun3/src/TRTMonitoringRun3RAW_Alg.cxx

This merge request affects 10 packages:

• Control/AthenaMonitoring
• InnerDetector/InDetMonitoring/InDetAlignmentMonitoringRun3
• InnerDetector/InDetMonitoring/InDetGlobalMonitoringRun3Test
• InnerDetector/InDetMonitoring/PixelMonitoring
• InnerDetector/InDetMonitoring/SCT_Monitoring
• InnerDetector/InDetMonitoring/TRTMonitoringRun3
• LArCalorimeter/LArConfiguration
• Reconstruction/RecJobTransforms
• TileCalorimeter/TileMonitoring
• Trigger/TrigT1/TrigT1CaloMonitoring

Affected files list will not be printed in this case

Adding @goetz ,@cgrefe ,@kzoch ,@ibragimo ,@calfayan ,@pavol ,@battagl ,@sroe ,@harkusha ,@solodkov ,@ssnyder ,@jajimene ,@stsuno ,@thompson ,@lshan ,@martis ,@afaulkne as watchers

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

added NewConfig label

CI Result FAILURE (hash fb468882)

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

added 1 commit

• c33df761 - DQ steering flag off by default

Compare with previous version