#5956: image dimensions for show() are in inches
-------------------------------------+-------------------------------------
       Reporter:  mvngu              |        Owner:  was
           Type:  defect             |       Status:  needs_review
       Priority:  minor              |    Milestone:  sage-6.4
      Component:  graphics           |   Resolution:
       Keywords:  image dimensions,  |    Merged in:
  figsize, beginner                  |    Reviewers:  Karl-Dieter Crisman,
        Authors:  Emily Chen,        |  Punarbasu Purkayastha
  Punarbasu Purkayastha, Karl-       |  Work issues:
  Dieter Crisman                     |       Commit:
Report Upstream:  None of the above  |  47c705c6ee6ab969ec70eb6ed1628e56b59042fe
  - read trac for reasoning.         |     Stopgaps:
         Branch:                     |
  u/kcrisman/figsize                 |
   Dependencies:                     |
-------------------------------------+-------------------------------------

Comment (by jdemeyer):

 Replying to [comment:22 ppurka]:
 > Why should we go around checking for numeric types? This is not a
 "critical" application that should ensure sane inputs for all kinds of
 inputs. If the user sends in garbage, they will eventually get garbage out
 from at least matplotlib, if not earlier in the process.

 Well, it could be that the user will send some Sage numeric type, which
 matplotlib might treat like garbage. A conversion like
 {{{
 x = float(x)   # assuming that it's a float that you need
 }}}
 will solve both problems: it will raise a `TypeError` when true garbage is
 given but it should work for all Sage numeric types.

--
Ticket URL: <http://trac.sagemath.org/ticket/5956#comment:32>
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.
For more options, visit https://groups.google.com/d/optout.

Reply via email to