-
Notifications
You must be signed in to change notification settings - Fork 23
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
[staging] Monthly sensing metrics are not showing up; overall ones do work #93
Comments
I see this error
And then we move on to
|
This is because we didn't fill in the section summaries (backwards compat) for staging. |
Older months are still broken with the same error.
|
This may be just because we are not dealing properly with an empty dataframe (e.g. a month in which we have no data).
which was added to handle the case that the column didn't exist Let's add a similar check to the sensed values. |
After adding the check, we move past that cell, but run into an error while generating the graph.
|
It looks like |
Ah but I am not sure where we got the Maybe there is a copy somewhere that does use logging? |
OK, with those fixes, I don't see errors any more
|
Tested against a blank database, works
|
Fixed in #94 After pushing to staging, we see entries for monthly metrics. Note that the very old months still don't have any values, but neither do the metrics for the labeled values. At some point, we need to see why the older months are not showing up on staging, but that is outside the scope of this change. |
This is working fine on production for:
However, MassCEC is not even displaying the aggregate values On investigating further, I see
from
This is because the last few entries don't have a cleaned section summary
|
Or
|
They appear to occur intermittently through the dataset
|
Huh! The values are fairly old.
They were also generated a fairly long time ago
|
There are definitely some from fairly recent runs though - let's look them up and see what happened...
|
There's probably an error fairly high in the stack for the monthly launches.
The text was updated successfully, but these errors were encountered: