ciemss / pyciemss

Causal and probabilistic reasoning with continuous time dynamical systems
Other
12 stars 4 forks source link

adding error message to calibrate and test for no uncertainty in models - DO NOT MERGE #532

Closed sabinala closed 3 months ago

sabinala commented 3 months ago

This PR adds an error message to calibrate when a model does not include uncertainty in any parameter values.

Closes #527

sabinala commented 3 months ago

@SamWitty @JosephCottam did we decide this is no longer necessary? Feel free to close if so, but this PR is complete otherwise.

SamWitty commented 3 months ago

Yes, this will be subsumed by a follow-up PR that directly uses the existing model inventory. Let's just leave it here for now, we can close if/when that PR comes.

sabinala commented 3 months ago

@SamWitty I believe this can be closed, subsumed by #546