Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Already on GitHub? Sign in to your account

ImageDraw.ellipse() with square bounding box doesn't produce circle #463

Closed
gronostajo opened this Issue Dec 30, 2013 · 10 comments

Comments

Projects
None yet
4 participants

The code below should produce a circle:

>>> from PIL import Image, ImageDraw
>>> box = (20, 20, 40, 40)
>>> im = Image.new('L', (60, 60), 0)
>>> draw = ImageDraw.Draw(im)
>>> draw.ellipse(box, 255)
>>> im.show()

The result isn't a 20x20 px circle as expected, but 21x20 px ellipse:

test

I'm using Pillow 2.2.2.

Owner

aclark4life commented Dec 30, 2013

Got a pull request to fix this? Any idea where the regression is?

Owner

wiredfool commented Dec 31, 2013

It's at least back to Pillow 1.7.8

Owner

aclark4life commented Jan 1, 2014

Now targeting 2.4.0

@wiredfool wiredfool modified the milestone: Future, 2.4.0 Mar 20, 2014

Member

hugovk commented Mar 27, 2014

Reproducible with Pillow 1.76 and PIL 1.1.7 on Python 2.7/Windows.

Reproducible with PIL 1.1.4 on Python 2.3/Windows, the oldest PIL here.

pillow-1 7 6-py2 7 . pil-1 1 7-py2 7 . pil-1 1 4-py2 3

Owner

wiredfool commented Mar 27, 2014

Pointer: related to #367

@aclark4life aclark4life modified the milestone: 2.5.0, Future Apr 1, 2014

Owner

aclark4life commented Apr 1, 2014

2.5.0 time!

@hugovk hugovk added a commit to hugovk/Pillow that referenced this issue May 12, 2014

@hugovk hugovk More ImageDraw tests. Some may need redoing after issues #367 and #463
…are sorted.
6b274b4
Owner

aclark4life commented Jun 1, 2014

What's going on here?

Member

hugovk commented Jun 1, 2014

@aclark4life I expect this will be fixed when @Terseus' work is finalised in #610. Currently awaiting further feedback from @wiredfool [https://github.com/python-imaging/Pillow/pull/610#issuecomment-42601238].

Owner

aclark4life commented Jun 1, 2014

Ah, thanks @hugovk

Owner

aclark4life commented Jun 27, 2014

#610 superseded by #737

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment