Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add stat uncertainty and rebin options in plotter #269

Merged
merged 3 commits into from Apr 18, 2023

Conversation

jeyserma
Copy link
Contributor

@jeyserma jeyserma commented Apr 3, 2023

Added options to plot the stat uncertainty (sig, bkg, sig+bkg) and optional uniform rebinning

config/doPlots.py Outdated Show resolved Hide resolved
@kjvbrt
Copy link
Contributor

kjvbrt commented Apr 4, 2023

Hello @jeyserma,

do you have any example plot?

@jeyserma
Copy link
Contributor Author

jeyserma commented Apr 5, 2023

An example of using the stat unc and rebinning is given in examples/FCCee/higgs/mH-recoil/mumu/analysis_plots.py:

plotStatUnc = True
rebin = [1, 1, 1, 1, 2] # uniform rebin per variable (optional)

@kjvbrt
Copy link
Contributor

kjvbrt commented Apr 12, 2023

OK, I generated it :)

I find it a bit hard to read, maybe increase the pitch of the diagonal lines or use just solid grey?

@BrieucF What do you think?

leptonic_recoil_m_zoom2_ZH_stack_lin

@BrieucF
Copy link
Contributor

BrieucF commented Apr 12, 2023

Yes, the example I pointed in the FCCSW forum thread [1] is obtained with something like:

stat_and_syst->SetMarkerSize(0);
stat_and_syst->SetMarkerStyle(0);
stat_and_syst->SetFillStyle(3154);
stat_and_syst->SetFillColor(42);
stat_and_syst->Draw("E2 same");

Some inspiration could probably be taken from [2].

[1] https://fccsw-forum.web.cern.ch/t/histogram-uncertainties/154/2
[2] https://github.com/cp3-llbb/plotIt/blob/master/src/TH1Plotter.cc#L600-L608

@jeyserma
Copy link
Contributor Author

Updated the hatch spacing. The PNG is always horrible, but PDF looks good.

leptonic_recoil_m_zoom2_ZH_stack_lin

@kjvbrt
Copy link
Contributor

kjvbrt commented Apr 18, 2023

looks good :)

@kjvbrt kjvbrt merged commit 3cb2a70 into HEP-FCC:master Apr 18, 2023
3 checks passed
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.

None yet

3 participants