-
-
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
📊 marriages and divorces: add OECD data on average age at marriage #3827
Conversation
Quick links (staging server):
Login: chart-diff: ✅No charts for review.data-diff: ✅ No differences found+ Dataset garden/oecd/2025-01-11/mean_age_at_marriage
+ + Table mean_age_at_marriage
+ + Column mean_age_at_first_marriage
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: 2025-01-15 11:19:23 UTC |
etl/steps/data/garden/oecd/2025-01-11/mean_age_at_marriage.meta.yml
Outdated
Show resolved
Hide resolved
etl/steps/data/garden/oecd/2025-01-11/mean_age_at_marriage.meta.yml
Outdated
Show resolved
Hide resolved
etl/steps/data/garden/oecd/2025-01-11/mean_age_at_marriage.meta.yml
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM! Thanks for adding the data set! I only left some small comments re: metadata.
Mainly I think the description/ definition should be clearer: I am not sure what this really measures - I would assume average age of people who get married this year, but we could make that more explicit. If it is not that, then that should be clear from the metadata as well.
1b569bc
to
80562c9
Compare
no charts updates yet as will be combined with the UN data later on once that's ready