-
-
Notifications
You must be signed in to change notification settings - Fork 404
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
Box whisker improvements #2187
Box whisker improvements #2187
Conversation
representing the lowest datum still within 1.5 IQR of the lower | ||
quartile, and the highest datum still within 1.5 IQR of the upper | ||
quartile. Any points falling outside this range are shown as | ||
distinct outlier points. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
maybe 'standard Tukey boxplot definition' if it is standard? Otherwise sounds like it is just a definition for boxplots...
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Might be worth expanding the name to 'John W. Tukey'...
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Tukey box plots is the common way to refer to this style of plot, don't think the full name adds anything.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
To quote Wikipedia:
But the ends of the whiskers can represent several possible alternative values, among them:
the lowest datum still within 1.5 IQR of the lower quartile, and the highest datum still within 1.5 IQR of the upper quartile (often called the Tukey boxplot)
One minor docstring comment. Otherwise looks good. |
Looks good. Merging. |
This pull request has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs. |
Improved BoxWhisker element docstrings as requested in #1838 and fixed hover issues as requested in #2142.