X Tutup
Skip to content

FIX: don't compute flier positions if not showing#7276

Merged
QuLogic merged 1 commit intomatplotlib:masterfrom
phobson:bxp-optional-fliers
Oct 16, 2016
Merged

FIX: don't compute flier positions if not showing#7276
QuLogic merged 1 commit intomatplotlib:masterfrom
phobson:bxp-optional-fliers

Conversation

@phobson
Copy link
Copy Markdown
Member

@phobson phobson commented Oct 16, 2016

Closes #7263

Fliers are documented as being optional when showfliers=False.
However the Axes.bxp method was trying to compute the x-values
of the flier artists based on the length of the flier array
before the logic to draw them.

Closes GH matplotlib#7263

Fliers are documented as being optional when showfliers=False.
However the Axes.bxp method was trying to compute the x-values
of the flier artists based on the length of the flier array
before the logic to draw them.
@NelleV NelleV changed the title FIX: don't compute flier positions if not showing [MRG+1] FIX: don't compute flier positions if not showing Oct 16, 2016
@NelleV
Copy link
Copy Markdown
Member

NelleV commented Oct 16, 2016

LGTM!
Thanks for the patch!

@QuLogic QuLogic added this to the 2.0.1 (next bug fix release) milestone Oct 16, 2016
@QuLogic QuLogic changed the title [MRG+1] FIX: don't compute flier positions if not showing FIX: don't compute flier positions if not showing Oct 16, 2016
@QuLogic QuLogic merged commit 389a7ed into matplotlib:master Oct 16, 2016
@QuLogic
Copy link
Copy Markdown
Member

QuLogic commented Oct 16, 2016

Backported to v2.x as 7e44d70.

@phobson phobson deleted the bxp-optional-fliers branch October 16, 2016 16:19
@QuLogic QuLogic modified the milestones: 2.0.1 (next bug fix release), 2.0 (style change major release) Dec 7, 2016
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.

4 participants

X Tutup