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

Comment (by ppurka):

 This ticket is awesome! Brought my laptop to a crawl! :)

 @emchennyc: I might come back to this ticket if kcrisman doesn't. For now,
 it seems that matplotlib (even the latest 1.2.0 release) does not handle
 the parameters `[256,256]` properly. It tries to allocate memory and then
 fails, resulting in a segfault.

 What kcrisman asks is a doctest like this (under a `TEST:` section; look
 at how these sections are written in other functions or other files):
 {{{
 TEST:

 The following plot segfaults sage and should not be doctested.::

     sage: plot(x).show(figsize=[256,256]) # not tested

 }}}

 If you do happen to run this command, be very careful. I suggest you set a
 ulimit before you start sage. If you have, say 3G, of memory/RAM, then set
 ulimit to 2G and only then start sage. If you have even less memory then
 you will need to set even lower ulimit, but sage won't run properly below
 1G.
 {{{
 $ ulimit -v 2000000
 $ sage
 sage: plot(x).show(figsize=[256,256]) # boom, segv!!
 }}}

 That said, I would like you to set some defaults in your editor.
 Currently, your patch contains a mixture of tabs and spaces. I suggest you
 change the settings in your editor to not insert tabs, and instead insert
 4 spaces every time you press a tab key.

-- 
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/5956#comment:7>
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 post to this group, send email to [email protected].
To unsubscribe from this group, send email to 
[email protected].
For more options, visit this group at 
http://groups.google.com/group/sage-trac?hl=en.

Reply via email to