BTaggingSV1Flip_forHL

Implement missing variables of Flip version of the SV1 b-tagger needed for high level flipped b-taggers.

requested review from @duperrin, @vdao, @dguest, and @thuffman

marked this merge request as draft

This merge request affects 3 packages:

• PhysicsAnalysis/JetTagging/JetTagAlgs/BTagging
• PhysicsAnalysis/JetTagging/JetTagConfig
• PhysicsAnalysis/JetTagging/JetTagTools

Affected files list will not be printed in this case

Adding @duperrin ,@dguest ,@lidiaz as watchers

added 22.0 Analysis BTagging Derivation JetEtmiss review-pending-level-1 labels

CI Result SUCCESS (hash acd4334e)

	Athena	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
DetCommon: number of compilation errors 0, warnings 0
For experts only: Jenkins output [CI-MERGE-REQUEST-CC7 52973]

marked this merge request as ready

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

Changes and CI look fine. Approving.

Kira (L1)

added review-approved-point1 label

hi @anburger, could you please just clarify:

• that there is indeed a good reason to have this in 22.0 (is it one of the inputs to the taggers you're monitoring?)
• whether it will change current AOD output?

I do not recall this being mentioned explictly in Arnaud's list at a recent RIG meeting (slides), hence my questions.

Thanks!

Hi @jmaurer,

Thanks. Maybe just to complement what Dan said, as mentioned in the slides you have linked, FTAG is not saved in AOD so we have no risk of changing the output :) . It's indeed not a tagger we monitor but the synchronization between master and 22.0 is important to us.

Hope this helps.

Cheers, Arnaud

resolved all threads

added review-approved-tier0 label

mentioned in commit 2d4480fc

merged

mentioned in merge request !53488 (closed)