Skip to content

[MNT] Probe statsmodels 0.15 with sktime-next - #137

Draft
amotl wants to merge 2 commits into
mainfrom
statsmodels-0.15
Draft

[MNT] Probe statsmodels 0.15 with sktime-next#137
amotl wants to merge 2 commits into
mainfrom
statsmodels-0.15

Conversation

@amotl

@amotl amotl commented Sep 5, 2026

Copy link
Copy Markdown
Member

About

Probe statsmodels 0.15 with a patch to sktime by @aryamanDutta. Thank you!
Probe statsmodels 0.15 with sktime@main.

Details

Another automatic update failed CI.

TypeError: ETSResults.simulate() got an unexpected keyword argument 'random_state'
TypeError: ccf() got an unexpected keyword argument 'unbiased'

References

@amotl

amotl commented Sep 5, 2026

Copy link
Copy Markdown
Member Author

We've reported @aryamanDutta's patch works well, according to the outcome on this project's CI. Thank you again.

Comment thread pyproject.toml Outdated

@fkiraly fkiraly left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ok, I understand this was for testing, right?

So we just need to wait for the new sktime release, and the problem will fix itself, correct?

@amotl

amotl commented Sep 6, 2026

Copy link
Copy Markdown
Member Author

Ok, I understand this was for testing, right?

Exactly. It was about validating the upstream patch by @aryamanDutta.

So we just need to wait for the new sktime release, and the problem will fix itself, correct?

Right. First, this patch needs to converge, and afterwards we need a new release.

@amotl

amotl commented Sep 6, 2026

Copy link
Copy Markdown
Member Author

Let me close this again. GH-110 will be sufficient to resolve this.

@amotl amotl closed this Sep 6, 2026
@amotl
amotl deleted the statsmodels-0.15 branch September 6, 2026 12:44
@amotl
amotl restored the statsmodels-0.15 branch September 6, 2026 16:44
@amotl amotl reopened this Sep 6, 2026
@amotl

amotl commented Sep 6, 2026

Copy link
Copy Markdown
Member Author

@fkiraly says the flaw was already fixed, so I am now validating against sktime@main per 25541ff.

@amotl amotl changed the title [MNT] Permit installation of statsmodels 0.15 [MNT] Probe statsmodels 0.15 with sktime-next Sep 6, 2026
@fkiraly

fkiraly commented Sep 6, 2026

Copy link
Copy Markdown
Contributor

hm, looks like it is failing still. Strange.
I am sure there were ets related failures, but I thought they were fixed.

Could you open an issue with the exact call?
And the patch that fixes it?

@amotl

amotl commented Sep 6, 2026

Copy link
Copy Markdown
Member Author

I am sure there were ets related failures, but I thought they were fixed.

I think it is about a different topic here.

Could you open an issue with the exact call? And the patch that fixes it?

I believe it is all done by @siddharth7113 and @aryamanDutta already. When using the patch like before 25541ff, the integration tests succeed. 💯

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants