epiforecasts / EpiSoon

Forecasting the effective reproduction number over short timescales
https://epiforecasts.io/EpiSoon/
Other
7 stars 3 forks source link

Generalise to other packages #30

Closed seabbs closed 4 years ago

seabbs commented 4 years ago

Comment

fable has several dependencies for different models that are required in order for it to run them. The error checking in compare_timeseries screens out these messages which can make it hard to debug. The model implementation process should include an initial check that the fable model works at the lowest level (i.e. with fable_model as per the examples). Similarly fable natively uses the future package for parallelisation but also requires the future.apply package which is not installed by default. If a future is detected (i.e. for use with compare_timeseries) then future.apply must be installed in order for a fable model to be run, even though it is not running in parallel in this instance. Obviously these features are not ideal and quite annoying on initial set up if not aware. Some flagging has been added to the fable_model docs to indicate the problem.