#7889: revolution plot
-----------------------------------------+----------------------------------
Reporter: olazo | Owner: olazo
Type: enhancement | Status: needs_review
Priority: major | Milestone: sage-4.3.4
Component: graphics | Keywords: revolution,plot
Author: Oscar Gerardo Lazo Arjona | Upstream: N/A
Reviewer: Karl-Dieter Crisman | Merged:
Work_issues: |
-----------------------------------------+----------------------------------
Changes (by olazo):
* status: needs_work => needs_review
Comment:
Replying to [comment:11 mhampton]:
> Coverage is not complete, unless I mis-applied the patches (but I don't
think I did):
>
> {{{
> sage -coverage devel/sage-p1/sage/plot/plot3d/plot3d.py
> ----------------------------------------------------------------------
> devel/sage-p1/sage/plot/plot3d/plot3d.py
> ERROR: Please add a `TestSuite(s).run()` doctest.
> SCORE devel/sage-p1/sage/plot/plot3d/plot3d.py: 81% (13 of 16)
>
> Missing documentation:
> * triangle(self, a, b, c, color = None):
> * smooth_triangle(self, a, b, c, da, db, dc, color = None):
> * axes(scale=1, radius=None, **kwds):
> }}}
>
> So I'm switching this to needs work. Otherwise things look OK, doctests
pass, documentation looks good.
In fact, this patch has nothing to do with those functions. So I'll move
this to needs review.
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/7889#comment:15>
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.
