search

epiforecasts / forecast.vocs

Contains models and tools to produce short-term forecasts for both case and sequence notifications assuming circulation of either one or two variants. Tools are also provided to allow the evaluation of the use of sequence data for short-term forecasts in both real-world settings and in user generated scenarios.
http://epiforecasts.io/forecast.vocs
Other
8 stars 3 forks source link

Fix author list #116

Closed Bisaloo closed 2 years ago

Bisaloo commented 2 years ago

see https://github.com/r-universe-org/help/issues/152#issuecomment-1125680609

jeroen commented 2 years ago

This is still incorrect, the c() needs to wrap both persons.

Bisaloo commented 2 years ago

This is still incorrect, the c() needs to wrap both persons.

It does. The diff is just confusing because I'm fixing two mistakes in this PR (missing c() and missing closing parenthesis).

jeroen commented 2 years ago

Oh right haha

seabbs commented 2 years ago

Thanks for this @Bisaloo. Sadly catching up on my notifications in a very illogical order and so fixed this in https://github.com/epiforecasts/forecast.vocs/commit/a190db3a8f239ec716ef01466ce8178144d18269