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

Fix pdf export of graphs #15116

Closed
defeo opened this issue Aug 28, 2013 · 7 comments
Closed

Fix pdf export of graphs #15116

defeo opened this issue Aug 28, 2013 · 7 comments

Comments

@defeo
Copy link
Member

defeo commented Aug 28, 2013

The following code raises an error

sage: p = graphs.CompleteGraph(2).plot()
sage: p.save('tmp.pdf')
---------------------------------------------------------------------------
AssertionError                            Traceback (most recent call last)
...
AssertionError: 

The error seems to be caused by edges, indeed

sage: p = graphs.CompleteGraph(1).plot()
sage: p.save('tmp.pdf')

works fine.

Depends on #14993

CC: @nthiery

Component: graphics

Keywords: graph plot

Issue created by migration from https://trac.sagemath.org/ticket/15116

@defeo defeo added this to the sage-6.1 milestone Aug 28, 2013
@sagetrac-edward-scheinerman
Copy link
Mannequin

comment:2

Interesting. Indeed a two vertex, zero edge graph also works fine:

sage: g = Graph(2)
sage: pic = g.plot()
sage: pic.save('two-verts.pdf')

@mezzarobba
Copy link
Member

comment:3

This looks very much like matplotlib/matplotlib#2418. The patch given there (included in matplotlib-1.3.1) fixes the issue.

@nathanncohen
Copy link
Mannequin

nathanncohen mannequin commented Dec 7, 2013

comment:4

Gosh ! They have quivers there too ? O_O

Good news, though. Well, it means that we need to upgrade matplotlib now :-)

Nathann

@sagetrac-vbraun-spam sagetrac-vbraun-spam mannequin modified the milestones: sage-6.1, sage-6.2 Jan 30, 2014
@mezzarobba
Copy link
Member

Dependencies: 14993

@fchapoton
Copy link
Contributor

Changed dependencies from 14993 to #14993

@sagetrac-vbraun-spam sagetrac-vbraun-spam mannequin modified the milestones: sage-6.2, sage-6.3 May 6, 2014
@fchapoton
Copy link
Contributor

comment:9

works in 6.3.beta1

@fchapoton fchapoton removed this from the sage-6.3 milestone May 23, 2014
@nathanncohen
Copy link
Mannequin

nathanncohen mannequin commented May 23, 2014

comment:10

Frederic : when you set a ticket to "wontfix", please change its status to positive_review.

Nathann

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants