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

minimize frame skip so FPS will be calculated ASAP #13658

Open
wants to merge 4 commits into
base: development
Choose a base branch
from

Conversation

gilpazintel
Copy link

@gilpazintel gilpazintel commented Jan 7, 2025

The root cause is the unsigned long underflow when we playback a bag file.
when we finish the file and go back to the first frame, the current frame - last frame turns out to be negative (underflow) which cases the invalid value display on the UI

Tracked on [RSDSO-19945]

also handled some non-proper comparison with floating point number.
until FPS calculation is ready, display '--' on screen (instead of zero)
@gilpazintel gilpazintel requested a review from Nir-Az January 7, 2025 09:19
@Nir-Az Nir-Az changed the title minimize frame skip so FPS will be calculated ASAP. RSDSO-19945 minimize frame skip so FPS will be calculated ASAP Jan 7, 2025
common/rendering.h Outdated Show resolved Hide resolved
common/rendering.h Outdated Show resolved Hide resolved
common/stream-model.cpp Outdated Show resolved Hide resolved
the root cause is the unsigned long underflow when we playback a bag file.
when we finish the file and go back to the first frame, the current frame - last frame turnes out top be negative (underflow) which cases the invalid value display on the UI

Revert "minimize frame skip so FPS will be calculated ASAP."

This reverts commit 8a5c42c.
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