Skip to content

minimize frame skip so FPS will be calculated ASAP - #13658

Merged
Nir-Az merged 4 commits into
realsenseai:developmentfrom
gilpazintel:FPS_metadata_invalid_RSDSO_19945
Jan 12, 2025
Merged

minimize frame skip so FPS will be calculated ASAP#13658
Nir-Az merged 4 commits into
realsenseai:developmentfrom
gilpazintel:FPS_metadata_invalid_RSDSO_19945

Conversation

@gilpazintel

@gilpazintel gilpazintel commented Jan 7, 2025

Copy link
Copy Markdown
Contributor

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
Comment thread common/rendering.h Outdated
Comment thread common/rendering.h Outdated
Comment thread common/stream-model.cpp Outdated
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.
@Nir-Az
Nir-Az merged commit fbcc37d into realsenseai:development Jan 12, 2025
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