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

FIX #26, FIX #10: MISCALCULATING MIN AND MAX BARPLOT VALUES #34

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

FIX #26, FIX #10: MISCALCULATING MIN AND MAX BARPLOT VALUES #34

wants to merge 2 commits into from

Conversation

tligier
Copy link

@tligier tligier commented Oct 1, 2020

What?

Calculate min and max values according to https://www.data-to-viz.com/caveat/boxplot.html.
Previous values were calculated as if there would always be outliers values.

Why?

Fix #26
Fix #10

How?

Calculate the value for the lower whisker as max(minimum value, q1 - 1.5 * IQR)
Calculate the value for the upper whisker as min(maximum value, q3 + 1.5 * IQR)

Calculate min and max values according to https://www.data-to-viz.com/caveat/boxplot.html.
Previous values were calculated as if there would always be outliers values.
@tligier tligier changed the title FIX #26: MISCALCULATING MIN AND MAX BARPLOT VALUES FIX #26, FIX #10: MISCALCULATING MIN AND MAX BARPLOT VALUES Oct 1, 2020
Documentation changed to correspond to the "Categorical color scale" example.
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.

Basic boxplot in d3.js "max" value is incorrect? Boxplot calculations are incorrect
1 participant