Implement total phase space for unfolding#661
Open
davidwalter2 wants to merge 41 commits intoWMass:mainfrom
Open
Implement total phase space for unfolding#661davidwalter2 wants to merge 41 commits intoWMass:mainfrom
davidwalter2 wants to merge 41 commits intoWMass:mainfrom
Conversation
…ants into 260216_totalPhaseSpace
… if not necessary
…s into 260216_totalPhaseSpace
…hort description of their intented usage
…n to disable these
… add option to disable these'
…city smoothing); Use corrections by default
The last bin of each axis now explicitly captures [edge, ∞) so rabbit can include it in the fit without relying on histogram overflow. - binning.get_binning_fakes_mt/relIso: append np.inf as last edge - binning.axis_relIsoCat: [0, 0.15, 0.3, inf], overflow=False - mw histmaker axis_mtCat, axis_isoCat: overflow=False - mz_wlike histmaker: replace large-number proxies (1000, 100) with np.inf - histselections: cap regressor x_max at last finite mt edge Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
- rebin_pt: recompute new_edges after trimming input when odd-edge branch fires - systematics.add_syst_hist: fix mutable default tensor_axes=[] -> None - generator_level_definitions: define postfsrPTV for wlike mode (was missing, causing RDF crash when unfolding aliases postfsrV_pt <- postfsrPTV) - unfolding_tools.UnfolderZ: use self.unfolding_levels[0] explicitly instead of relying on loop-variable persistence for the fitresult reweighting block - theory_fit_writer: raise ValueError (not silent None) when required ptV/absYV axes not found; avoid calling axis lookup twice in set_reference Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
…nants into 260216_totalPhaseSpace
Collaborator
Author
|
This one is ready for review/merge when the rabbit PR is merged WMass/rabbit#130 . The CI results are unchanged as expected for this PR: The ordering of plots has changed due to name changes in the new version of rabbit. The plot style has changed due to update of wums. AlphaS has changed due to another PR merged earlier this morning (could check ReferenceRun from tomorrow for better comparison). |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Total phase space extrapolation (main feature)
Nonprompt background (fakes)
OnesSelectorhistselection to allow fitting sideband regions simultaneousscripts/rabbit/regen_smoothing_params.pyto save fake parameters Chebyshev basis directlyOther
theory_helpers->helicity_smoothing_helpersto avoid confusion withcorr_helpersrabbitsubmodule