-
Notifications
You must be signed in to change notification settings - Fork 309
profile: Display user's local time #292
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
Comments
Adding for future note: switching to the 10y db (that is 25% the size of the normal db) cut the initialization time to ~50ms. However, since all we are doing is showing the current time, I wonder how much we can cut this database down by as we won't need historic information at all. Will we ever need to render historic datetimes to some arbitrary local time? |
Added some thoughts in chat. In short: that 10y database is a good find and we should totally use it; but probably cutting the database down further wouldn't get us below like 30-40ms, so not enough gain to be worth modifying the package. |
Also it'd be good to repeat the measurement of the initialization time (a) in release mode or profile mode rather than debug mode, (b) on phone hardware rather than desktop hardware. I expect it won't be so radically faster as to change the conclusion that we should defer the initialization away from app startup, but it might affect how we feel about different kinds of mitigations at the time when the user opens the profile screen. |
Hello, I'm an new contributor. |
This is my draft PR: #F1318 |
In the profile screen (#195 ) we should show the current time in the user's configured timezone. This will involve integrating timezone data.
We've identified https://pub.dev/packages/timezone as a good candidate (chat thread) and an initial attempt was already made to integrate it, but the initialization of the database was measured to take ~100ms during bootup. It would make sense to delay the initialization of the database until we need to display the profile screen.
A followup issue will be:
The text was updated successfully, but these errors were encountered: