#4342: Add legends to plot.py
---------------------------+------------------------------------------------
   Reporter:  anakha       |       Owner:  anakha      
       Type:  enhancement  |      Status:  needs_review
   Priority:  major        |   Milestone:  sage-4.4.3  
  Component:  graphics     |    Keywords:              
     Author:               |    Upstream:  N/A         
   Reviewer:               |      Merged:              
Work_issues:               |  
---------------------------+------------------------------------------------
Changes (by kcrisman):

  * status:  needs_info => needs_review


Comment:

 Ready for review.  I made legend -> show_legend and that seems to clear up
 the problems with the @suboptions dictionary.  I expect it to be "needs
 work", by the way, but I want extensive testing of this.  Things to watch
 out for include:

 * Whether there are any missing/needed plot3d method removals

 * What the defaults should really be for all the legend_options; I don't
 think they are that good.

 * Whether there are any other places where a KeyError might arise.

 Incidentally, the rebase wasn't quite as bad as it seemed - the earlier
 patch for some reason was a "double patch" that sometimes occurs, as will
 be obvious if read the above (which I didn't).

-- 
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/4342#comment:34>
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