Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Some things:
I am least confident about the MSTL implementation if you can take a took.
Am I right in thinking that
prediction_intervals=ConformalIntervals(h=13, n_windows=2)
means that you need to doforecast(h=13)
, otherwise the shapes of the intervals will not match and you can't form the output DataFrame. I feel like we should either catch this and raise an explicit error rather than a numpy shape error, or we can handle implicitly and override?The type hint for level:
Optional[Tuple[int]]
causes a mypy issue because of the following behavior:so the line:
casts tuple to list, which mypy doesn't like. So I think all the levels hints should be
Optional[List[int]]