#5438: [with patch, needs review] Incorrect documentation and/or functionality 
in
plot filling
----------------------+-----------------------------------------------------
 Reporter:  kcrisman  |       Owner:  kcrisman  
     Type:  defect    |      Status:  assigned  
 Priority:  minor     |   Milestone:  sage-3.4.1
Component:  graphics  |    Keywords:  plot fill 
----------------------+-----------------------------------------------------

Comment(by kcrisman):

 In addition to changing "==" to "is", this patch fixes a lot of minor
 issues in plot that I discovered while fixing this issue, mostly around
 documentation of the adaptive refinement, but also some weird bugs in
 error handling that nobody noticed since who sets verbose=1 on a regular
 basis?

 I didn't add doctests for the change from (msg, x) to (msg, xi) since it
 refers to the line of code and I figured that is a big hassle to change
 every time the plot code is changed, but if a referee disagrees they may
 add some.  I also removed the final exceptions+=1 block in the
 generate_plot_points since it seemed to be redundant, but if that's wrong
 then bring it back.  Otherwise I think everything works.

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