-
-
Notifications
You must be signed in to change notification settings - Fork 2.2k
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
Added ImageDraw circle() #8085
Added ImageDraw circle() #8085
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please could you add unit tests, and mention this in the release notes?
Co-authored-by: Hugo van Kemenade <1324225+hugovk@users.noreply.github.com>
…te it is new Co-authored-by: Hugo van Kemenade <1324225+hugovk@users.noreply.github.com>
for more information, see https://pre-commit.ci
Co-authored-by: Hugo van Kemenade <1324225+hugovk@users.noreply.github.com>
…[float] instead of Coords, radius to float Co-authored-by: Andrew Murray <3112309+radarhere@users.noreply.github.com>
I've added it to the release notes but am not quite sure how the unit tests work, especially the parametrization :/ |
I've created void4#1 with my suggestion for a test - I've copied To provide some information though, parametrization is essentially a loop - it runs the test with different items from a sequence. The idea is that if a particular scenario fails, pytest will clearly let us know which item from the parametrization sequence is the culprit. If you're interested more info, you can read up at https://docs.pytest.org/en/7.4.x/how-to/parametrize.html |
Thank you - merged :) |
Thank you! |
Resolves #8083
Implements the ImageDraw.circle feature as described and desired in the issue