#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 john_perry):

 Replying to [comment:2 kcrisman]:
 > Hi, nice start!  Very comprehensive.

 Thanks :-)

 >  * 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
 >
 > }}}

 Oh, wow. I'll take care of that.

 >  * 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.

 Ack. I'd had so many problems with alpha that I just assumed saving it
 wouldn't work on anything. Still, that makes sense, considering that
 coordinates don't raise objections from matplotlib.

 >  * 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 (?).

 Alright; I'll look for that.

 >  * When referencing Trac tickets, you might as well put it in the main
 line, e.g.
 > {{{
 > Save alpha information in pdf (see :trac:`13732`)::
 > }}}

 I thought there was something, but I couldn't remember it, and I figured
 that someone would correct me if so. Bingo. :-D

 Okay, I'll get to work fixing this now.

-- 
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/13732#comment:3>
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.


Reply via email to