Currently, the rationale for when and why to include/exclude ER diagnoses is not very clear in the function documentation. Provide additional explanation and reference for lower validity of ER diagnoses (Surain may know reference). Also see issue #43.
To be discussed: We may want to make a small change to the input arguments for consistency with other functions using diagnosis codes (Charlson Comorbidity, frailty, disability) - which take 2 separate inputs ipdiag and erdiag. erdiag can be set to NULL by user if they don't want to include ER diagnoses. Probably want to output a message explaining whether ER diagnoses are included or not, similarly to disability function (see #25).
Note: ED diagnosis documentation also needs to be improved in charlson_comorbidity (issue #43) and disability functions.
New Feature Request
Currently, the rationale for when and why to include/exclude ER diagnoses is not very clear in the function documentation. Provide additional explanation and reference for lower validity of ER diagnoses (Surain may know reference). Also see issue #43.
To be discussed: We may want to make a small change to the input arguments for consistency with other functions using diagnosis codes (Charlson Comorbidity, frailty, disability) - which take 2 separate inputs
ipdiag
anderdiag
.erdiag
can be set toNULL
by user if they don't want to include ER diagnoses. Probably want to output a message explaining whether ER diagnoses are included or not, similarly to disability function (see #25).Note: ED diagnosis documentation also needs to be improved in
charlson_comorbidity
(issue #43) anddisability
functions.