#4878: [with patch; positive review] Add a density_plot() function
-----------------------+----------------------------------------------------
 Reporter:  abergeron  |        Owner:  abergeron
     Type:  defect     |       Status:  assigned 
 Priority:  major      |    Milestone:  sage-3.4 
Component:  graphics   |   Resolution:           
 Keywords:             |  
-----------------------+----------------------------------------------------
Changes (by wdj):

  * summary:  [with patch; needs review] Add a density_plot() function =>
              [with patch; positive review] Add a
              density_plot() function

Comment:

 Arnaud: I think you have addressed William's complaint in your second
 patch. (I was basically arguing the same thing, though maybe not so
 clearly.) You describe cmap as one of the options to density_plot and
 "Type "cmap_hlp() for more information". The patch 4884 (which has a
 positive review) impliments that.

 With the second patch, I think this is an excellent addition. Both patches
 to be applied together with that in #4884.

 Positive review. Thanks Arnaud!

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