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

Predictions are stale too quickly #67

Open
mathcolo opened this issue Aug 9, 2016 · 4 comments
Open

Predictions are stale too quickly #67

mathcolo opened this issue Aug 9, 2016 · 4 comments
Labels

Comments

@mathcolo
Copy link
Owner

mathcolo commented Aug 9, 2016

Predictions are currently pulled every 60 seconds, as are trip positions. Unfortunately, if a client refreshes predictions the second before a pull, then the data age at client refresh is almost a full 120sec.

The fix, I think, is including the pull date in the nextservice response payload. Then we can calculate the data age in client side and subtract that from the prediction. We do that every second and update the value...and boom we have simulated real-time a la Track the T.

thoughts?

/cc issue #65

@mathcolo mathcolo added the bug label Aug 9, 2016
@mathcolo mathcolo self-assigned this Aug 9, 2016
@tinzerillo
Copy link
Contributor

tinzerillo commented Aug 9, 2016

I think this is a good idea! Thinking about the best way to implement

@mathcolo
Copy link
Owner Author

mathcolo commented Aug 10, 2016

Client side of predictions streaming done in commit c2fe057, leaving issue open as the data age piece as a part of the http request is not started

@mathcolo
Copy link
Owner Author

Completed across the stack as of commit 2daae05, keeping ticket open until PR complete

@mathcolo
Copy link
Owner Author

TODO: fix favoritescontroller

@mathcolo mathcolo removed their assignment Jun 8, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants