Skip to content

US_Texas_Diabetes - added download.py - #2141

Open
goutamabrol wants to merge 3 commits into
datacommonsorg:masterfrom
goutamabrol:ustexasdiabetes
Open

US_Texas_Diabetes - added download.py#2141
goutamabrol wants to merge 3 commits into
datacommonsorg:masterfrom
goutamabrol:ustexasdiabetes

Conversation

@goutamabrol

@goutamabrol goutamabrol commented Aug 6, 2026

Copy link
Copy Markdown
Contributor

import name - US_Texas_Diabetes

PR checklist - Doc

Summary of Changes

  • Source is changed because previous source url was not accessable due to geographic restrictions.
  • Created download.py script to download the data from source.
  • Created pvmap in accordance with the new source data.
  • Updated manifest which was pointing towards wrong directories.
  • Updated README.md with correct steps.
  • validation_output from test run - link

@gemini-code-assist gemini-code-assist Bot 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 automates the ingestion of Texas diabetes mortality data by introducing a download script (download.py) for CDC WONDER, updating the PV mapping file, and revising the workflow documentation. Feedback focuses on correcting a configuration filename mismatch in manifest.json, increasing the download retry limit to handle external source instability, and updating the provenance description to consistently reference CDC WONDER.

Comment thread statvar_imports/nyu_diabetes/texas/manifest.json Outdated
Comment thread statvar_imports/nyu_diabetes/texas/download.py Outdated
Comment thread statvar_imports/nyu_diabetes/texas/manifest.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