-
Notifications
You must be signed in to change notification settings - Fork 15
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
Status #1
Comments
@mojovski , I try respond:
Actual implementation does not do proper consistency check except the basic check related to have minimum of 3 point from 3 different cameras meeting together within a small distance constrain. |
thank you very much for your comments! as for the scale invariance, would it make sence to rectify an image pair? |
What consistency checks or other improvements do you suggest to add to this repo? |
first of all, thanks a lot for sharing!
I would like to contribute to this project but would like to know your opinion first.
Does it make sense to use Daisy for dense matching or is this obsolete today?
The kermit.ply file provided in the project looks like there is still some space for improvements yet. :)
However, I guess that you are not doing any consistency checks yet before exporting the ply, right?
The text was updated successfully, but these errors were encountered: