Uh oh!
There was an error while loading. Please reload this page.
Mexico_census_aa2|validation failure fix - #2140
Conversation
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. |
There was a problem hiding this comment.
Code Review
This pull request adds validation configuration, golden data, and counters for the Mexico subnational population statistics import. Specifically, it introduces validation rules in validation_config.json, adds several golden observation and summary report CSV files, includes counter files, updates mexico_places.csv to map 'MX' to country/MEX, and updates manifest.json to integrate these validation steps and output counters. I have no feedback to provide as there are no review comments and the changes appear correct.
091daea to
cce44b2Comparepulkeet-wq
commented
Aug 7, 2026
Counter folder not required in Github. Please remove the folder and all file @Ashwani725 |
…o_census_aa2/counters/mexico_census_data_counters_0.csv delete counter file as not required
…o_census_aa2/counters/mexico_census_data_counters_1.csv
…o_census_aa2/counters/mexico_census_data_counters_2.csv
saanikaaa
left a comment
There was a problem hiding this comment.
Lets add a better PR description
The PR is raised to fix Mexico_census_aa2 import cloud job which was failing at validation stage due to missing mcf file and also to add the golden test for the same.
The Auto-refresh job errors caused by upstream place_name changes that were resulting in blank output CSVs in the previous timestamp.
Testing done successfully. Added Golden Checks and counters.
Files added are : golden_data/, validation_config.json, mexico_places.csv, counters/