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

Flutter: measurements.frames_total less than measurements.frames_slow #2304

Closed
ReneGreen27 opened this issue Sep 19, 2024 · 4 comments · Fixed by #2308
Closed

Flutter: measurements.frames_total less than measurements.frames_slow #2304

ReneGreen27 opened this issue Sep 19, 2024 · 4 comments · Fixed by #2308

Comments

@ReneGreen27
Copy link
Member

Problem Statement

When viewing measurements.frames_total on the Discover page there are instances where it is less than measurements.frames_slow, which is causing measurements.frames_slow_rate to be 200%.

Image

User is currently using sentry_flutter: ^8.6.0

Solution Brainstorm

total frames should always be greater or equal to the number of slow frames.

Can reference latest exchange and org in internal ticket 131803 for more examples if needed

Product Area

Discover

@getsantry
Copy link

getsantry bot commented Sep 19, 2024

Assigning to @getsentry/support for routing ⏲️

@getsantry
Copy link

getsantry bot commented Sep 19, 2024

Routing to @getsentry/product-owners-discover for triage ⏲️

@Dhrumil-Sentry Dhrumil-Sentry transferred this issue from getsentry/sentry Sep 20, 2024
@buenaflor
Copy link
Contributor

thanks for the report, we'll have a look

@buenaflor
Copy link
Contributor

buenaflor commented Sep 25, 2024

@ReneGreen27 do you know if this is reproducible on your end?

I'll add a safe guard that will prevent invalid data from being sent but will investigate why this is happening

Edit: this is happening because of a rounding error

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Archived in project
Archived in project
Development

Successfully merging a pull request may close this issue.

2 participants