Skip to content

Conversation

@melange396
Copy link
Contributor

@melange396 melange396 commented Oct 23, 2025

adds a set of flu signals to the output, taken from a previously unused column in the input files.

also includes an annoying amount of formatting changes to unrelated or barely related code, because the linter is being applied to these ancient files in drips and drops 😡 (look at the first commit for a more concise view of the functional changes)

a quick review would be appreciated so i can get this deployed to the staging server as soon as possible.

Copy link
Contributor

@aysim319 aysim319 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

claims local run just finished.

{"elapsed_time_in_seconds": 2083.01, "csv_export_count": 50112, "max_lag_in_days": 4, "oldest_final_export_date": "2025-10-19", "event": "Completed indicator run", "logger": "delphi_claims_hosp.run", "level": "info", "pid": 12584, "timestamp": "2025-10-23T21:41:29.131204Z"}

and looks like there are equal number of flu signal as covid which is 25056
and given the date difference of 2088 days inclusive gives us

25056 / 2088 = 12 (6 geo resolution * 2 time transformation )

I took a quick look and it seems that at least there are same amount of data and looking at one of the latest data the generate numbers seems to correlate with https://www.cdc.gov/respiratory-viruses/data/activity-levels.html

Image

for 20250920 the nation flu was about 0.01 and covid was 0.3 which seems to be proportional to the data table.

since this is a high priority I approved it knowing that I only did superficial checks. Are we planning on going over the statistical analysis for the flu signal?

max_dates.append(updater.output_dates[-1])
n_csv_export.append(len(updater.output_dates))

for numerator_name in ["Covid_like", "Flu1"]:
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Non-blocking: do you know when the Flu1 column became available / do you know if existence of this column is flaky?

@melange396 melange396 merged commit 6ee92cc into main Oct 23, 2025
17 checks passed
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

Successfully merging this pull request may close these issues.

3 participants