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

enable py312 client #985

Merged
merged 10 commits into from
Jun 26, 2024
Merged

enable py312 client #985

merged 10 commits into from
Jun 26, 2024

Conversation

rcano-baseten
Copy link
Contributor

@rcano-baseten rcano-baseten commented Jun 18, 2024

🚀 What

https://linear.app/baseten/issue/BT-11088/truss-support-python-312

💻 How

Updated the allowed versions in pyproject.toml and removed deprecated packages

🔬 Testing

python3.12 -m venv env312
source ~/baseten/env312/bin/activate
cd truss
pip install .
python
>>> import truss
>>>
  • import truss for 3.8 -3.12
  • push model for 3.8 - 3.12

poetry.lock Outdated Show resolved Hide resolved
truss/util/path.py Outdated Show resolved Hide resolved
@rcano-baseten
Copy link
Contributor Author

rcano-baseten commented Jun 24, 2024

TODO: look through #955 and basically just do a full text search throughout the code base where the current upper limit 3.11 / 311 appears and see where you need to update anything

Edit: Doing this as a follow up https://linear.app/baseten/issue/BT-11307/truss-support-python-312-in-context-builder . Going to merge this as a stop gap to enable py312 client to use truss

@rcano-baseten
Copy link
Contributor Author

awaiting 👍 from @squidarth

Copy link
Collaborator

@squidarth squidarth left a comment

Choose a reason for hiding this comment

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

one q about the distutils thing, lgtm otherwise

@rcano-baseten rcano-baseten merged commit 500cf02 into main Jun 26, 2024
3 checks passed
@rcano-baseten rcano-baseten deleted the rc-2024.06.18-py312_client branch June 26, 2024 14:29
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.

3 participants