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

NA Handling: Mechanistic model and more generally #10

Open
3 tasks
SOLV-Code opened this issue Dec 6, 2020 · 0 comments
Open
3 tasks

NA Handling: Mechanistic model and more generally #10

SOLV-Code opened this issue Dec 6, 2020 · 0 comments

Comments

@SOLV-Code
Copy link
Collaborator

Issue

In the Weighted Avg version of the mechanistic model (currently the default), NA handling is implemented as follows:

  • drop incomplete rows (i.e. either the Abd or the Predictor has NA, or both)
  • for the remaining records, sum the Abd and the Predictor, then calc the rate and do the rest

However, in most other functions, and more generally in the app, missing values are currently not allowed.

To Do

  • check through the code and app to verify current set-up
  • develop overarching approach (to ensure consistency as we add new model forms)
  • document approach on the wiki
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant