Repository navigation
Fix #1118: Update stumpy to >=1.11.1 to fix Python 3.11+ uv resolutio… #1120
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Changes from 9 commits
d18254e
6318312
b705dc8
cbd8bb0
64e2423
421fe2b
2c632db
97a5d56
1d58e96
d22c3a7
7d48321
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,6 @@ | ||
| [build-system] | ||
| # setuptools is capped below 70 because PyScaffold <4 (used by this project) | ||
| # has known compatibility issues with setuptools>=70. Once the project | ||
| # migrates to PyScaffold 4+, this upper bound can be removed. | ||
| requires = ["setuptools<70", "wheel", "pyscaffold>=3.3a0,<4"] | ||
| build-backend = "setuptools.build_meta" | ||
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -8,6 +8,7 @@ | |
| from unittest import TestCase | ||
|
|
||
| import numpy as np | ||
| import pandas as pd | ||
| from pandas import DataFrame, Series | ||
|
|
||
| from tsfresh import extract_features | ||
|
|
@@ -54,4 +55,4 @@ def test_multilabel_target_on_request(self): | |
| ) | ||
|
|
||
| assert len(y.unique()) > 2 | ||
| assert y.dtype == object | ||
| assert y.dtype == object or isinstance(y.dtype, pd.StringDtype) | ||
|
Collaborator
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. you probably want to import pd here :)
Contributor
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. @nils-braun Thanks for the approval! I will gladly take care of these last few items. I’ll fix the missing pd import, clear up the linting, and dig into those two 3.11/3.12 test failures today. I'll push an update shortly! |
||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
pyproject.tomlpinssetuptools<70as a build requirement. A hard upper bound can unnecessarily block builds on environments that have already moved to newer setuptools. If this cap is required due to a known incompatibility (e.g., PyScaffold <4), please add a brief rationale/comment or link; otherwise consider relaxing/removing the upper bound.There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think mentioning that this is a requirement from pyscaffold is indeed a good idea