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 support for reporting histogram (density) #35

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

rakhimov
Copy link
Member

This modifies the "quantiles" schema to support bins
similar (but not the same) to the MEF input.
The distribution/frequency of the data is meant to be in the histogram
as a statistical measure for reporting.

It would be somewhat beneficial
if histogram reporting schema matched exactly
the input histogram-distribution (deviate) schema
since the semantics is the same.

@cfolleau
Copy link
Member

From the result point of view, quantile are histograms (but with the same value for each interval since it is the definition).
If we want to be more general for the result provided by Motecarlo Simultation, histogram could replace quantile.
But if people want to keep the word "quantile" because it is more mathematically correct, no problem.

@rakhimov
Copy link
Member Author

@cfolleau I think you are right.
I find both of them necessary/useful to calculate and report.
Correct me if I am wrong, but quantiles mask out the shape of the distribution.
The histogram derived from the quantiles is just boring flat line
with each bin having the same weight but different size;
Histograms, on the other hand, reveal the shape of the density.

@cfolleau
Copy link
Member

yes exactly

This modifies the "quantiles" schema to support bins
similar to the MEF input.
The distribution/frequency of the data is meant to be in the histogram
as a statistical measure for reporting.

It would be somewhat beneficial
if histogram reporting schema matched exactly
the input histogram-distribution (deviate) schema
since the semantics is the same.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants