Patches item #1513695, was opened at 2006-06-28 02:50
Message generated for change (Comment added) made by loewis
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=305470&aid=1513695&group_id=5470

Please note that this message will contain a full copy of the comment thread,
including the initial issue submission, for this request,
not just the latest update.
Category: Tkinter
Group: Python 2.4
Status: Open
Resolution: None
Priority: 5
Submitted By: Lingl (gregorlingl)
Assigned to: Martin v. Löwis (loewis)
Summary: new turtle module

Initial Comment:
The file, at present named xturtle.py is a
reimplementation of turtle.py with additional features,
among them:

-Better animation of the turtle movements, especially
of turning the turtle. So the turtles can more easily
be used as a visual feedback instrument by the
(beginning) programmer.
-Different turtle shapes, gif-images as turtle shapes,
user defined and user controllable turtle shapes, among
them compound (multicolored) shapes.
-Fine control over turtle movement and screen updates
via delay(), and enhanced tracer() and speed(),
update()  methods.
-Aliases for the most commonly used commands, like fd
for forward etc. ...
-Some simple commands/methods for creating event driven
programs (mouse-, key-, timer-events). Especially
useful for programming simple games.
-A scrollable Canvas class. The scrollable Canvas can
be extended interactively as needed while playing
around with the turtle(s) (e. g. to follow some escaped
turtle).
-Commands for controlling background color or
background image.

detailed documentation is in the docstrings (similar to
the last version of turtle.py) which copmrise approx.
50% of the filesize.

xturtle.py is widely backward compatible to turtle.py
(i. e. except the appearance of the turtle(s) and their
animation)

I propose to replace turtle.py by xturtle.py

Separate Documentation is available as a text file
reassembled and enhanced from the docstring.
I'll supply this in a separate patch (as it seems
impossible here to attach 2 files)

Regards,
Gregor Lingl

----------------------------------------------------------------------

>Comment By: Martin v. Löwis (loewis)
Date: 2006-06-29 20:16

Message:
Logged In: YES 
user_id=21627

A number of remarks:

1. would you like to see the demo applications included as
well? They could go somewhere into Demo. If so, please
prepare the patch as a single tar/zip file that unpacks on
top of a Python source tree.

2. The doc string refers to the module as xturtle, and says
it replaces turtle. If the module becomes turtle, this
remark becomes self-referential. Please update the doc
string accordingly.

3. Somebody should investigate the rationale for
re-exporting math in the current turtle.py. If that
rationale is still valid, the new module should comply.

4. The module should grow an __all__ attribute.

5. _Vec does not appear to be a general vector class, but be
restricted to 2D cartesian coordinates. It should be
renamed. Perhaps to "Point?"

6. Try to follow PEP 8, in particular, drop camelCase for
method names.

7. Avoid passive voice for doc strings; use active voice
instead (e.g. addshape: "Adds a turtle" -> "Add a turtle")

8. What happens to the n= argument of clear()?

9. What is the purpose/status of checkargs? Do you want to
keep it or drop it?

10. There are some German doc strings left (e.g. _rotate).


----------------------------------------------------------------------

You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=305470&aid=1513695&group_id=5470
_______________________________________________
Patches mailing list
[email protected]
http://mail.python.org/mailman/listinfo/patches

Reply via email to