Skip to content

added golden#2048

Open
niveditasing wants to merge 5 commits into
datacommonsorg:masterfrom
niveditasing:goldes_pop_estimate
Open

added golden#2048
niveditasing wants to merge 5 commits into
datacommonsorg:masterfrom
niveditasing:goldes_pop_estimate

Conversation

@niveditasing
Copy link
Copy Markdown
Contributor

@niveditasing niveditasing commented May 29, 2026

This import is experiencing temporary deletions—ranging from less than 0.001% up to 0.13% of the data—due to an unresponsive source URL. To ensure the pipeline remains functional, we increased the deletion threshold to 0.15%. This prevents pipeline failure when the source URL is down, allowing the pipeline to refresh properly once the URL is operational again.

Copy link
Copy Markdown
Contributor

@gemini-code-assist gemini-code-assist Bot left a comment

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 adds several golden data CSV files and introduces a validation configuration file (validation_config.json) along with its reference in the manifest to enable automated validation checks for the US Census PEP population estimate by race import. Feedback on the changes highlights a critical filename mismatch in the validation configuration that would cause a file-not-found error, as well as recommendations to ensure all validation rules have unique rule_id values for clearer reporting.

Comment thread scripts/us_census/pep/population_estimate_by_race/validation_config.json Outdated
Comment thread scripts/us_census/pep/population_estimate_by_race/validation_config.json Outdated
Comment thread scripts/us_census/pep/population_estimate_by_race/validation_config.json Outdated
Sign up for free to 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