Skip to content

Conversation

@kathsherratt
Copy link
Contributor

We previously considered including the absolute level of incidence, as well as the trend.
This was dropped and replaced with a term for the interaction of location x time.
55f0c4a#diff-826594aa74b90636f116b9ab5f64199cbbf5597a35237b758226ea6312fb2323L31

We're now replacing that interaction, with a random effect on dominant variant phase (#138).

Given this, do we want to bring back the incidence smooth term?

  • Pros: it complements the idea that we're doing as much as we can to account for the variation across forecast target-specific properties.
  • Cons: I don't think it hugely matters either way and possibly better to leave as-is given it's very late in the day for this work

Not keen on changing things around again unnecessarily, so thought I'd raise as a question in draft.

@kathsherratt kathsherratt requested a review from sbfnk January 5, 2026 16:58
@kathsherratt kathsherratt marked this pull request as ready for review January 12, 2026 11:36
@kathsherratt kathsherratt merged commit 0f44b07 into main Jan 12, 2026
1 check failed
@kathsherratt kathsherratt deleted the model-incidence branch January 12, 2026 11:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants