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

Fix check release by allowing Python 3.11 #210

Closed
wants to merge 2 commits into from

Conversation

krassowski
Copy link
Member

@krassowski krassowski commented Jun 4, 2023

Python 3.10 was pinned in #74 because ray 2.2 did not support Python 3.11 (see #73).

Python 3.11 is supported since ray 2.3. Ray 2.4 is already required since #127, so it should be possible to move on - let's see what CI has to say.

python-version was incorrectly added in #197 and generates a warning on CI since:

Unexpected input(s) 'python-version', valid inputs are ['token', 'version_spec', 'steps_to_skip']

image

@krassowski krassowski added the bug Something isn't working label Jun 4, 2023
@3coins
Copy link
Collaborator

3coins commented Jun 4, 2023

@krassowski
Thanks for working on this and fixing the CI. As per Ray's documentation, 3.11 is still experimental on Linux, and not supported on Mac OS and Windows.
https://docs.ray.io/en/latest/ray-overview/installation.html

I had reached out to the Ray team via their Slack channel and they did confirm this, and don't have a firm date to complete support for 3.11 yet. Here is their roadmap for 3.11 support.
https://github.com/ray-project/ray/milestone/104

I am not certain at this point, if we can update our documentation to call out that Jupyter AI works with Python 3.11.

@krassowski
Copy link
Member Author

Indeed the support is Linux only, thanks for a quick reply.

@krassowski krassowski closed this Jun 4, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants