Propagate new cell time cut on master

Propagating the changes to cell time cut in cluster making, already merged into r21: !36860 (merged) (sweep failed). We are also adding an option for restricting the time cut to seed cells with E/sigma smaller than a given threshold (WIP). All options are switched off by default and can be turned on at preExec.

added 2 commits

• f1dc2d59 - add info message about time cut in initialize
• a22523b1 - propagate to new style config

Compare with previous version

unmarked as a Work In Progress

added JetEtmiss alsoTargeting:21.0 master labels

This merge request affects 1 package:

• Calorimeter/CaloRec

Adding @pavol as watcher

added Calorimeter review-pending-level-1 labels

CI Result FAILURE (hash a22523b1)

	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 22371]

Test errors are unrelated, approving.

Tadej (L2)

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

merged

mentioned in commit 87a1cca0

added sweep:done label

added sweep:failed label

Sweep summary
failed:

• 21.0

Hi @mspalla, should that really be swept back to 21.0?

Tadej (L2)

Hi @tadej,

yes, we want to test the 'upper limit' option in r21, together with other time cut options which have already been tested there. Since multiple combinations of these cuts are possible, we have to select the optimal one. We went for master first because of the deadline for the feature freeze.

We were actually expecting the sweep to fail due to the different folder structure, I'll migrate it back by hand.

Best, Margherita

mentioned in merge request !38280 (merged)

removed sweep:failed label