epiforecasts / EpiNow2

Estimate Realtime Case Counts and Time-varying Epidemiological Parameters
https://epiforecasts.io/EpiNow2/dev/
Other
112 stars 31 forks source link

Add vignette on modelling temporally aggregated data (i.e., `obs_opts(na = "accumulate")` ) #772

Open jamesmbaazam opened 1 week ago

jamesmbaazam commented 1 week ago

Thanks a lot all - I've tested it out and it's working well now! I've included the updated figure below on the same data for reference, in case helpful!

@jamesmbaazam regarding extra documentation, I think that at some point it could be useful to add a short, dedicated section on applying to weekly data. The current documentation mentions the rw estimator for weekly, but I think having a section with an example as you do for daily could be beneficial to others as we work a lot with weekly aggregated data. The other thing would probably be then to be able to visualize on weekly scale too, but as it's possible for end users to extract the estimates and create visuals I don't think it's really necessary.

Thanks again for your help and for a great package!

weekly_est_epinow

Originally posted by @martinamcm in https://github.com/epiforecasts/EpiNow2/issues/767#issuecomment-2351691083

jamesmbaazam commented 1 week ago

By "surface", I mean we need a vignette walkthrough. My argument for this need is here https://github.com/epiforecasts/EpiNow2/pull/774#discussion_r1760886569.

seabbs commented 1 week ago

Agree. Perhaps edit the title and add a plan/outline?

jamesmbaazam commented 1 week ago

Outline of vignette