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

New features for boxplot #217

Closed
ddale opened this issue Jun 20, 2011 · 8 comments
Closed

New features for boxplot #217

ddale opened this issue Jun 20, 2011 · 8 comments
Milestone

Comments

@ddale
Copy link
Contributor

ddale commented Jun 20, 2011

Original report at SourceForge, opened Thu Mar 8 13:20:11 2007

Added options to:

  • plot everything in black only
  • fix notch size and switch off box outline around notch when notch is larger than inter-quartile distance
  • add a line corresponding to the mean average
  • fill the boxes in white
  • prevent automatic axis limit setting

Added docstring text and minor edits to existing text.
Existing functionality is unaffected.

See my detailed description, example output, and the code posted on the matplotlib-devel list. Output from diff is attached, relative to matplotlib 0.90 axes.py method 'boxplot'.

SourceForge History

  • On Thu Mar 8 13:20:12 2007, by robclewley: File Added: 219476: diff_output.txt
@dmcdougall
Copy link
Member

Is this patch still around to look at? I was never on sourceforge...

@efiring
Copy link
Member

efiring commented Aug 28, 2012

All issue attachments were lost in the transition from sf to github, as far as I know. It is not entirely clear whether robclewley also posted the modified boxplot to matplotlib-devel, but if he did, then you should be able to find it in one of the mailing list archives. Also, you may be able to track him down on the web.

@dmcdougall
Copy link
Member

@efiring Thanks! I found it. Now it's time to run patch and see what breaks. Win.

@dmcdougall
Copy link
Member

So the diff is super noisy (unsurprisingly). I'll take a look through it later this week and try to pull out the relevant parts.

@dmcdougall
Copy link
Member

I'm creating a PR with this diff so we can figure out what the new features are. That way everyone can chime in.

@dmcdougall
Copy link
Member

See #1199.

@mdboom
Copy link
Member

mdboom commented Jan 27, 2014

@phobson: I think now that #2643 is merged, this can be closed. If I'm wrong, just let me know...

@mdboom mdboom closed this as completed Jan 27, 2014
@phobson
Copy link
Member

phobson commented Jan 27, 2014

@mdboom you're correct.

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

No branches or pull requests

5 participants