#14640: Refactor the plot_expose function into a method
--------------------------------+-------------------------------------------
       Reporter:  vbraun        |         Owner:  jason, was      
           Type:  defect        |        Status:  positive_review 
       Priority:  major         |     Milestone:  sage-5.10       
      Component:  graphics      |    Resolution:                  
       Keywords:                |   Work issues:                  
Report Upstream:  N/A           |     Reviewers:  Travis Scrimshaw
        Authors:  Volker Braun  |     Merged in:                  
   Dependencies:                |      Stopgaps:                  
--------------------------------+-------------------------------------------

Comment (by nthiery):

 Hi Volker,

 +1 for the change: I was just lazy at this point touching yet another
 thing outside of root systems and risking to create a conflict; and
 also launching a discussion about what would be the right output,
 etc. Thanks for fixing my lazyness!

 As for the name, why is active better? Isn't the convention to use a
 noun describing the result for methods whose main purpose is to return
 something, and a verb describing the action for methods whose main
 purpose is to change ``self``? At this point I would lean for
 "description".

 Oh, and we might want to check if e.g. matplotlib does not have
 already a convention for this. Related things are the x3d_str and
 mtl_str methods in 3d plots.

 Anyway, just throwing ideas in the air. Please proceed as you see fit!

-- 
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/14640#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 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?hl=en.
For more options, visit https://groups.google.com/groups/opt_out.


Reply via email to