This issue tracker has been migrated to GitHub, and is currently read-only.
For more information, see the GitHub FAQs in the Python's Developer Guide.

classification
Title: Turtle, fix right/left rotation orientation
Type: Stage: resolved
Components: Documentation Versions: Python 3.2, Python 3.3, Python 2.7
process
Status: closed Resolution: fixed
Dependencies: Superseder:
Assigned To: docs@python Nosy List: docs@python, ezio.melotti, orsenthil, python-dev, sandro.tosi
Priority: normal Keywords: patch

Created on 2011-08-01 13:37 by sandro.tosi, last changed 2022-04-11 14:57 by admin. This issue is now closed.

Files
File name Uploaded Description Edit
turtle_right-default.patch sandro.tosi, 2011-08-01 13:37 review
turtle_right-2.7.patch sandro.tosi, 2011-08-01 13:38 review
turtle_right-default-v2.patch sandro.tosi, 2011-08-04 13:51 review
turtle_right-2.7-v2.patch sandro.tosi, 2011-08-04 13:51
Messages (6)
msg141511 - (view) Author: Sandro Tosi (sandro.tosi) * (Python committer) Date: 2011-08-01 13:37
Hello, following up http://mail.python.org/pipermail/docs/2011-July/005235.html here's 2 patch (for 3.3 + 3.2 and 2.7) to correct the orientation in the documentation, using turtle.right instead of turtle.left, since the work 'counterclockwise' (even if correct and already used) might be "too heavy". I don't think the sense of the introductory part is voided with the change in the method name instead of fixing what it does.
msg141587 - (view) Author: Senthil Kumaran (orsenthil) * (Python committer) Date: 2011-08-03 00:08
The orientation depends upon the mode, which is explained further
down.
http://docs.python.org/py3k/library/turtle.html#turtle.left

So, the correct fix would be: 
"would turn clockwise/counterclockwise depending upon the mode."
msg141610 - (view) Author: Ezio Melotti (ezio.melotti) * (Python committer) Date: 2011-08-03 12:58
It seems to me that right() always turns the turtle clockwise, both with the "standard" and the "logo" mode.
I agree that changing left->right and leave clockwise is better than changing clockwise->counterclockwise.
I don't think it's worth mentioning the mode here.
(On a side note: I was expecting the 'turtle' to be an instance of some Turtle class defined in the turtle module, or perhaps a singleton, but it turned out that in 'turtle.right()' turtle is the module itself.  It might be better to say something like "After importing the :mod:`turtle` module, ...".)
msg141628 - (view) Author: Sandro Tosi (sandro.tosi) * (Python committer) Date: 2011-08-04 13:51
As mentioned by Ezio, and I also verified myself, mode() doesn't influence the orientation on left()/right() methods (probably it affects some other functs?). Following was Ezio and I discussed on IRC, I've updated the 2 patches to add a small note that you have to import turtle before trying the mentioned commands.
msg141737 - (view) Author: Senthil Kumaran (orsenthil) * (Python committer) Date: 2011-08-07 01:44
Sandro - You could go ahead and commit this.
msg141745 - (view) Author: Roundup Robot (python-dev) (Python triager) Date: 2011-08-07 15:15
New changeset a30e6c4bf486 by Sandro Tosi in branch '2.7':
#12677: correct turtle orientation in doc
http://hg.python.org/cpython/rev/a30e6c4bf486

New changeset bf4f65043d7d by Sandro Tosi in branch '3.2':
#12677: correct turtle orientation in doc
http://hg.python.org/cpython/rev/bf4f65043d7d

New changeset a7365994f4d6 by Sandro Tosi in branch 'default':
#12677: merge with 3.2
http://hg.python.org/cpython/rev/a7365994f4d6
History
Date User Action Args
2022-04-11 14:57:20adminsetgithub: 56886
2011-08-07 15:17:41sandro.tosisetstatus: open -> closed
resolution: fixed
stage: patch review -> resolved
2011-08-07 15:15:03python-devsetnosy: + python-dev
messages: + msg141745
2011-08-07 01:44:51orsenthilsetmessages: + msg141737
2011-08-04 13:51:50sandro.tosisetfiles: + turtle_right-2.7-v2.patch
2011-08-04 13:51:38sandro.tosisetfiles: + turtle_right-default-v2.patch

messages: + msg141628
2011-08-03 12:58:51ezio.melottisetnosy: + ezio.melotti
messages: + msg141610
2011-08-03 00:08:19orsenthilsetnosy: + orsenthil
messages: + msg141587
2011-08-01 13:38:12sandro.tosisetfiles: + turtle_right-2.7.patch
2011-08-01 13:37:48sandro.tosicreate