Skip to content

Dev sf logic - #99

Merged
ludovicomori merged 34 commits into
mainfrom
dev-SF-logic
Oct 27, 2026
Merged

ludovicomori merged 34 commits into
mainfrom
dev-SF-logic

Conversation

@ludovicomori

Copy link
Copy Markdown
Collaborator

Supersedes #97 and includes a first iteration of the machinery to include the tagger SF. Some choices of implementation might not be definitive (eg what samples to apply the SF to, the correction structure in boosted hh, etc. )

Mori Ludovico and others added 30 commits August 12, 2026 13:56
…ted significantly (compare '26July10' results)
A plain .squeeze() drops axis 0 (the event axis) whenever a sample has
exactly one row, silently turning a genuine (1, num_fatjets) array into
1-D or a (1,) scalar-per-event array into a bare Python scalar -- the
latter breaks downstream boolean-mask indexing. Add
_squeeze_keep_event_axis() and use it everywhere get_var() squeezes.
Also fixes the same issue locally in derive_variables()'s Xbb/QCD
extraction via np.atleast_2d, ahead of the general fix landing.
…ivalent, not cross-normalized against the other two tau-channel scores; cross-norm formula kept commented out for later
…) from lepton content instead of the CHANNEL_ORDERING veto chain
…DeltaEta/VBFMassjj PAD_VAL handling (spurious deta=0 when 0 valid VBF jets), add to control plots (wip)
…oesn't exist for those meta-year values; sum the resolved per-year list instead)
…paration

merge new skimmer functionalities
…trigger matching for bb jets; other smaller fixes
…as we integrate the validation region logic into the core workflow

@billy000400 billy000400 left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

approve for rapid integration

@ludovicomori
ludovicomori merged commit e2c3ba8 into main Oct 27, 2026
2 of 3 checks passed
@ludovicomori
ludovicomori deleted the dev-SF-logic branch October 27, 2026 03:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants