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

🐛 harmonizer import #3738

Merged
merged 2 commits into from
Dec 18, 2024
Merged

🐛 harmonizer import #3738

merged 2 commits into from
Dec 18, 2024

Conversation

lucasrodes
Copy link
Member

@lucasrodes lucasrodes commented Dec 17, 2024

A bug was introduced into Harmonizer (likely in #3718)

image

/schedule

@lucasrodes lucasrodes marked this pull request as ready for review December 17, 2024 23:16
Copy link

Merge Schedule
Scheduled to be merged the next time the merge action is scheduled via the cron expressions

@owidbot
Copy link
Contributor

owidbot commented Dec 17, 2024

Quick links (staging server):

Site Dev Site Preview Admin Wizard Docs

Login: ssh owid@staging-site-bug-harmonizer-import

chart-diff: ✅ No charts for review.
data-diff: ❌ Found differences
= Dataset garden/health/latest/global_health_mpox
  = Table global_health_mpox
= Dataset garden/who/2024-09-09/flu_test
  = Table flu_test
    ~ Dim country
-       - Removed values: 5 / 72622 (0.01%)
                date   country
          2024-12-09   Algeria
          2024-12-09 Australia
          2024-12-09  Cameroon
          2024-12-09     China
          2024-12-02 Hong Kong
    ~ Dim date
-       - Removed values: 5 / 72622 (0.01%)
            country       date
            Algeria 2024-12-09
          Australia 2024-12-09
           Cameroon 2024-12-09
              China 2024-12-09
          Hong Kong 2024-12-02
    ~ Column denomcombined (changed data)
-       - Removed values: 5 / 72622 (0.01%)
            country       date  denomcombined
            Algeria 2024-12-09             39
          Australia 2024-12-09           2850
           Cameroon 2024-12-09             40
              China 2024-12-09          24892
          Hong Kong 2024-12-02           7751
        ~ Changed values: 4 / 72622 (0.01%)
            country       date  denomcombined -  denomcombined +
          Australia 2024-11-18             2851             2833
          Australia 2024-11-25             3016             2997
          Australia 2024-12-02             2813             2809
              China 2024-12-02            28074            24816
    ~ Column pcnt_poscombined (changed data)
-       - Removed values: 5 / 72622 (0.01%)
            country       date  pcnt_poscombined
            Algeria 2024-12-09         20.512821
          Australia 2024-12-09           1.22807
           Cameroon 2024-12-09              10.0
              China 2024-12-09         13.152821
          Hong Kong 2024-12-02          1.045026
        ~ Changed values: 4 / 72622 (0.01%)
            country       date  pcnt_poscombined -  pcnt_poscombined +
          Australia 2024-11-18            2.279902            2.294388
          Australia 2024-11-25            2.254642            2.268936
          Australia 2024-12-02            2.204053            2.207191
              China 2024-12-02            9.222056           11.266925
= Dataset garden/who/latest/monkeypox
  = Table monkeypox


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-12-18 11:14:46 UTC
Execution time: 17.83 seconds

@lucasrodes lucasrodes merged commit 56ca879 into master Dec 18, 2024
11 checks passed
@lucasrodes lucasrodes deleted the bug-harmonizer-import branch December 18, 2024 11:24
antea04 pushed a commit that referenced this pull request Feb 5, 2025
* 🐛 harmonizer import

* gix
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.

2 participants