Skip to content

use fused kernels to reduce overhead in corner detector implementations - #426

Merged
rapids-bot[bot] merged 6 commits into
rapidsai:branch-22.12from
grlee77:corners-fused-kernels
Nov 18, 2022
Merged

rapids-bot[bot] merged 6 commits into
rapidsai:branch-22.12from
grlee77:corners-fused-kernels

Conversation

@grlee77

@grlee77 grlee77 commented Nov 1, 2022

Copy link
Copy Markdown
Contributor

This PR does not change behavior, but merely replaces some existing code with equivalent fused kernels for efficiency. The functions updated are: corner_foerstner, corner_harris, corner_kitchen_rosenfeld, corner_shi_tomasi.

@grlee77 grlee77 added non-breaking Introduces a non-breaking change performance Performance improvement labels Nov 1, 2022
@grlee77 grlee77 added this to the v22.12.00 milestone Nov 1, 2022
@grlee77
grlee77 requested a review from a team as a code owner November 1, 2022 17:32
@grlee77 grlee77 added the improvement Improves an existing functionality label Nov 1, 2022

@gigony gigony left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @grlee77 ! Looks good to me!

@grlee77

grlee77 commented Nov 18, 2022

Copy link
Copy Markdown
Contributor Author

@gpucibot merge

@rapids-bot
rapids-bot Bot merged commit 56d3a9e into rapidsai:branch-22.12 Nov 18, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

improvement Improves an existing functionality non-breaking Introduces a non-breaking change performance Performance improvement

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

2 participants