JetForwardJvtTool: Add flag to read in instead of calculate fJVT

This allows the JetForwardJvtTool to be used on DAOD's in release 22 where Track MET is not stored but the fJVT for primary vertex is.

This merge request affects 1 package:

• Reconstruction/Jet/JetMomentTools

This merge request affects 2 files:

• Reconstruction/Jet/JetMomentTools/JetMomentTools/JetForwardJvtTool.h
• Reconstruction/Jet/JetMomentTools/Root/JetForwardJvtTool.cxx

Adding @goetz as watcher

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

CI Result SUCCESS (hash c64b512d)

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

Thanks @amorley !

@cdelitzs @schaarsc @sawyer @wbalunas @camacho @hpacey @miholzbo are the relevant JETM folks

I /think/ we would actually prefer to switch the default to use the value from the DAOD, if present. Are there any objections tot hat?

Approving from L1 side

-- L1

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

@mswiatlo no problem just let me know what you would prefer. I left as it was to not change the default behaviour just incase there where clients using it to calculate fJVT.

If the default uses the value from DAOD, then the tool will crash when it's used to create the DAOD in the first place (unless it's specifically configured there to "re"-compute fJVT, which is an option). The same getFJVT(jet) function call is used by the decorate(...) function, which is what gets called in the derivation framework. Either setup will work, we just need to make sure that whichever place doesn't want to use the default has it correctly configured.

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

mentioned in commit f47dfa3c

merged

added sweep:ignore label

mentioned in merge request !47820 (merged)