Skip to content
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

Add py3.12 #95

Merged
merged 1 commit into from
Apr 1, 2024
Merged

Add py3.12 #95

merged 1 commit into from
Apr 1, 2024

Conversation

fealho
Copy link
Member

@fealho fealho commented Mar 26, 2024

CU-86ayu71by, Resolve #85.

@sdv-team
Copy link
Contributor

@codecov-commenter
Copy link

codecov-commenter commented Mar 26, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 79.12%. Comparing base (4ad94da) to head (45c4cee).

❗ Your organization needs to install the Codecov GitHub app to enable full functionality.

Additional details and impacted files
@@            Coverage Diff             @@
##             main      #95      +/-   ##
==========================================
- Coverage   79.39%   79.12%   -0.28%     
==========================================
  Files           7        7              
  Lines         728      728              
==========================================
- Hits          578      576       -2     
- Misses        150      152       +2     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@fealho fealho force-pushed the issue-85-py12 branch 2 times, most recently from 9fe73a1 to 7f52361 Compare March 26, 2024 19:52
pyproject.toml Outdated Show resolved Hide resolved
@@ -54,7 +55,7 @@ dev = [
'setuptools<49.2',
'bump-my-version>=0.18.3,<1',
'pip>=9.0.1',
'watchdog>=0.8.3,<0.11',
'watchdog>=1.0.1,<5',
Copy link
Member Author

Choose a reason for hiding this comment

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

Python 3.12 has removed support for the imp library, which watchdog uses up to version 0.10.7.

@fealho fealho force-pushed the issue-85-py12 branch 3 times, most recently from 68dc7aa to f729e51 Compare March 26, 2024 22:25
"torch>=2.0.0;python_version>='3.11'",
'tqdm>=4.15,<5',
"torch>=2.0.0;python_version>='3.11' and python_version<'3.12'",
"torch>=2.2.0;python_version>='3.12'",
Copy link
Member Author

Choose a reason for hiding this comment

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

Because we updated numpy's version we get this: ERROR: Could not find a version that satisfies the requirement torch==2.1.0 (from versions: 2.2.0, 2.2.1), so we need to increase torch's version as well to pass the minimum tests.

readme = 'README.md'
dependencies = [
"numpy>=1.20.0,<2;python_version<'3.10'",
"numpy>=1.23.3,<2;python_version>='3.10'",
"numpy>=1.23.3,<2;python_version>='3.10' and python_version<'3.12'",
"numpy>=1.26.0,<2;python_version>='3.12'",
Copy link
Member Author

Choose a reason for hiding this comment

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

Versions before numpy>=1.26.0 run into ModuleNotFoundError: No module named 'distutils'.

Copy link
Contributor

Choose a reason for hiding this comment

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

Yeah it seems like numpy doesn't support python 3.12 on versions before 1.26 on PyPi either

'tqdm>=4.15,<5',
"torch>=2.0.0;python_version>='3.11' and python_version<'3.12'",
"torch>=2.2.0;python_version>='3.12'",
'tqdm>=4.29,<5',
Copy link
Member Author

Choose a reason for hiding this comment

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

Versions older than 4.29 run into this issue for the minimum tests.

@fealho fealho marked this pull request as ready for review March 27, 2024 15:19
@fealho fealho requested a review from a team as a code owner March 27, 2024 15:19
@fealho fealho requested review from amontanez24 and pvk-developer and removed request for a team March 27, 2024 15:19
readme = 'README.md'
dependencies = [
"numpy>=1.20.0,<2;python_version<'3.10'",
"numpy>=1.23.3,<2;python_version>='3.10'",
"numpy>=1.23.3,<2;python_version>='3.10' and python_version<'3.12'",
"numpy>=1.26.0,<2;python_version>='3.12'",
Copy link
Contributor

Choose a reason for hiding this comment

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

Yeah it seems like numpy doesn't support python 3.12 on versions before 1.26 on PyPi either

@fealho fealho requested review from frances-h and removed request for pvk-developer March 29, 2024 17:22
@fealho fealho merged commit 51cf1ee into main Apr 1, 2024
56 checks passed
@fealho fealho deleted the issue-85-py12 branch April 1, 2024 14:39
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.

Support Python 3.12
5 participants