-
Notifications
You must be signed in to change notification settings - Fork 0
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
Change variable name to condom_change_year_i #289
Conversation
@@ -17003,7 +17002,7 @@ if 15 <= age and (death = . or caldate&j = death ) then do; | |||
|
|||
s_all_prep_criteria + all_prep_criteria ; s_all_prep_criteria_hivneg + all_prep_criteria_hivneg; s_prep_elig_hivneg + prep_elig_hivneg ; | |||
s_prep_elig_hivneg_onprep + prep_elig_hivneg_onprep ; s_prep_elig_onprep + prep_elig_onprep ;s_prep_elig_onprep_inj + prep_elig_onprep_inj; | |||
s_started_prep_inj_hiv + started_prep_inj_hiv; s_prep_elig_onprep_vr + prep_elig_onprep_vr; s_started_prep_vr_hiv + started_prep_vr_hiv; | |||
s_started_prep_oral_hiv + started_prep_oral_hiv; s_started_prep_inj_hiv + started_prep_inj_hiv; s_prep_elig_onprep_vr + prep_elig_onprep_vr; s_started_prep_vr_hiv + started_prep_vr_hiv; |
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.
just noticed that here you added:
- s_started_prep_oral_hiv
- s_prep_elig_onprep_vr
Is it intentional that you added the latter?
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.
I think that s_prep_elig_onprep_vr was already there but the order is a bit confusing and I can also see that s_prep_elig_onprep_oral doesn't exist. I will add that in and reorder the outputs so it's a bit clearer. Now inputted is:
s_prep_elig_onprep s_prep_elig_onprep_oral s_prep_elig_onprep_inj s_prep_elig_onprep_vr
s_started_prep_any_hiv s_started_prep_oral_hiv s_started_prep_inj_hiv s_started_prep_vr_hiv
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.
I just left a comment, otherwise fine with me
Jenny I noticed in the self-testing code we have "eponart=1" when this should be "epart=1" Could you please add to this PR ? |
Hi Andrew, thanks - I've made that change. |
Variable previously called condom_incr_year_i but condom use does not necessarily increase. Changing to be consistent with MIHPSA Zim code