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

ENH: pass dash_offset through to gc for Line2D #5433

Merged
merged 4 commits into from Nov 24, 2015

Conversation

tacaswell
Copy link
Member

Previously, the user supplied offset to Line2D set_linestyle
was silently discarded. This commit makes the value to be passed
through to the underlying GC on draw, but not all of the backends do
anything with the offset (ex Agg does not).

@mdboom
Copy link
Member

mdboom commented Nov 12, 2015

See my enh_dashoffset branch. I fixed the Agg backend and the PostScript backend. All other backends seemed to already work. (Can't make PR against this because we have too many forks and I can't set yours as the base ?!?)

We should add a test based on your example in #5430.

@tacaswell
Copy link
Member Author

We have emailed gh about that issue, we are just too big. The work around
is url hacking as the bug is in generating the drop down menu.

On Thu, Nov 12, 2015, 09:02 Michael Droettboom notifications@github.com
wrote:

See my enh_dashoffset branch. I fixed the Agg backend and the PostScript
backend. All other backends seemed to already work. (Can't make PR against
this because we have too many forks and I can't set yours as the base ?!?)

We should add a test based on your example in #5430
#5430.


Reply to this email directly or view it on GitHub
#5433 (comment)
.

@tacaswell tacaswell modified the milestones: proposed next point release (2.1), unassigned Nov 24, 2015
tacaswell and others added 4 commits November 24, 2015 00:23
Previously, the user supplied offset to Line2D set_linestyle
was silently discarded.  This commit makes the value to be passed
through to the underlying GC on draw, but not all of the backends do
anything with the offset (ex Agg does not).
@tacaswell
Copy link
Member Author

I think this is ready to go.

mdboom added a commit that referenced this pull request Nov 24, 2015
ENH: pass dash_offset through to gc for Line2D
@mdboom mdboom merged commit 06a08e3 into matplotlib:master Nov 24, 2015
@mdboom mdboom modified the milestones: next major release (2.0), proposed next point release (2.1) Dec 11, 2015
@mdboom
Copy link
Member

mdboom commented Dec 11, 2015

Backported to 2.0.x as d11e344

mdboom added a commit that referenced this pull request Dec 11, 2015
ENH: pass dash_offset through to gc for Line2D
@mdboom mdboom mentioned this pull request Dec 11, 2015
@tacaswell tacaswell deleted the enh_dashoffset branch December 13, 2015 04:46
@afvincent
Copy link
Contributor

In test_dash_offset(), is it normal to have plt.show() at the end of the test? I just ask this because when I run test_axes in an IPython session, it is the only test that plots something on my screen (and waits for me to close the window to finish the other tests, as plt.ion() is not used).

@tacaswell
Copy link
Member Author

That can probably be removed.

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.

None yet

3 participants