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

Auto Time to Initial Display for React Navigation is always 15 seconds #3786

Closed
krystofwoldrich opened this issue Apr 29, 2024 · 10 comments
Closed

Comments

@krystofwoldrich
Copy link
Member

Description

On some Android devices the auto instrumentation for time to initial display results in heartbeat timeout and the TTID span is always 15 seconds.

To avoid this bug you can you the manual instrumentation <Sentry.TimeToInitialDisplay record={true} />

https://docs.sentry.io/platforms/react-native/performance/instrumentation/time-to-display/

@krystofwoldrich
Copy link
Member Author

@matheuscouto
Copy link

matheuscouto commented Jun 3, 2024

Moving the discussion to this thread, as you asked in the issue, I was using only real devices for those tests, and my navigators are structured like this:

Stack Navigator (root)
   └─ Tab Navigator
      └─ Stack Navigator

Tbh I didn't know we also could define the TimeToInitialDisplay the same way you can with the TimeToFullDisplay, I was relying only on auto instrumentation for that. I'll add it and see if it solves the issue.

@krystofwoldrich
Copy link
Member Author

@matheuscouto Thank you for the details.

@krystofwoldrich krystofwoldrich moved this from Needs Investigation to Backlog in Mobile & Cross Platform SDK Jun 7, 2024
@krystofwoldrich krystofwoldrich moved this from Backlog to Todo in Mobile & Cross Platform SDK Jun 7, 2024
@kahest
Copy link
Member

kahest commented Jul 29, 2024

Just dropping a note here that we have at least one more instance of this reported on another channel

@sregg
Copy link

sregg commented Aug 29, 2024

It seems that manually using <Sentry.TimeToInitialDisplay record={true} /> is helping.
But for screens without that, we're seeing those 15s TTID and TTFD even with enableTimeToInitialDisplay: false.
Is there a way to ignore/filter-out those 15s TTID and TTFD spans?

@getsantry getsantry bot moved this to Waiting for: Product Owner in GitHub Issues with 👀 3 Aug 29, 2024
@krystofwoldrich krystofwoldrich moved this from Todo to In Progress in Mobile & Cross Platform SDK Aug 30, 2024
@krystofwoldrich
Copy link
Member Author

Hi @sregg,
thank you for confirming that the manual instrumentation works.

But for screens without that, we're seeing those 15s TTID and TTFD even with enableTimeToInitialDisplay: false.

Are those possibly from old releases? Or do you see it when testing locally?

@getsantry getsantry bot moved this from Waiting for: Community to Waiting for: Product Owner in GitHub Issues with 👀 3 Sep 2, 2024
@getsantry getsantry bot moved this to Waiting for: Community in GitHub Issues with 👀 3 Sep 2, 2024
@getsantry getsantry bot moved this from Waiting for: Product Owner to Waiting for: Community in GitHub Issues with 👀 3 Sep 2, 2024
@sregg
Copy link

sregg commented Sep 2, 2024

Are those possibly from old releases? Or do you see it when testing locally?

Locally. Even when we instrument manually a screen properly, we see the 15s timeout when navigating back to that screen.
Tests are on iOS.

@getsantry getsantry bot moved this from Waiting for: Community to Waiting for: Product Owner in GitHub Issues with 👀 3 Sep 2, 2024
@krystofwoldrich
Copy link
Member Author

Thank you for the explanation @sregg

@sregg
Copy link

sregg commented Sep 3, 2024

Thank you for the explanation @sregg

I'm happy to jump on a quick call to show you how this Time to Display is working in our app. There seems to be a few bugs at the moment.

@getsantry getsantry bot moved this to Waiting for: Product Owner in GitHub Issues with 👀 3 Sep 3, 2024
@krystofwoldrich
Copy link
Member Author

Thank you,

we are currently working on a fix -> #4042,

let's wait for it.

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

No branches or pull requests

5 participants