[IndiaNSS_HealthAilments] Made a change in pv map - #2196
[IndiaNSS_HealthAilments] Made a change in pv map#2196Krishnam24maheshwari wants to merge 2 commits into
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 updates the CSV mapping file for India NSS health ailments by changing the classification of 'Rural + Urban' from ignored to 'placeOfResidenceClassification'. However, the modified row contains an extra trailing comma, which could lead to parsing errors during ingestion. This formatting issue should be corrected to match the rest of the file.
Krishnam24maheshwari
left a comment
There was a problem hiding this comment.
Done
In this PR - The one line of pvmap is restored to make sure no nodes are deleted.
validation output - https://pantheon.corp.google.com/storage/browser/_details/datcom-import-test/statvar_imports/india_ndap/india_nss_health_ailments/IndiaNSS_HealthAilments/2026_08_31T23_35_10_310556_07_00/input0/validation/validation_output.csv;tab=live_object?pageState=(%22StorageObjectListTable%22:(%22f%22:%22%255B%255D%22))&project=datcom-infosys-dev