change handling of JVT in CP algorithms (ATLASG-2447)

There was a difference between sequence and block configuration in the nightlies, which caused the test failures in the nightlies. I changed a number of minor things about the jvt configuration:

• add SuppressInputDependence flag to JetVertexTaggerTool
• disable runJvtUpdate for the test sequence
• don't include the JVT selection in the global jet selection, but introduce a selection named JVT.

The last is indeed what fixed the discrepancy, though I'm unsure if this is the right thing to do, or if instead I ought to have a baseline for all jets that includes JVT. I leave that for the jet experts.

Also reordered configuration blocks to match configuration sequences. This is mostly meant to allow easier comparisons of the generated sequence to spot differences more easily.

Build area was cleaned as per request posted in the DB. The full software build will be performed

This merge request affects 2 packages:

• PhysicsAnalysis/Algorithms/AnalysisAlgorithmsConfig
• PhysicsAnalysis/Algorithms/JetAnalysisAlgorithms

Affected files list will not be printed in this case

Adding @krumnack ,@tadej ,@pagessin as watchers

added Analysis JetEtmiss analysis-review-required master review-pending-level-1 labels

added bugfix urgent labels

Marking this as urgent, as it address a test failure in the nightlies.

added 8 commits

• 2dab36bd...d65af973 - 4 commits from branch atlas:master
• e94df681 - reorder configuration blocks to match configuration sequences
• 2d622415 - add SuppressInputDependence flag to JetVertexTaggerTool
• f35ee036 - set runJvtUpdate to False for the test sequence
• 7a2a69b4 - move JVT selection into its own selection flag (solves ATLASG-2447)

Compare with previous version

This merge request affects 2 packages:

• PhysicsAnalysis/Algorithms/AnalysisAlgorithmsConfig
• PhysicsAnalysis/Algorithms/JetAnalysisAlgorithms

Affected files list will not be printed in this case

Adding @krumnack ,@tadej ,@pagessin as watchers

changed title from add SuppressInputDependence flag to JetVertexTaggerTool (ATLASG-2447) to change handling of JVT in CP algorithms (ATLASG-2447)

changed the description

added alsoTargeting:21.2 label

CI Result FAILURE (hash 2dab36bd)

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

CI Result SUCCESS (hash 7a2a69b4)

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

resolved all threads

removed review-pending-level-1 label

removed analysis-review-required label

added analysis-review-approved analysis-review-required review-approved labels

removed analysis-review-required label

Approving as CP algs expert.

merged

mentioned in commit c0012012

added sweep:done label

added sweep:failed label

Sweep summary
failed:

• 21.2

mentioned in merge request !60046 (closed)

removed alsoTargeting:21.2 label

removed sweep:failed label