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

Null poissonian #67

Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

Conversation

DBAnthony
Copy link
Contributor

@DBAnthony DBAnthony commented Jul 9, 2021

The minimum error of any bin in n should be 1.15, which is the poissonian error for 0 counts. We don't want this error to be plotted for every bin in our standard plots, but it can be turned on in the config with:
is_null_poissonian: True.

We won't want this for standard plots like below, but is useful for e.g. mountain range plots, or fit-binned plots. Obviously better solution would be explicitly calculating the poisson error and plotting asymmetric errors, but this is better than nothing. I'm taking the quad sum of 1.15 and the 'default' error so that the error is > 1.15 for nonzero yields. Ideally would find a way to ignore the underflow bin.

I'm also not sure whether this minimum value should be applied to monte-carlo, as I am currently, or just data.

With and without this option, and also with autoscaling also turned on, which looks better:
plot_dataset met--test--weight_nominal--project_met-yscale_log
plot_dataset met--sig_regions_fit--weight_nominal--project_met-yscale_log
plot_dataset met--sig_regions_fit--weight_nominal--project_met-yscale_log

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.

1 participant