Uh oh!
There was an error while loading.Please reload this page.
- Notifications
You must be signed in to change notification settings - Fork56.5k
hal/riscv-rvv: implement FAST keypoint detection#27391
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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
Merged
Uh oh!
There was an error while loading.Please reload this page.
Merged
Changes from1 commit
Commits
Show all changes
27 commits Select commitHold shift + click to select a range
e0724db Empty implementation for FAST function
a088804 CALL_HAL modification for cv_hal_fast
8dac44c basic implementation of fast_9_16 for riscv
ca88641 debug test modification
1a1b2e8 Add features2d module in new rvv-hal
393ab4c Merge branch 'github' into rvv-hal-fast
2147287 Test FAST.noNMS success
8d116fa weird memory read
a15574f Pass NMS test
09e0a44 delete debug messages
fa709c3 Add perf tests
3cfdf0a Dead code deleted in test_fast.cpp
77a30d9 Fix requested changes
835176f Disable sanity check
8f20b66 Delete trailing whitespace
96756ba Rollback CALL_HAL
7f77a15 Requested changes in perf_fast.cpp
08414a4 Requested changes done
621a65f Put memset in default branch & keypoint_cnt not initialized bug
7c40a56 feat: introduce new API for FAST hal
fengyuentaud137e0c fix: fix bugs in the new API
fengyuentaua41f63b feat: drop func typedef and use function ptr instead
fengyuentau7f8e331 fix: fix build
fengyuentau390bcf5 doc: add documentation for realloc_func
fengyuentau5f8d2f5 fix: add cv_hal_FASTv2
fengyuentau38bdd05 fix: use different lmul sizes for different specs
fengyuentau3955429 fix: define cv_hal_FASTv2
fengyuentauFile filter
Filter by extension
Conversations
Failed to load comments.
Loading
Uh oh!
There was an error while loading.Please reload this page.
Jump to
Jump to file
Failed to load files.
Loading
Uh oh!
There was an error while loading.Please reload this page.
Diff view
Diff view
feat: introduce new API for FAST hal
- Loading branch information
Uh oh!
There was an error while loading.Please reload this page.
commit7c40a56de6bd37d62e7d94ccb2a6e66319bb6013
There are no files selected for viewing
5 changes: 3 additions & 2 deletionshal/riscv-rvv/include/features2d.hpp
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -11,9 +11,10 @@ namespace cv { namespace rvv_hal { namespace features2d { | ||
| #if CV_HAL_RVV_1P0_ENABLED | ||
| using realloc_func = void* (*)(void*, size_t); | ||
| int FAST(const uchar* src_data, size_t src_step, int width, int height, | ||
Haosonn marked this conversation as resolved. Show resolvedHide resolvedUh oh!There was an error while loading.Please reload this page. | ||
| uchar** keypoints_data, size_t* keypoints_count, | ||
| int threshold, bool nonmax_suppression, int detector_type, realloc_func f); | ||
| #undef cv_hal_FAST | ||
| #define cv_hal_FAST cv::rvv_hal::features2d::FAST | ||
39 changes: 23 additions & 16 deletionshal/riscv-rvv/src/features2d/fast.cpp
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.Learn more about bidirectional Unicode characters
10 changes: 6 additions & 4 deletionsmodules/features2d/src/fast.cpp
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.Learn more about bidirectional Unicode characters
3 changes: 2 additions & 1 deletionmodules/features2d/src/hal_replacement.hpp
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.Learn more about bidirectional Unicode characters
Oops, something went wrong.
Uh oh!
There was an error while loading.Please reload this page.
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.