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

Cache travel time results in the database #168

Closed
Nate-Wessel opened this issue Dec 10, 2024 · 2 comments · Fixed by #171
Closed

Cache travel time results in the database #168

Nate-Wessel opened this issue Dec 10, 2024 · 2 comments · Fixed by #171

Comments

@Nate-Wessel
Copy link
Contributor

Nate-Wessel commented Dec 10, 2024

Once results are calculated, it wouldn't take much extra time/effort to store them in the database, along with all the params that defined the request, such as we started doing here: https://github.com/Toronto-Big-Data-Innovation-Team/cycling_before_after/blob/main/motor_vehicle_travel_times/motor_vehicle_travel_times.sql

Any travel time request then could start with a quick lookup to see if we already know the answer... and end if necessary with an update to that table (possibly after returning results to keep things snappy?).

Would want to make the app backend aware of its version (commit hash) as well to ensure consistency in case the calculations change.

@Nate-Wessel
Copy link
Contributor Author

@AdK0101 was working on a script in a similar vein here

@Nate-Wessel
Copy link
Contributor Author

To make this really snappy, we might want to drop the links, etc from the travel time output. They're never really used there anyway, to my knowledge.

@Nate-Wessel Nate-Wessel linked a pull request Dec 17, 2024 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant