Skip to content

Dev gen info in sr - #95

Merged
billy000400 merged 4 commits into
mainfrom
dev-GenInfoInSR
Jul 30, 2026
Merged

Dev gen info in sr#95
billy000400 merged 4 commits into
mainfrom
dev-GenInfoInSR

Conversation

@ludovicomori

Copy link
Copy Markdown
Collaborator

Overrides #94 . Takes #94 and adds fixes on the signal splitting logic across channels. Now in templates and optimization signal is not gen-split (it is in BDT training or other studies). Kept a split version of histograms to be able to plot split templates and compare shapes. Other minor fixes, including new logic in SensitivityStudy.py to add a fixed GloParT tautau cut on top of BDT and GloParT bb cut.

Mori Ludovico added 4 commits July 9, 2026 07:30
…ckend fix

Postprocessing / control plots:
- singleVarHist: optional `transform` callable applied to raw values before
  filling (axis must already span the transformed range).
- control_plots: display flagged discriminator-score vars under
  x -> -log(1 - x + eps) (TRANSFORM_VARS / TRANSFORM_EPS), stretching the
  pile-up near 1. ratioHistPlot just renders the hist's axis, so the transform
  is done at fill time.
- --sr flag for the --control-plots pipeline: apply the signal-region pass cut
  (resolved from --bmin / --sensitivity-dir via Regions.get_selection_regions)
  before plotting.
Control-region closure (new control_region_closure.py):
- Closure test for the data-driven QCD+DY estimate using the --control-region
  templates; now renders each MC background as its own stacked sample (reusing
  plotting.ratioHistPlot) plus the data-driven QCD+DY and a data/pred ratio.
- Includes the --control-region template plumbing it depends on
  (control_region kwarg through main/get_templates/parse_args, "control"
  template subdir).
Notebook fix:
- bdt_utils only forces the Agg backend when an interactive (module://...)
  backend is not already active, so importing it from a notebook no longer
  silently breaks plt.show().
Cleanup:
- Move personal scratch (notes/, paper helpers, inspector.ipynb, etc.) into
  untracked_utils/ and gitignore it.
…need to do full scan, should change default signals arg to SIGNALS

@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.

Looks good to me

@billy000400
billy000400 merged commit b4bb3e0 into main Jul 30, 2026
3 checks passed
@ludovicomori
ludovicomori deleted the dev-GenInfoInSR branch August 25, 2026 17:09
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