#6367: polygon2d -- several issues: typo in docs, shouldn't have been renamed
----------------------+-----------------------------------------------------
Reporter: was | Owner: was
Type: defect | Status: new
Priority: major | Milestone: sage-4.1
Component: graphics | Keywords:
Reviewer: | Author:
Merged: |
----------------------+-----------------------------------------------------
The help for polygon2d? says:
{{{
Type ``polygon.options`` for a dictionary of the default
options for polygons. You can change this to change
}}}
but it should say
{{{
Type ``polygon2d.options`` for a dictionary of the default
options for polygons. You can change this to change
}}}
And for the record I think it is unfortunate that somebody changed the
function name from polygon to polygon2d, since that it inconsistent with
almost all the rest of plotting in Sage, where the 2d version of the
function doesn't specifically say it is 2d (e.g., plot, line, text, etc.)
Well, there is line2d, to add to the confusion. The design of Sage
graphics is *supposed* to follow the naming scheme in Matheamtica. In
Mathematica there is Polygon and Polygon3D. That's what Sage should have.
To resolve this patch, either fix the docstring or change the name back to
polygon (not polygon2d). I prefer the latter for consistency with the
rest fo the design of sage 2d graphics.
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/6367>
Sage <http://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
-~----------~----~----~----~------~----~------~--~---