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

Systemic API to handle NA #30

Open
cicdguy opened this issue Aug 5, 2021 · 0 comments
Open

Systemic API to handle NA #30

cicdguy opened this issue Aug 5, 2021 · 0 comments

Comments

@cicdguy
Copy link
Contributor

cicdguy commented Aug 5, 2021

I notice that we don't have any functionality to systematically handle NA in data.

For example:

  plot_r <- reactive({
    chunks_reset()

    ANL <- merged_data()$data() # nolint
    # at this point the developer should be able to make decision of what to do with rows with NA
    # for example, to remove rows with NA
    # and then there should be some UI module that will output a warning message
    # letting user know that some rows were removed
    validate_has_data(ANL, 10)

Each column could be treated separately as well, with their own rules of how to handle NA.

If we add this feature, all code below this chunk will not have to ever worry about NA edge cases that will cause it to fail.

Provenance:

Creator: junlue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant