Skip to content

bug: total_inflows mislabelled for old-and-young model #59

@papsti

Description

@papsti

simulating with default initial state and parameters, it looks like what should be the inflow into $E_y$ (force of infection on age group $y$) gets labelled as the inflow into $R_y$:

Image

(indeed, this is verified by manual calculation). mismatched label is also true for inflow reported into $E_y$; it's equal to the value that should be reported for $I_y$.

this may be because the state vector lists R states right after S states and there's an order dependency we were previously unaware of (despite the fact that the state vector is named!):

Image

i imagine this cascades down into the simulated values reported under state. this also affects the main package vignette, which uses the old-and-young model for the first demo. it should not affect other models, which were checked.

either:

  • fix the implementation (may already be done with kevin's re-implementation)
  • remove this model from the package (it's never really used) and update the vignette to use a new model instead

Metadata

Metadata

Assignees

No one assigned

    Labels

    bugSomething isn't working

    Type

    No type
    No fields configured for issues without a type.

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions