-
Notifications
You must be signed in to change notification settings - Fork 66
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 python==3.12
#615
Add python==3.12
#615
Conversation
Ok, too early |
maybe we can start testing this without |
|
@@ -19,7 +19,7 @@ jobs: | |||
runs-on: ubuntu-latest | |||
strategy: | |||
matrix: | |||
python-version: ["3.8", "3.9", "3.10", "3.11"] | |||
python-version: ["3.8", "3.9", "3.10", "3.11", "3.12"] |
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.
Just a heads up on Python 3.8 EOL https://devguide.python.org/versions/ (in case we want to reduce the CI time already)
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 we can keep it until it's deprecated, it gets executed in parallel so we're not going to gain CI time.
238af3c
to
7b9ca4c
Compare
7b9ca4c
to
8f1cc76
Compare
ee6cc0e
to
ee71536
Compare
ee71536
to
a8389c6
Compare
Description
This PR adds compatibility for
python==3.12
.Caveats
vllm
is not yet compatible withpython==3.12
becauseray
is not yet compatible with3.12
vllm
needsoutlines == 0.0.34
, while our integration requires>= 0.0.40
. I'll create a PR tovllm
to relax theoutlines
dependency.