Skip to content

Who Treatment Outcome TB and HIV#1937

Merged
niveditasing merged 8 commits intodatacommonsorg:masterfrom
HarishC727:treatment_tbhiv
Apr 15, 2026
Merged

Who Treatment Outcome TB and HIV#1937
niveditasing merged 8 commits intodatacommonsorg:masterfrom
HarishC727:treatment_tbhiv

Conversation

@HarishC727
Copy link
Copy Markdown
Contributor

@HarishC727 HarishC727 commented Mar 27, 2026

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 introduces a new statvar import for WHO Tuberculosis treatment outcomes among people living with HIV, including a data download script, manifest, and mapping files. The reviewer feedback highlights a redundant network request in the Python script, incorrect indicator URLs in the documentation and manifest, and several formatting issues such as trailing spaces and missing trailing newlines across multiple files.

Comment thread statvar_imports/who_tuberculosis/treatmentoutcome_tb_hiv/tb_data_download_who.py Outdated
Comment thread statvar_imports/who_tuberculosis/treatmentoutcome_tb_hiv/README.md
Comment thread statvar_imports/who_tuberculosis/treatmentoutcome_tb_hiv/README.md
Comment thread statvar_imports/who_tuberculosis/treatmentoutcome_tb_hiv/metadata.csv Outdated
Comment thread statvar_imports/who_tuberculosis/treatmentoutcome_tb_hiv/manifest.json Outdated
@HarishC727
Copy link
Copy Markdown
Contributor Author

HarishC727 commented Mar 30, 2026 via email

Comment thread statvar_imports/who_tuberculosis/treatmentoutcome_tb_hiv/README.md
Comment thread statvar_imports/who_tuberculosis/treatmentoutcome_tb_hiv/manifest.json Outdated
@niveditasing
Copy link
Copy Markdown
Contributor

/gcbrun

@niveditasing niveditasing merged commit f5900c7 into datacommonsorg:master Apr 15, 2026
3 checks passed
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.

4 participants