-
Notifications
You must be signed in to change notification settings - Fork 1
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
WIP: Corner detection #5
base: master
Are you sure you want to change the base?
Conversation
Codecov Report
@@ Coverage Diff @@
## master #5 +/- ##
==========================================
+ Coverage 89.9% 91.67% +1.76%
==========================================
Files 12 12
Lines 1506 1921 +415
==========================================
+ Hits 1354 1761 +407
- Misses 152 160 +8
Continue to review full report at Codecov.
|
Is there a blocker here? Would you like a review? I can also rebase the branch on master after mine #11 is merged and fix any linting issues. |
I haven't worked on this crate for a little while, so as far as I remember, no, no real blockers, this just went to the back burner before I was finished. IIRC I had the Harris detector sort of working (though there may be a few issues left to resolve). I gave it a quick read and it looks like there's some stuff left to clean up. I was also trying to come up with a way to unit test that to gain a bit more confidence in that code, I guess I'll have to take a look at how (if?) other image libraries do it. I'll try to finish it soon-ish after fixing the CI and merging #11. Reviews are always welcome, of course. |
I am interested in making an orb detector + descriptor, but I think I will probably make that as a separate crate downstream of this repo. I will let you know how that goes and possibly make it easier during that process (by sending in some PRs) to allow you to refactor the actual detector stuff into its own crate as well. |
Okay. In that case I'll just remind you that this crate should be considered wildly unstable (I haven't even released it on crates.io yet) and that you might (i.e. probably will) encounter limitations I haven't thought of, weird APIs and/or outdated docs. I'm not saying it's impossible, but image and imageproc are probably safer bets right now. I didn't really expect anyone to use this yet 😸 But that should encourage me to get a usable release out the door when I'm less busy with other projects. If you do work on it then absolutely please do let me know! |
6c11c74
to
4d05467
Compare
Implement Harris and Shi-Tomasi corner detectors, and some kind of corner refinement algorithm.