Param Labeling - TB

[mmsuarezcosta]

To Note:

• Code and comment not aligned. Comment states 14d between tb test, and it is coded as 7 days:

	# if person has tested within last 14 days, do nothing
	if person["tb_date_tested"] >= (
	    self.sim.date - DateOffset(days=self.module.parameters["tb_min_days_between_tests"])
	):
	    return self.sim.modules["HealthSystem"].get_blank_appt_footprint()

• Calibration is not disclosed in the write-up
• Variables documented as 'assumed'
  • rr_tb_hiv ,5, universal, 0.01, 100, vague prior, 20.6getahun2010, assumed
  • rr_tb_aids, 26, universal, 0.01, 100, vague prior,26williams_pnas, assumed
  • rr_tb_child

Undetermined, Calibration

• death_rate_adult_treated : note that changed from 0.086, so assuming this was for calibration
• rr_tb_hiv: note 20.6getahun2010 and value is 5, so assuming this was changed for calibration

Local

• Treatment success rates deemed to be local (come from National Tuberculosis Programme 2019)
• Regimen for first and second line tests
• _scheduling_window_months/days: believe that these parameters are inherent to the local healthcare infrastructure and day estimates to attain care or medication

Assumed

The following state to have been assumed in the code, but they have references documented. Are they assumed because they have been calibrated? rr_tb_hiv has a different value than the reference, while rr_tb_aids has the same value as the reference.

• rr_tb_hiv ,5, universal, 0.01, 100, vague prior, 20.6getahun2010, assumed
• rr_tb_aids, 26, universal, 0.01, 100, vague prior,26williams_pnas, assumed

[marghe-molaro]

Hi @mmsuarezcosta, this looks good, I have just modified the scale-up parameters to be labelled as "scenario" without priors

I think this is now ready for @tdm32's review