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

✨ hfd: review metadata #3606

Closed
wants to merge 6 commits into from
Closed

✨ hfd: review metadata #3606

wants to merge 6 commits into from

Conversation

lucasrodes
Copy link
Member

@lucasrodes lucasrodes commented Nov 25, 2024

This PR is so that Saloni can review the metadata.

@owidbot
Copy link
Contributor

owidbot commented Nov 25, 2024

Quick links (staging server):

Site Dev Site Preview Admin Wizard Docs

Login: ssh owid@staging-site-hfd-metadata-review

chart-diff: ✅ No charts for review.
data-diff: ❌ Found differences
= Dataset garden/hmd/2024-11-19/hfd
  = Table period
    ~ Column adjtfr (changed metadata)
-       -   <% set title = "Tempo-adjusted total fertility rates" %>
        ?                                                      -
+       +   <% set title = "Tempo-adjusted total fertility rate" %>
+       +   <%- elif birth_order == '5p' %>
+       +   << title >> - Birth order: ≥5
    ~ Column b (changed metadata)
+       +   <%- elif birth_order == '5p' %>
+       +   << title >> - Birth order: ≥5
    ~ Column cbr (changed metadata)
+       +   <%- elif birth_order == '5p' %>
+       +   << title >> - Birth order: ≥5
    ~ Column mab (changed metadata)
+       +   <%- elif birth_order == '5p' %>
+       +   << title >> - Birth order: ≥5
    ~ Column mab40 (changed metadata)
+       +   <%- elif birth_order == '5p' %>
+       +   << title >> - Birth order: ≥5
    ~ Column patfr (changed metadata)
+       +   <%- elif birth_order == '5p' %>
+       +   << title >> - Birth order: ≥5
    ~ Column sdmab (changed metadata)
+       +   <%- elif birth_order == '5p' %>
+       +   << title >> - Birth order: ≥5
    ~ Column sdmab40 (changed metadata)
+       +   <%- elif birth_order == '5p' %>
+       +   << title >> - Birth order: ≥5
    ~ Column tfr (changed metadata)
-       -   <% set title = "Period total fertility rates" %>
        ?                                              -
+       +   <% set title = "Period total fertility rate" %>
+       +   <%- elif birth_order == '5p' %>
+       +   << title >> - Birth order: ≥5
    ~ Column tfr40 (changed metadata)
-       -   <% set title = "Period total fertility rates by age 40" %>
        ?                                              -
+       +   <% set title = "Period total fertility rate by age 40" %>
+       +   <%- elif birth_order == '5p' %>
+       +   << title >> - Birth order: ≥5
    ~ Column tmab (changed metadata)
+       +   <%- elif birth_order == '5p' %>
+       +   << title >> - Birth order: ≥5
  = Table cohort
    ~ Column ccf (changed metadata)
+       +   <%- elif birth_order == '5p' %>
+       +   << title >> - Birth order: ≥5
    ~ Column ccf40 (changed metadata)
+       +   <%- elif birth_order == '5p' %>
+       +   << title >> - Birth order: ≥5
    ~ Column cmab (changed metadata)
+       +   <%- elif birth_order == '5p' %>
+       +   << title >> - Birth order: ≥5
    ~ Column cmab40 (changed metadata)
+       +   <%- elif birth_order == '5p' %>
+       +   << title >> - Birth order: ≥5
    ~ Column ppr (changed metadata)
-       - title: Cohort parity progression ratios - << birth_order | int >> to << (birth_order | int) + 1 >> birth
+       + title: Cohort parity progression ratio - << (birth_order | int) - 1 >> to << (birth_order | int) >> birth
    ~ Column sdcmab (changed metadata)
+       +   <%- elif birth_order == '5p' %>
+       +   << title >> - Birth order: ≥5
    ~ Column sdcmab40 (changed metadata)
+       +   <%- elif birth_order == '5p' %>
+       +   << title >> - Birth order: ≥5
  = Table period_ages
    ~ Column asfr_period (changed metadata)
-       -   <% set title = "Period fertility rates" %>
        ?                                        -
+       +   <% set title = "Period fertility rate" %>
+       +   <% elif birth_order == '5p' %>
+       +   << title >> - Mother age: << age_str >> - Birth order: ≥5
  = Table cohort_ages
    ~ Column asfr_cohort (changed metadata)
-       -   <% set title = "Cohort fertility rates" %>
        ?                                        -
+       +   <% set title = "Cohort fertility rate" %>
+       +   <% elif birth_order == '5p' %>
+       +   << title >> - Mother age: << age_str >> - Birth order: ≥5
+ Dataset garden/imf/2024-11-25/world_economic_outlook
+ + Table world_economic_outlook
+   + Column gross_domestic_product__constant_prices__percent_change_forecast
+   + Column gross_domestic_product__constant_prices__percent_change_observation
+   + Column unemployment_rate__percent_of_total_labor_force_forecast
+   + Column unemployment_rate__percent_of_total_labor_force_observation


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-26 10:28:41 UTC
Execution time: 14.83 seconds

@lucasrodes lucasrodes requested a review from saloni-nd November 25, 2024 18:28
@lucasrodes
Copy link
Member Author

closing because it was not used

@lucasrodes lucasrodes closed this Dec 6, 2024
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