-
-
Notifications
You must be signed in to change notification settings - Fork 31.8k
Documentation of turtle.circle() #84464
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Comments
refer to circle() in the url: https://docs.python.org/2/library/turtle.html#turtle.circle [current] [suggestion] [explanation] |
It would be bizarre if the current doc were correct, but have you verified by experiment that the change is correct? The patch itself would be trivial, hence the keyword addition. PR author can request that I review. |
When I checked, I found the following: turtle.circle(50, 50) # counter-clockwise
turtle.circle(-50, -50) # counter-clockwise
turtle.circle(-50, 50) # clockwise
turtle.circle(50, -50) # clockwise So it seems both radius and extent need to be of the same signage to effect a counter-clockwise draw. However, for the 2 cases where extent is negative, the draw happens through the turtle's tail rather than the head. Looks weird. |
I tested it and can confirm the findings from @cajetan.rodrigues. However, this does not seem to be _not_ documented. The documentation clearly states that using a positive value for radius results in a counter-clockwise direction, whereas a negative number results in a clockwise direction. This is true. Now, let's have a look at using radius _together_ with extent, which is not (clearly) documented as this point. Using a positive value results in the same direction, but in a not full drawn circle. This seems intuitive and is documented:
Using a negative value changes the direction of radius, which does also seem reasonable and intuitive. My suggestion is not to replace "radius" by "extent" as suggested by @guchao as it is IMHO wrong. But I agree that it would be good to leave a note there or to add examples demonstrating the behaviour of different (signed) values of radius and extent, so the intent and behaviour is clearly documented. |
Note: these values reflect the state of the issue at the time it was migrated and might not reflect the current state.
Show more details
GitHub fields:
bugs.python.org fields:
Linked PRs
turtle.circle()
drawing direction doc #128986The text was updated successfully, but these errors were encountered: