-
Notifications
You must be signed in to change notification settings - Fork 357
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
test if we can update python version in tests #364
base: main
Are you sure you want to change the base?
Conversation
@@ -8,15 +8,15 @@ jobs: | |||
runs-on: ubuntu-latest | |||
strategy: | |||
matrix: | |||
python-version: ['3.8', '3.10', '3.12'] | |||
python-version: ['3.9', '3.11', '3.13'] | |||
# currently there is conflict between tf, oneflow and paddle in protobuf versions. | |||
# cupy is not tested because it demands gpu | |||
# oneflow testing is dropped, see details at https://github.com/Oneflow-Inc/oneflow/issues/10340 | |||
# paddle was switched off because of divergence with numpy in py3.10, paddle==2.6.1 | |||
# The last pytensor release that supports python 3.8 doesn't include einsum, so we skip that combination. |
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.
Update to mention the opposite, that last pytensor does not yet support python3.13?
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.
will do if end with 3.13 working. Figuring out blockers right now.
Ugh python3.9 test also fails with PyTensor backend. We only pinned numpy < 2.0 later :( If you skip that it should work on 3.10, 3.11, 3.12. I don't think we support 3.13 yet |
No description provided.