Skip to content

Commuter Testing should not be coupled with dynamic NPIs #1116

@HenrZu

Description

@HenrZu

Motivation / Current Behaviour

in the advance function of the secirvvs model, we always set

this->get_model().parameters.get_start_commuter_detection() = (ScalarType)t_start;
this->get_model().parameters.get_end_commuter_detection()   = (ScalarType)t_end;

Testing should not necessarily be coupled to the dynamic NPIs in this way. Also, we can't change the non detection value during the simulation.

Enhancement description

make the code more modular and split up features instead of combining them.

Additional context

No response

Checklist

  • Attached labels, especially loc:: or model:: labels.
  • Linked to project

Depends on #1523

Metadata

Metadata

Assignees

Labels

class::improvementCleanup that doesn't affect functionalityloc::backendThis issue concerns the C++ backend implementation.model::odeThis issue concerns any kind of ODE-based model.prio::lowThe priority of this task is low. It is an interesting item but with low priority.

Type

No type

Projects

Status

Product Backlog

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions