Skip to content
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

[summarize_sir_results] improve calculation of n_base and ref_population_pyar #63

Open
marianschmidt opened this issue Dec 9, 2020 · 0 comments
Labels
enhancement New feature or request

Comments

@marianschmidt
Copy link
Owner

since the results created by sir_byfutime() are filtered to exclude non informative strata (i.e. no case occured in the reference population and no FU time contributed in the dataset), the summary of n_base and ref_population_pyar gives ambigous results, because for some ICD codes, single addition n will count to n_base and single strata will contribute additional ref_population_pyar. This occurs for strata where cases are observed that don't occur in refrates_df.

e.g.
Zfkd, 35-39, DE6 Hamburg, Male 2012, total_var, to 6 months, C26

This issue could be solved by

  • first expanding the sir_results_df before summarizing it to get constant n_base and ref_population_pyar

or

  • leaving the warning as currently implemented in summarize_sir_results()
@marianschmidt marianschmidt added the enhancement New feature or request label Dec 9, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

1 participant