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 simple box plots #145

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

Add simple box plots #145

wants to merge 6 commits into from

Conversation

grero
Copy link

@grero grero commented Apr 27, 2014

This adds a basic boxplot function to Winston. I've included code to indicate notches around the median, which uses a simple analytical expression for the notch size (http://en.wikipedia.org/wiki/Box_plot).
I'm mainly submitting this to get comments, as I'm sure the way I'm currently doing this is less than optimal.
Eagerly awaiting any criticisms : )

@nolta
Copy link
Member

nolta commented Apr 27, 2014

Thanks! Just a few minor suggestions:

  • I'm not sure about the name Boxplot. How about something like QuartileBoxes or TukeyBoxes?
  • Needs docs. Specifically, a doc/fun/boxplot.rst file and a line in NEWS.

@grero
Copy link
Author

grero commented Apr 28, 2014

Thanks for the comments
I think QuartileBoxes might be a better name, I guess I used Boxplot since that's what Matlab and Matplotlib calls these kinds of plots. I actually prefer looking at distributions like this rather than as a histogram.
I'll put together some documentation as well.

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.

2 participants