-
Notifications
You must be signed in to change notification settings - Fork 10
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
Read only data for the past week by default #51
Comments
we might also want to change all the implementations from |
which is why, I suggest reading for the past 7 days on login Note that in this case, the initial screen that we see when we login cannot show the full sign up trend and trip trend by default because we will only load one week of trip data initially. So we may want to think about UX design of that screen. Should we show the last week and clearly label it as "last week data" or should we just remove the trip trend and show something else meaningful instead? I am fine with starting with something reasonable (e.g. last week trip trend) so we can get this out and then redesigning later. Note also that not all of our tables and data are time dependent. So the participant-level data such as the user table and the demographic table need not be filtered. But the trip table and the trajectory table should be filtered by default. I guess an alternate option might be to also read the trip table only when we access the tab. But I think that is complementary to the one week issue. Because we still may have 20,000 trips and even when we access the trip table, we don't necessarily always want to load it all and hang. So that is a separate change, but we might as well do that when handling this scalability change. |
|
Again, based on our experience from emdash, the default should not be to read data for a short duration (ideally one week/one month). If users choose to read more, they are welcome to.
Right now, the default is everything, and we read it as soon as the user has logged in to the dashboard. So we literally perform the most resource intensive query immediately. When I tried this on one of the admin dashboards, both the user and the trip tables were unresponsive, and I found these errors in the console.
The webapp layer was essentially unresponsive.
I think that we discussed this in one of the early meetings related to the dashboard as well.
The text was updated successfully, but these errors were encountered: