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

Optimizing admin dashboard add_user_stats #152

Open
JGreenlee opened this issue Nov 1, 2024 · 2 comments
Open

Optimizing admin dashboard add_user_stats #152

JGreenlee opened this issue Nov 1, 2024 · 2 comments

Comments

@JGreenlee
Copy link
Contributor

JGreenlee commented Nov 1, 2024

Optimizing admin dashboard add_user_stats

def add_user_stats(user_data, batch_size=5):

_get_and_store_range on the server already stores pipeline_range to the user profile, which has start_ts and end_ts.
We can extend _get_and_store_range to also store total_trips and labeled_trips.

For last_call, we can look at stats/server_api_time and look at the last GET and last PUT calls

@TeachMeTW

@shankari
Copy link
Contributor

shankari commented Nov 1, 2024

start by moving the _get_and_store_range to a real pipeline stage with its own timer. We can deploy that right now (today). Then when the real changes are done we can see the tradeoff in increased time on the server with decreased time on the admin dashboard.

@shankari
Copy link
Contributor

shankari commented Nov 1, 2024

@TeachMeTW by tonight

  1. add a timer to the "ad-hoc" pipeline step
  2. add timers to parts of the trip segmentation, with suffixes
  3. keep it clean and small so we can get a quick review

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

No branches or pull requests

2 participants