#11028: More Modular ComplexPlot
---------------------------+------------------------------------------------
   Reporter:  evanandel    |          Owner:  jason, was              
       Type:  enhancement  |         Status:  needs_work              
   Priority:  minor        |      Milestone:  sage-4.7                
  Component:  graphics     |       Keywords:  complex plot riemann map
Work_issues:               |       Upstream:  N/A                     
   Reviewer:               |         Author:  evanandel               
     Merged:               |   Dependencies:                          
---------------------------+------------------------------------------------
Changes (by kcrisman):

  * status:  needs_review => needs_work


Comment:

 It seems like this has a slight redundancy with the (positively reviewed)
 #10821.  In fact, it looks like whatever this is based on has some, but
 not all, of the changes there, and the other changes are in this patch.
 ?? Anyway, the specific order and so forth of dependencies should be
 completely clarified.

 The patch is also apparently a double patch.  Read it and you'll see what
 I mean.  Which situation with regard to the not being able to test cdef'd
 functions is current?  (Sometimes we create a def'd
 `test_my_cdef_function` for these cases.)

 Will we need to introduce a deprecation period for the change to
 initializing with `complex_to_rgb(z_values)`, since the keyword has
 changed?

 Also, what is the situation with the `complex_to_rgb` functions?  By that
 I mean to ask how many of them there are, and why there is a separate one
 in riemann.pyx.

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