Enable scheduling of reco jets in IDPVM standalone

This MR enables the scheduling of reco small-R jet finding in standalone IDPVM jobs if these jets are required by the IDPVM run (e.g. sum(pt,w) ) and not present. Based on work by @tstreble.

May need some cleaning up / tuning. CC @gfrattar @pbhattar @kostyuk @keli @tstreble @vcairo @gfacini @sroe @goetz in case of comments.

added InnerDetector master labels

added 1 commit

• 25ddf1f9 - update The Copyright Message to avoid L1 trouble

Compare with previous version

added 165 commits

• 25ddf1f9...0e21d686 - 164 commits from branch atlas:master
• 4011fc7c - Merge remote-tracking branch 'upstream/master' into AddRecoJetsIDPVM

Compare with previous version

now I got philosopher Alfred Ewing but still no JetEtMiss input. Re-pinging @mswiatlo @camacho

added 3 commits

• 97cfaad6 - back to EMTopo
• d444d6e1 - typo
• 9566cf31 - copyright

Compare with previous version

resolved all threads

marked this merge request as ready

This merge request affects 2 packages:

• InnerDetector/InDetRecTools/InDetHardScatterSelectionTool
• InnerDetector/InDetValidation/InDetPhysValMonitoring

Affected files list will not be printed in this case

Adding @goetz ,@amorley ,@sroe as watchers

added JetEtmiss review-pending-level-1 labels

Reverted to AntiKt4EMTopoJets after consultation with experts (thanks @mswiatlo for the useful feedback!), solved the philosopher, this MR is ready to go

CI Result SUCCESS (hash 9566cf31)

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

Amazing stuff. Thank you @goblirsc

the code looks fine from L1 perspective. Once the above threads are resolved, it should be approved.

Roman(L1)

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

added 1 commit

• 61c790e9 - trying to satisfy jenkins. Delaying a MR 24h on a copyright message seems...

Compare with previous version

This merge request affects 2 packages:

• InnerDetector/InDetRecTools/InDetHardScatterSelectionTool
• InnerDetector/InDetValidation/InDetPhysValMonitoring

Affected files list will not be printed in this case

Adding @goetz ,@amorley ,@sroe as watchers

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

added 1 commit

• 9798c190 - remove a double-def

Compare with previous version

This merge request affects 2 packages:

• InnerDetector/InDetRecTools/InDetHardScatterSelectionTool
• InnerDetector/InDetValidation/InDetPhysValMonitoring

Affected files list will not be printed in this case

Adding @goetz ,@amorley ,@sroe as watchers

CI Result SUCCESS (hash 9798c190)

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

CI Result SUCCESS (hash 61c790e9)

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

resolved all threads

based on above and CI passed, approving.

Roman(L1)

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

merged

mentioned in commit c0de75f0

added sweep:ignore label