#4976: [with patch, needs review] fill option for plot, polar_plot and
parametric_plot
-------------------------+--------------------------------------------------
Reporter: whuss | Owner: whuss
Type: enhancement | Status: new
Priority: major | Milestone: sage-3.4.1
Component: graphics | Resolution:
Keywords: |
-------------------------+--------------------------------------------------
Comment (by kcrisman):
I think that someone who does heavier-duty plotting (and who has a machine
new enough that plot.py doesn't time out while testing) should give a
final review, but it very nicely solves a lot of problems, and didn't
break any of the examples I thought it might when I tried it. After some
thought, probably putting generate_plot_points separately (and with very
nice documentation) is a wise idea as well, and hopefully someone who
originated that code will agree.
My only functional caveat is that you may want to put in a catch for
fill=min and fill=max (as opposed to fill='min' and fill='max') because it
parses these as functions, which they are, but it should probably raise an
error (in both cases seems to have net effect of fill='axis'), since max
and min are not really functions of one variable.
There should probably also be more explicit documentation of how the
fillcolor option works, as opposed to seeing fillcolor='#ccc' in the tests
- all legal input categories should be listed, perhaps there are others.
But that's all quibbles and my own ignorance; this is a beautiful
addition, and someone else should review it very soon to avoid bitrot.
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/4976#comment:1>
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
-~----------~----~----~----~------~----~------~--~---
