Skip to content
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

Development improve performance on zero dist frames #13657

Conversation

gilpazintel
Copy link

@gilpazintel gilpazintel commented Jan 7, 2025

in case the intrinsic coefficients are zero, we can avoid going thru a loop 10 times and by that improve code efficiency.
per a check, image process time reduced from ~90ms to ~10ms

Should address #12660

Tracked on [LRS-1030]

@gilpazintel gilpazintel marked this pull request as ready for review January 7, 2025 07:54
@gilpazintel gilpazintel closed this Jan 7, 2025
@gilpazintel gilpazintel reopened this Jan 7, 2025
@gilpazintel gilpazintel requested a review from Nir-Az January 7, 2025 08:43
src/rs.cpp Outdated Show resolved Hide resolved
@Nir-Az Nir-Az requested a review from OhadMeir January 7, 2025 09:00
@Nir-Az
Copy link
Collaborator

Nir-Az commented Jan 7, 2025

Hi @OhadMeir since you are familiar with this code can you help me review it please?

@OhadMeir
Copy link
Contributor

OhadMeir commented Jan 7, 2025

Please add a short description to the PR.
If source of change is a Jira ticket you can refer to it - Tracked on [ticket ID]

src/rs.cpp Outdated Show resolved Hide resolved
src/rs.cpp Outdated Show resolved Hide resolved
@Nir-Az Nir-Az changed the title Development improve performance on zero dist frames lrs 1030 Development improve performance on zero dist frames Jan 7, 2025
@Nir-Az Nir-Az merged commit fd02c62 into IntelRealSense:development Jan 7, 2025
23 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants