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

✨ flu: use only positive and negative tests in denominator #3271

Draft
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

spoonerf
Copy link
Contributor

No description provided.

@owidbot
Copy link
Contributor

owidbot commented Sep 11, 2024

Quick links (staging server):

Site Dev Site Preview Admin Wizard Docs

Login: ssh owid@staging-site-flunet-simp

chart-diff: ✅ No charts for review.
data-diff: ❌ Found differences
= Dataset garden/who/2024-09-09/flu_test
  = Table flu_test
    ~ Dim country
-       - Removed values: 49671 / 21515 (230.87%)
                date     country
          2015-01-19      Russia
          2013-10-14 South Korea
          2008-05-05    Thailand
          2011-03-21        Togo
          2021-10-11  Uzbekistan
    ~ Dim date
-       - Removed values: 49671 / 21515 (230.87%)
              country       date
               Russia 2015-01-19
          South Korea 2013-10-14
             Thailand 2008-05-05
                 Togo 2011-03-21
           Uzbekistan 2021-10-11
    ~ Column denomcombined (changed data)
-       - Removed values: 49671 / 21515 (230.87%)
              country       date  denomcombined
               Russia 2015-01-19           1967
          South Korea 2013-10-14            277
             Thailand 2008-05-05             48
                 Togo 2011-03-21              1
           Uzbekistan 2021-10-11              5
    ~ Column pcnt_poscombined (changed data)
-       - Removed values: 49671 / 21515 (230.87%)
              country       date  pcnt_poscombined
               Russia 2015-01-19         22.013218
          South Korea 2013-10-14          0.361011
             Thailand 2008-05-05          8.333333
                 Togo 2011-03-21        100.000000
           Uzbekistan 2021-10-11         60.000000


Legend: +New  ~Modified  -Removed  =Identical  Details
Hint: Run this locally with etl diff REMOTE data/ --include yourdataset --verbose --snippet

Automatically updated datasets matching weekly_wildfires|excess_mortality|covid|fluid|flunet|country_profile|garden/ihme_gbd/2019/gbd_risk are not included

Edited: 2024-11-20 10:37:56 UTC
Execution time: 3.93 seconds

@lucasrodes lucasrodes changed the title ✨ Using only positive and negative tests in denominator ✨ flu: use only positive and negative tests in denominator Sep 18, 2024
@lucasrodes
Copy link
Member

@spoonerf
what's the status of this PR?

(no urgency, just doing some 🧹)

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