Jet config block fixes for R22

FTag for VR jets

The R22 calib file and settings for VR track jets are only compatible with the 'legacy' CDI file set in this MR. We have no more use for R21 recommendations. The container name hacks are also not relevant.

However, the functional CDI file refers to a track jet container name that omits 'PV0', so we need to substitute that.

Large R jets

The ConfigBlock __init__ needs to be called with the container name to pass the makeConfig assert. Evidently not being tested.

This merge request affects 1 package:

• PhysicsAnalysis/Algorithms/FTagAnalysisAlgorithms

This merge request affects 1 file:

• PhysicsAnalysis/Algorithms/FTagAnalysisAlgorithms/python/FTagAnalysisConfig.py

Adding @krumnack ,@tadej ,@pagessin as watchers

added Analysis analysis-review-required main review-pending-level-1 labels

added 1 commit

• 9e326dc5 - Correctly initialise LargeRJetAnalysisConfig

Compare with previous version

This merge request affects 2 packages:

• PhysicsAnalysis/Algorithms/FTagAnalysisAlgorithms
• PhysicsAnalysis/Algorithms/JetAnalysisAlgorithms

Affected files list will not be printed in this case

Adding @krumnack ,@tadej ,@pagessin as watchers

added JetEtmiss label

changed title from Make FTag config work for R22 VR jets to Jet config block fixes for R22

changed the description

removed JetEtmiss label

The system determined that CI tests (with names matching "^CITest_SimulationRun(2|3)(FullSim|Hit).*$") are not needed for this code change. They are not run. This is not an indicator to restart the job.

CI Result SUCCESS (hash b613637b)

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

The system determined that CI tests (with names matching "^CITest_SimulationRun(2|3)(FullSim|Hit).*$") are not needed for this code change. They are not run. This is not an indicator to restart the job.

CI Result SUCCESS (hash 9e326dc5)

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

Approved from the analysis side.

Nils [Analysis RC]

added analysis-review-approved label and removed analysis-review-required label

MR looks fine from L1 point of view. Just one small comment about a reference for a twiki in a comment. @khoo Can you clarify whether this reference is intended?

Setting review-user-action-required

Cheers, Steffen (L1)

removed review-pending-level-1 label

added review-user-action-required label

resolved all threads

added 1 commit

• 426e277f - Correct URL

Compare with previous version