#13732: Fix output of real data in matplotlib PDF backend
------------------------------------------------+---------------------------
Reporter: john_perry | Owner: jason, was
Type: defect | Status: needs_review
Priority: trivial | Milestone: sage-5.7
Component: graphics | Resolution:
Keywords: PDF, matplotlib, save, alpha | Work issues:
Report Upstream: N/A | Reviewers:
Authors: | Merged in:
Dependencies: | Stopgaps:
------------------------------------------------+---------------------------
Comment (by kcrisman):
Hi, nice start! Very comprehensive. Here are some comments, some of
which make it "needs work".
* I think the preferred syntax is `if alpha is not None`.
{{{
In [12]: % timeit D is not None
10000000 loops, best of 3: 43.8 ns per loop
In [13]: % timeit D != None
10000000 loops, best of 3: 65.1 ns per loop
}}}
* Note that at least sometimes alpha does work. Let's only fix the ones
that do (see #14074 for that polygons apparently work). Similarly for the
doctests of it, naturally.
* You cannot just save `p.save("test.pdf")`, unfortunately, as that will
produce files left over after doctesting. There is a framework for
temporary files - in fact, several competing ones - for doctesting, there
should be some good examples in the `save` method (?).
* When referencing Trac tickets, you might as well put it in the main
line, e.g.
{{{
Save alpha information in pdf (see :trac:`13732`)::
}}}
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/13732#comment:2>
Sage <http://www.sagemath.org>
Sage: Creating a Viable Open Source Alternative to Magma, Maple, Mathematica,
and MATLAB
--
You received this message because you are subscribed to the Google Groups
"sage-trac" group.
To unsubscribe from this group and stop receiving emails from it, send an email
to [email protected].
To post to this group, send email to [email protected].
Visit this group at http://groups.google.com/group/sage-trac?hl=en.
For more options, visit https://groups.google.com/groups/opt_out.
