[DOC] ExponentialSmoothing default method change from L-BFGS-B to SLSQP #6186
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
While exploring #6175, realized that the default parameter for
method
in Exponential Smoothing should beSLSQP
. This also affects statsmodels documentation and there is already an open PR in their repo to fix this.What does this implement/fix? Explain your changes.
Changes
method='L-BFGS-B'
tomethod='SLSQP'
.Does your contribution introduce a new dependency? If yes, which one?
No
What should a reviewer concentrate their feedback on?
See statsmodels default
Did you add any tests for the change?
N/A
Any other comments?
N/A
PR checklist
For all contributions
How to: add yourself to the all-contributors file in the
sktime
root directory (not theCONTRIBUTORS.md
). Common badges:code
- fixing a bug, or adding code logic.doc
- writing or improving documentation or docstrings.bug
- reporting or diagnosing a bug (get this pluscode
if you also fixed the bug in the PR).maintenance
- CI, test framework, release.See here for full badge reference
See here for further details on the algorithm maintainer role.