-
Notifications
You must be signed in to change notification settings - Fork 5
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
Wasting Module #543
base: master
Are you sure you want to change the base?
Wasting Module #543
Conversation
…running without errors. added wasting analysis file to get more understanding of the current outputs before applying some major changes. added wasting resource file
# Conflicts: # src/tlo/methods/hsi_generic_first_appts.py
called the function nutritional_oedema_present within the WastingNaturalRecoveryEvent to reset the values for the oedema status based on WHZ>=-2
Removed progression to severe following treatment failure for MAM, at initialise population. These individuals will be called at the next polling event (in 3 months) to determine their outcome - recovery, remain, or death.
…rameter values in wasting resource file
…lett/wasting_module # Conflicts: # src/tlo/methods/wasting.py
# Conflicts: # resources/healthsystem/priority_policies/ResourceFile_PriorityRanking_ALLPOLICIES.xlsx
…lett/wasting_module # Conflicts: # resources/ResourceFile_Wasting.xlsx # src/scripts/wasting_analyses/analysis_wasting.py # src/tlo/methods/hsi_generic_first_appts.py # src/tlo/methods/wasting.py # tests/test_wasting.py
…lett/wasting_module
A few more questions related to the growth monitoring:
|
@timcolbourn, Or, do we maybe assume that the parameters such as |
@tbhallett, @marghe-molaro, @sakshimohan, I'm adding a new HSI event, |
@tbhallett, @timcolbourn, What footprint should we assign? I would say smt like: What do you think? |
Hi @EvaJanouskova, I include below the values that should be assigned to HSI_Wasting_GrowthMonitoring in all policies except EHP_III and LCOA_EHP, on which @sakshimohan is better able to advise. Please let me know if anything is unclear! Values of [Priority, FT_if_5orUnder, FT_if_pregnancy, FT_if_tbdiagnosed, FT_if_Hivdiagnosed] should be: In default and test: In Naive In Test mode 1 In Clinically Vulnerable Vertical Programmes RMNCH CVD |
Hi @EvaJanouskova. For the remaining policies, I would assign the following - LCOA_EHP |
… well group from seeking care sooner
I think this means that growth monitoring clinics are held every month - probably in level 1a health centres, but also level 0 community clinics by HSAs. We'd need to estimate what proportion of all children aged 0-59 months go each month - I'd imagine that mothers don't necessarily bring their children each months, maybe only every few months or if they are worried about the child not growing enough. It'd be worth doing a quick search to see if there's any data on this e.g. from published papers, or from DHIS2 (could work out with knowledge of population denominators of number of children 0-59 months in the catchment population of the clinic) |
I think we need to link to it actual equipment availability. The children attending can also be based on care seeking, noting also my reply to the previous comment above about not all children attending the clinic every month.
I think there are two different ones? a shorr board for equivalent for length for infants who can't yet stand or walk, and a height pole for older children who can stand?
I think it would be low priority (priority 2?). Though I think if clinics are run every month health workers would be available. However if the health facility has emergencies or high priority things in the queue they should run first. So set to priority 2 I think |
please see my earlier reply above - I think this |
Yes that looks good I think |
@tbhallett, @matt-graham, @tamuri. I think that within one event, we are only able to log one use of each equipment item, aren’t we? However, with PopulationScopeEventMixin, I would need to log multiple uses of the equipment. |
@marghe-molaro,Should this one have been 3, -1, -1, -1, -1 (rather than 3, -1, -1, -1, +1), or? |
I’ve just found out from Margherita that the priority setting within the modules is somewhat outdated, so we don’t need to worry about it as those are overridden by the Naive priorities from RF as the status quo anyway. |
@tamuri, @matt-graham, When the tests ran after the wast: priority=2 for HSI_Wasting_GrowthMonitoring commit, there were two failures: test_healthsystem (I know the reason for that, so it’s fine) and test_continuous_and_interrupted_simulations_equal from test_simulation. However, when I reran test_simulation, it passed. It also passes when run on my computer. Is there any randomness within the test that causes it to sometimes fail and sometimes pass? |
…ng added" This reverts commit c2028d6.
…; lowest_priority_considered mv to the bottom
…pulation hsi_event.target
@tamuri , @matt-graham, Is it okay to make these changes: hs: enforce_priority_policy() fnc updated to work for individual & po… to allow an |
Hi @EvaJanouskova -- I just spotted this. @marghe-molaro will want to comment on this. We had planned to deprecate the concept of population-wide HSI. #1336 Is this really needed? |
@tbhallett, I think I could use a |
Yes much better!! Thanks |
@tbhallett we could ensure an error is raised in the HealthSystem module whenever this tries to schedule/run a pop-level HSI event? This way disease module developers would be automatically warned not to include them. |
Fixes #542