Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Ellipse bounding box #36

Closed
AnushaTiwari5 opened this issue Sep 19, 2023 · 1 comment · Fixed by #73 or #95
Closed

Ellipse bounding box #36

AnushaTiwari5 opened this issue Sep 19, 2023 · 1 comment · Fixed by #73 or #95
Assignees
@AnushaTiwari5
Labels
Enhancement New feature required This feature is required to be implemented by the current implementation team
Milestone
Sprint 9/25

Comments

@AnushaTiwari5
Copy link
Collaborator

Calculate ellipse bounding box using shortest distance method.
"convert the ellipse to a polar coordinate parameterization in terms of an angle, use the center of both ellipses to get an angle to find the point on E1 closet to E2, then just run the is a point inside an ellipse check."

image
@AnushaTiwari5 AnushaTiwari5 added Enhancement New feature required This feature is required to be implemented by the current implementation team labels Sep 19, 2023
@AnushaTiwari5 AnushaTiwari5 added this to the Milestone 5: Demo 2 milestone Sep 19, 2023
@AnushaTiwari5 AnushaTiwari5 self-assigned this Sep 19, 2023
@James-Oswald
Copy link
Member

James-Oswald commented Sep 23, 2023 •
edited
Loading

One way of finding the point

https://www.desmos.com/calculator/ppv5tutqwx

@AnushaTiwari5 AnushaTiwari5 linked a pull request Oct 1, 2023 that will close this issue
36 ellipse bounding box #73
Merged
@AnushaTiwari5 AnushaTiwari5 linked a pull request Oct 9, 2023 that will close this issue
36 ellipse bounding box #95
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@AnushaTiwari5 AnushaTiwari5

Labels
Enhancement New feature required This feature is required to be implemented by the current implementation team
Projects
None yet
Milestone
Sprint 9/25
Development

Successfully merging a pull request may close this issue.

36 ellipse bounding box RAIRLab/Peirce-My-Heart
36 ellipse bounding box RAIRLab/Peirce-My-Heart
2 participants
@James-Oswald @AnushaTiwari5