#4885: fix fallout from sloppy review of 4535
----------------------+-----------------------------------------------------
 Reporter:  was       |       Owner:  was       
     Type:  defect    |      Status:  new       
 Priority:  major     |   Milestone:  sage-3.2.3
Component:  graphics  |    Keywords:            
----------------------+-----------------------------------------------------
 I refereed trac #4535 and I made some serious mistakes in accepting that
 patch.  This ticket will be about fixing all of those mistakes.

  * the function xmin/xmax/ymin/ymax were all removed.  They *must* be all
 added back exactly as before.  There is a lot of code out there that
 depends on it.  Also, we do not just delete functions in sage without
 deprecating them for at least 6 months first.

  * some of the new functions get_* have no documentation or docstrings.
 Fix this.

  * Do "sage -t" on the *old* plot.py using the newest version of sage, and
 make sure nothing breaks (e.g., examples that use .xmin, etc.)  Fix
 anything that does, or at least *discuss* it.


 Unless somebody beats me to it, I'll do this, since it is my fault #4535
 got a positive review.  I'll post to this ticket, as soon as I work on
 this, so if I haven't posted a message below that I'm working on it, I
 haven't started.

   -- William

-- 
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/4885>
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