Eurostat Data Marriages_by_sex_and_previous_marital_status & Population_on_1_January_by_age_and_sex#2051
Conversation
…on_on_1_January_by_age_and_sex
![gemini-code-assist[bot]](https://avatars.githubusercontent.com/in/956858?s=60&v=4){kind=small}
There was a problem hiding this comment.
Code Review
This pull request introduces two new Eurostat statistical variable imports: "Marriages by sex and previous marital status" and "Population on 1 January by age and sex". Each import includes metadata, property-value mappings, place resolution configurations, automated execution scripts, and test datasets. Feedback on the changes suggests mapping the infant cohort Y_LT1 to [0 0 Years] in the population property-value map instead of ignoring it, which prevents data loss for this critical age group.
| # --- Age Group Normalization --- | ||
| age:TOTAL,age,"",,, | ||
| age:UNK,#ignore,Zero values for unknown age,,, | ||
| age:Y_LT1,#ignore,Ignore infant rows,,, |
There was a problem hiding this comment.
Instead of ignoring the infant cohort Y_LT1 (under 1 year old), map it to [0 0 Years] so that the demographic data for age 0 is preserved. This prevents data loss for a critical cohort and ensures that total population sums across ages remain accurate.
age:Y_LT1,age,"[0 0 Years]",,,
2 participants
PL Checklist: https://drive.google.com/corp/drive/folders/1KR9Cd4AEgSzeI7I2x2mR0iTAicos4_tG?resourcekey=0-USjPGypctswXXvC-3avmoA