[ENH] Make fit in ARIMA iterative_forecast optional #3078 #3110
+7
−2
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.
Reference Issues/PRs
Fixes #3078
####Updated Files
aeon/forecasting/stats/_arima.pyWhat does this implement/fix? Explain your changes.
Quoting @JoaquinAmatRodrigo
For resolving this issue, I have not added any module or method but just edited aeon/forecasting/stats/_arima.py . I have made following changes in iterative_forecast method of ARIMA class (line 210 of _arima.py).
I added extra parameter 'fit' into the method
iterative__forecast().iterative_forecastwill raise error if model isn't fitted before, and fitting can be set as true or false.Does your contribution introduce a new dependency? If yes, which one?
No, my contribution does not introduce any new dependency
Any other comments?
I ran pytest on whole codebase and everything seems fine.
PR checklist
For all contributions