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

Edge case breaking summary stats output #392

Closed
Tracked by #423
jrcpulliam opened this issue May 16, 2023 · 1 comment · Fixed by #436
Closed
Tracked by #423

Edge case breaking summary stats output #392

jrcpulliam opened this issue May 16, 2023 · 1 comment · Fixed by #436
Labels
bug Something isn't working

Comments

@jrcpulliam
Copy link

When both the upper and lower bound of the latest estimate are 0, log_cases is set to NA in summarise.R (lines 311-312; 0/0 > 1000 returns NA, instead of a logical).

This produces an error when generating summary statistics: "Error in if (log_cases) {: missing value where TRUE/FALSE needed"

@seabbs seabbs added the bug Something isn't working label May 16, 2023
@seabbs
Copy link
Contributor

seabbs commented May 16, 2023

Thanks @jrcpulliam for reporting.

That does sound like an edge case we haven't considered. Will get on fixing that!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
Status: Done
Development

Successfully merging a pull request may close this issue.

2 participants