Skip to content

Adding orientation to draw.ellipse.#2352

Closed
alexdesiqueira wants to merge 1 commit intoscikit-image:masterfrom
alexdesiqueira:draw_ellipse_orientation
Closed

Adding orientation to draw.ellipse.#2352
alexdesiqueira wants to merge 1 commit intoscikit-image:masterfrom
alexdesiqueira:draw_ellipse_orientation

Conversation

@alexdesiqueira
Copy link
Copy Markdown
Member

@alexdesiqueira alexdesiqueira commented Oct 25, 2016

Description

Adds the orientation parameter to draw.ellipse, according to the general ellipse equation [1].

References

Related to issue #2135.
[1] https://en.wikipedia.org/wiki/Ellipse#General_ellipse

Notes

Tested on some cases. Example of test code:

for i in range(0, 181, 10):
    image = np.zeros((256, 256))
    rr, cc = ellipse(120, 120, 25, 55, i, shape=image.shape)
    image[rr, cc] = 1
    plt.imshow(image, cmap='viridis'), plt.show()

@alexdesiqueira alexdesiqueira mentioned this pull request Nov 4, 2016
Merged
4 tasks
@soupault
Copy link
Copy Markdown
Member

soupault commented Dec 7, 2016

Superseded by #2366.

@soupault soupault closed this Dec 7, 2016
@alexdesiqueira alexdesiqueira deleted the draw_ellipse_orientation branch July 3, 2018 12:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

🙏 Feature request

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@alexdesiqueira @soupault @grlee77