Skip to content

OECDRegionalDemography_Population| Fixed missing data issue - #2152

Open
Ashwani725 wants to merge 2 commits into
datacommonsorg:masterfrom
Ashwani725:OECDRegionalDemography_Population_missing_data
Open

OECDRegionalDemography_Population| Fixed missing data issue#2152
Ashwani725 wants to merge 2 commits into
datacommonsorg:masterfrom
Ashwani725:OECDRegionalDemography_Population_missing_data

Conversation

@Ashwani725

Copy link
Copy Markdown

2024 onwards source data mapping has changed due to which data is not refreshed. Fixed this by adding the missing mapping in the schema
files changed are: OECD_population_stat_vars.mcf, columns.py, manifest.json, golden_data/, validation_config.json

@google-cla

Copy link
Copy Markdown

Thanks for your pull request! It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).

View this failed invocation of the CLA check for more information.

For the most up to date status, view the checks section at the bottom of the pull request.

@gemini-code-assistgemini-code-assistBot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Review

This pull request introduces new statistical variables for age groups 20-64, 25-54, and 55-64, including gender-specific breakdowns, and updates the associated column mappings and golden data. Additionally, a new validation configuration file is added to the manifest. The reviewer recommended updating the rule_id in the validation configuration for consistency and noted that the expected test files must be updated to reflect these schema changes to prevent unit test failures.

Comment threadscripts/oecd/regional_demography/population/columns.py
@Ashwani725Ashwani725 changed the title fixed missing data issueOECDRegionalDemography_Population| Fixed missing data issueAug 11, 2026
Sign up for freeto 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.

1 participant

@Ashwani725