inclusion of LArSuperCell Monitoring Algorithm for Tier0 and a small fix for LArSCSimpleMaker

Warning @pavol and @lily..

This merge request affects 2 packages:

• LArCalorimeter/LArMonitoring
• LArCalorimeter/LArROD

Affected files list will not be printed in this case

Adding @bcarlson ,@joheinri ,@pavol ,@damazio as watchers

added DQ JetEtmiss LAr master review-pending-level-1 labels

CI Result SUCCESS (hash 560b4f2f)

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

Full details available on this CI monitor view
Athena: number of compilation errors 0, warnings 1
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 40374]

added 1 commit

• 9e283e73 - fix python file and clean up a lot

Compare with previous version

This merge request affects 2 packages:

• LArCalorimeter/LArMonitoring
• LArCalorimeter/LArROD

Affected files list will not be printed in this case

Adding @bcarlson ,@joheinri ,@pavol ,@damazio as watchers

CI Result SUCCESS (hash 9e283e73)

	Athena	AthSimulation	AthGeneration	AnalysisBase	AthAnalysis	DetCommon
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
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 40394]

There are several lines of commented code, is this intended or can it be removed?

Cheers, L1

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

resolved all threads

added review-approved label and removed review-user-action-required label

added review-user-action-required label and removed review-approved label

added 1 commit

• 42fde389 - removing part of comments

Compare with previous version

This merge request affects 2 packages:

• LArCalorimeter/LArMonitoring
• LArCalorimeter/LArROD

Affected files list will not be printed in this case

Adding @bcarlson ,@joheinri ,@pavol ,@damazio as watchers

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

Documentation for the previous job was not built due to widespread network problems. The job is restarted.

Jenkins please retry a build

Jenkins please retry a build

This merge request affects 2 packages:

• LArCalorimeter/LArMonitoring
• LArCalorimeter/LArROD

Affected files list will not be printed in this case

Adding @bcarlson ,@joheinri ,@pavol ,@damazio as watchers

CI Result SUCCESS (hash 42fde389)

	Athena	AthSimulation	AthGeneration	AnalysisBase	AthAnalysis	DetCommon
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
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 40484]