Skip to content

Use circular lut history buffer in computeGradient of HOG#15623

Merged
alalek merged 2 commits intoopencv:3.4from
ChipKerchner:optimizeHOGpipeline
Oct 29, 2019
Merged

Use circular lut history buffer in computeGradient of HOG#15623
alalek merged 2 commits intoopencv:3.4from
ChipKerchner:optimizeHOGpipeline

Conversation

@ChipKerchner
Copy link
Copy Markdown
Contributor

@ChipKerchner ChipKerchner commented Oct 2, 2019

Use circular lut history buffer in computeGradient of HOG.

Reduce lut reads by 2/3 and is 33-52% faster.

@ChipKerchner ChipKerchner changed the title Use circular lut hustory buffer in computeGradient of HOG Use circular lut history buffer in computeGradient of HOG Oct 2, 2019
@ChipKerchner
Copy link
Copy Markdown
Contributor Author

Any traction on this PR? It's been over 3 weeks. It would be nice to have more PRs merged by the end of the month.

Copy link
Copy Markdown
Member

@alalek alalek left a comment

Choose a reason for hiding this comment

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

Thank you for contribution!

Copy link
Copy Markdown
Member

@alalek alalek left a comment

Choose a reason for hiding this comment

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

Looks good to me! Thank you 👍

@alalek alalek merged commit a71ff50 into opencv:3.4 Oct 29, 2019
@alalek alalek mentioned this pull request Oct 29, 2019
collinbrake pushed a commit to collinbrake/opencv that referenced this pull request Nov 1, 2019
* Use circular lut hustory buffer in computeGradient of HOG

* Initialize prefetch data outside main loop.  Avoid code duplication.
@ChipKerchner ChipKerchner deleted the optimizeHOGpipeline branch November 5, 2019 17:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants