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.

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

Checking distance of support inliers to ellipse #15

Open
shatalinra opened this issue Jan 27, 2020 · 1 comment
Open

Checking distance of support inliers to ellipse #15

shatalinra opened this issue Jan 27, 2020 · 1 comment

Comments

@shatalinra
Copy link

According to "Adaptive Inliers Criterion" section of your paper, support inliers distance to initial ellipse candidate should be less than some threshold. However, function calcEllipseParametersAndValidate does not perform that check and function d_rosin is defined but not used. Therefore implementation allows for support inliers which are far from ellipse and still satsify regional limitation like some arc inside larger ellipse.

Am I missing something or this is on your roadmap?

@alubawzk
Copy link

alubawzk commented Jul 6, 2021

Hi, I also found this issue. But, I found that, in the function calcEllipseParametersAndValidate, the points used for "Adaptive Inliers Criterion" are the inliers of each LS rectangle in two LSs group. Maybe the author thought that the distance from these points to the current ellipse must meet the d_rosin-conition?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants