#4248: [with patch, needs review] aspect_ratio is buggy in plot3d
----------------------------------+-----------------------------------------
 Reporter:  jhpalmieri            |        Owner:  somebody
     Type:  defect                |       Status:  new     
 Priority:  major                 |    Milestone:  sage-3.2
Component:  graphics              |   Resolution:          
 Keywords:  aspect_ratio, plot3d  |  
----------------------------------+-----------------------------------------
Changes (by jhpalmieri):

  * summary:  aspect_ratio is buggy in plot3d => [with patch, needs review]
              aspect_ratio is buggy in plot3d

Comment:

 I'm attaching a patch.  Some comments:

 1. the only significant change is the removal of the line
 {{{
 if i != longest_side:
 }}}
 and the ensuing change in indentation.

 2. the variable {{{new_box}}}, defined at the beginning of the old code,
 wasn't used anywhere, so I deleted it.

 3. my other changes are basically cosmetic.

-- 
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/4248#comment:1>
Sage <http://sagemath.org/>
Sage - Open Source Mathematical Software: Building the Car Instead of 
Reinventing the Wheel
--~--~---------~--~----~------------~-------~--~----~
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