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

Investigate further inlier ratio w.r.t. GT model and w.r.t estimated model #306

Open
johnwlambert opened this issue Sep 19, 2021 · 1 comment · May be fixed by #374
Open

Investigate further inlier ratio w.r.t. GT model and w.r.t estimated model #306

johnwlambert opened this issue Sep 19, 2021 · 1 comment · May be fixed by #374
Labels
bug Something isn't working objective High level tasks to be accomplished

Comments

@johnwlambert
Copy link
Collaborator

Currently, the inlier ratio with respect to GT model is greater than that with respect to estimated model for USAC. This is perhaps expected, as the GT should explain the data most appropriately. But our inability to estimate a model with close to as many inliers is unfortunate, and perhaps could be explored further on some representative image pairs.

We see this with USAC on Skydio-8:
Screen Shot 2021-09-18 at 10 56 36 PM

However, we do not see this with LORANSAC on Skydio-8
Screen Shot 2021-09-18 at 11 01 24 PM

cc @akshay-krishnan @ayushbaid

@akshay-krishnan
Copy link
Collaborator

It is expected to have a high inlier ratio for the GT model (if the matcher output is good), but I don't understand why the inlier ratio with respect to the estimated model is lesser than the GT model.
The GT model was not estimated from our data, and if we had enough verifier iterations, we should have the model with the highest number of inliers. I'm not familiar with how USAC works though.

Another reason this could happen is if when we have degenerate cases (all inliers on a plane) and our estimated model is not good, but skydio-8 is not such a dataset right?

@akshay-krishnan akshay-krishnan added bug Something isn't working objective High level tasks to be accomplished labels Sep 22, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working objective High level tasks to be accomplished
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants