Closed gvegayon closed 1 month ago
Attention: Patch coverage is 52.94118%
with 8 lines
in your changes are missing coverage. Please review.
Project coverage is 93.04%. Comparing base (
b603eb5
) to head (8ead934
).
Files | Patch % | Lines |
---|---|---|
...odel/src/pyrenew/model/rtinfectionsrenewalmodel.py | 37.50% | 5 Missing :warning: |
model/src/pyrenew/model/admissionsmodel.py | 57.14% | 3 Missing :warning: |
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
Hold my horses, I realized this should be merged into #131 instead! Thank goodness we have git cherry-pick
. I'm closing this.
This addresses and should close #142. The PR does the following:
pyrenew.process
to return tuples with arrays of shape(duration,)
.n_timepoints
optional for models underpyrenew.model
. Only required when observed data is not passed, the length of the time series is inferred from the observed data.n_timepoints
argument when simulating data.assert
ions to the process testing to make sure the length of the output matchesduration
.Notice this PR will be merged in to #123.
cc @damonbayer and @dylanhmorris