EpiNow2

EpiNow2 copied to clipboard

Group deprecated functions together in `pkgdown` Reference Page

1

It would be nice to add a section in the `pkgdown` Reference, similar to [this section on superseded functions in dplyr](https://dplyr.tidyverse.org/reference/index.html#superseded) and [purrr](https://purrr.tidyverse.org/reference/index.html#superseded). That will probably help users identify which...

jamesmbaazam

Possibility to generalise probability distribution interface?

2

I've been taking a look through #504 and I really like the way distributions are specified and enable nesting for parameter uncertainty. I think this could be a nice interface...

joshwlambert

Bundle repeated checks

1

We should probably bundle these checks into one as it's getting repeated in several places. _Originally posted by @jamesmbaazam in https://github.com/epiforecasts/EpiNow2/pull/557#discussion_r1494641426_

sbfnk

Distinguish NA (missing) from NA (accumulated)

13

> Enabling this would, I think, require some sort of that marks dates explicitly as missing vs. NA. I think this would be my preferred option as it would be...

sbfnk

I guess I am saying that cases_time would become optionally an array with a sum. Another option might be to make this a matrix multiplication problem _Originally posted by @seabbs...

sbfnk

Move to `{cli}` for messaging

1

One reason to do so: https://github.com/epiforecasts/EpiNow2/pull/534#issuecomment-1943972293

sbfnk

Change all priors to new dist interface, not just delays

3

For a future PR this should probably be updated to use `normal()`. _Originally posted by @seabbs in https://github.com/epiforecasts/EpiNow2/pull/504#discussion_r1408162989_ This will require moving the conversion to discretised distribution to the processing...

sbfnk

repo size

9

The repo has grown fairly large (~1 GB), but the files currently in the repo are only 11 MB in size. It might be nice, particularly towards those on low...

sbfnk

streamline S3 classes

6

Currently the three models included in the package return S3 class with similar but slightly different contents. Whilst to some degree that's expected because of their different use cases it...

sbfnk

estimate_delays -> estimate_dist

4

It would make sense to generalise and simplify the current functionality of estimate_delays into a new function called estimate_dist (to highlight it isn't just distributions we are estimating). Based on...

seabbs