Skip to content

Fixes for fJVT implementation #1570

@jdandoy

Description

@jdandoy
  • fJVT current uses CentralMaxPt, but should use ForwardMaxPt.
  • It's not clear that JetForwardJvtTool is correct for PFlow jets, as there's also a JetForwardPFlowJvtTool. Currently following up with the ETmiss + pileup tagging subgroup. JetForwardJvtTool is the correct tool, JetForwardPFlowJvtTool performs a different task at the derivation stage.
  • May want the option to use the derivation-level decoration, though note this may not have the correct pt threshold set, requiring an additional check before using.
  • The fJVT tool currently doesn't use the correct maximum pt threshold, so we should set it correctly as a default option here. The R21.2 instance does set it correctly to 60 GeV.
  • Could remove the hardcoding of the JVT likelihood file in anticipation of future updates.

Metadata

Metadata

Assignees

Labels

No labels
No labels

Type

No type

Projects

No projects

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions