#14223: Fix 3d plots to not ignore user prespecified aspect_ratio.
-----------------------------------------+----------------------------
       Reporter:  nthiery                |        Owner:  jason, was
           Type:  defect                 |       Status:  needs_review
       Priority:  major                  |    Milestone:  sage-5.12
      Component:  graphics               |   Resolution:
       Keywords:                         |    Merged in:
        Authors:  Punarbasu Purkayastha  |    Reviewers:
Report Upstream:  N/A                    |  Work issues:
         Branch:                         |       Commit:
   Dependencies:                         |     Stopgaps:
-----------------------------------------+----------------------------

Comment (by twch):

 Hi Basu,

 here my remarks on the patch. It applies perfectly and in almost all cases
 which I tested it fixes the bug. I only found the following case, where
 setting the aspect ratio seems still to be ignored:


 {{{
 var('x,y')
 P=plot3d(x^2-9*y^2,(x,-3,3),(y,-1,1))
 P.aspect_ratio((1,1,1))
 P.show()
 }}}

 is different from

 {{{
 var('x,y')
 P=plot3d(x^2-9*y^2,(x,-3,3),(y,-1,1))
 P.show(aspect_ratio=(1,1,1))
 }}}

 furthermore I observed that by this patch

 {{{
 P.show(aspect_ratio=(1,2,3))
 }}}
 changes the value of
 {{{
 P._aspect_ratio
 }}}
 and thus also the behaviour in all following show() commands. One can
 surely discuss if this is usefull or not, but I think in the 2d plots the
 setting of aspect ration in a show command only affects this plot and not
 the following. I think the 3d behavior should be consistent.

--
Ticket URL: <http://trac.sagemath.org/ticket/14223#comment:12>
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/groups/opt_out.

Reply via email to