Add WHO Number of people diagnosed with a new episode of pulmonary TB whose disease was bacteriologically confirmed#1934
Conversation
Summary of ChangesHello, I'm Gemini Code Assist1! I'm currently reviewing this pull request and will post my feedback shortly. In the meantime, here's a summary to help you and other reviewers quickly get up to speed! This pull request introduces a new data import pipeline for World Health Organization (WHO) tuberculosis statistics, specifically focusing on bacteriologically confirmed pulmonary TB cases. This new data will enrich our health-related datasets. Concurrently, existing test data for the US Census ACS S1251 subject table has been updated and standardized, ensuring consistency and accuracy in demographic data processing. Highlights
Using Gemini Code AssistThe full guide for Gemini Code Assist can be found on our documentation page, here are some quick tips. Invoking Gemini You can request assistance from Gemini at any point by creating a comment using either
Customization To customize Gemini Code Assist for GitHub experience, repository maintainers can create a configuration file and/or provide a custom code review style guide (such as PEP-8 for Python) by creating and adding files to a Limitations & Feedback Gemini Code Assist may make mistakes. Please leave feedback on any instances where its feedback is incorrect or counter productive. You can react with 👍 and 👎 on @gemini-code-assist comments. If you're interested in giving your feedback about your experience with Gemini Code Assist for GitHub and other Google products, sign up here. Footnotes
|
There was a problem hiding this comment.
Code Review
This pull request introduces a new data import for the "WHO Tuberculosis Dataset: Bacteriologically Confirmed Pulmonary TB", including a Python script to download and process the data, and associated configuration files. It also updates existing ACS5YR subject table files (S1251) to reflect a new data year (2024) and adjusts some property ordering and node names. Feedback includes addressing a missing test data file or generation instructions for the new TB import, updating documentation for filename consistency, and adding warning logs for unmatched countries during data merging.
35c3da1 to
6879158
Compare
|
/gemini review |
There was a problem hiding this comment.
Code Review
This pull request removes an old ACS dataset and introduces a new import for the WHO Tuberculosis dataset. The new import includes a Python script to download and process the data, along with configuration files for statistical variable mapping and schema definitions. A critical issue was identified in the download_tb_data.py script where the master_response variable was used without being defined, which needs to be fixed to correctly fetch the master data.
2877bbb to
aaf18fd
Compare
description: "Number of people diagnosed with a new episode of pulmonary TB whose disease was bacteriologically confirmed."