-
Notifications
You must be signed in to change notification settings - Fork 15
Updated schistosomiasis model #1587
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
extract deaths per py
# Conflicts: # resources/ResourceFile_HIV.xlsx # resources/ResourceFile_TB.xlsx # src/tlo/methods/malaria.py # src/tlo/methods/tb.py
set up test runs
# Conflicts: # resources/ResourceFile_HIV.xlsx # src/tlo/methods/enhanced_lifestyle.py # src/tlo/methods/hiv.py
add new property susceptibility add susceptibility when initialising population
…or future MDA programs
…care seeking as no longer used
@tbhallett This is now ready for review again. Symptoms have been simplified, ReallySimplifiedBirths has been removed and some duplicative code has been restructured. |
…s (so that log file always look the same)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Some changes needed for idiomatic pandas.
add in update_infection_symtpoms for initial population
Co-authored-by: Asif Tamuri <tamuri@gmail.com>
@tamuri this is ready to go now I think |
self.sim.modules['SymptomManager'].register_symptom( | ||
Symptom(name='ss_sm_moderate')) | ||
|
||
self.sim.modules['SymptomManager'].register_symptom( | ||
Symptom(name='ss_sm_heavy')) | ||
|
||
self.sim.modules['SymptomManager'].register_symptom( | ||
Symptom(name='ss_sh_moderate')) | ||
|
||
self.sim.modules['SymptomManager'].register_symptom( | ||
Symptom(name='ss_sh_heavy')) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is it right that these are symptoms - wouldn't the symptom be "itchiness" or something like that, and these be a property of the individual? (Not suggesting we change this at this stage)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We did have that originally, but each set of symptoms was linked with one schisto species and an intensity level, so it was cleaner to just assign a generic "symptom" corresponding to each set. Each symptom has a fixed probability of healthcare seeking, diagnosis and the same treatment and treatment success rates so there was no benefit really to having all the separate symptoms (bladder pathology, dysuria, ascites... etc).
This updated version of the schistosomiasis code has the following features: