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

[shortfin] Pin pydantic-core to 2.27.0 #595

Closed
wants to merge 1 commit into from

Conversation

marbre
Copy link
Collaborator

@marbre marbre commented Nov 22, 2024

Pins pydantic-core to 2.27.0, as building 2.27.1 fails to build for Python 3.13t.

Pins pydantic-core to 2.27.0, as building 2.27.1 fails to build for
Python 3.13t.
@marbre
Copy link
Collaborator Author

marbre commented Nov 22, 2024

Actually, pydantic-core 2.27.1 cannot be build with free-threaded Python as PyO3 was bumped to 0.22.6 to prevent accidental installs on free-threaded Python, see pydantic/pydantic-core#1550.

Therefore, we should re-evaluate the deps in the requirements file: pydantic-core is required by pydantic which is required by fastapi.

marbre added a commit to marbre/shark-ai that referenced this pull request Nov 22, 2024
This removes fastapi as a requirements as it pulls in pydantic which
depends on pydantic-core. The latter cannot be build for a free-threaded
Python and as part of the 0.27.1 release bumped PyO3 to 0.22.6 to
prevent accidental installs on free-threaded Python.

Supersedes and closes nod-ai#595.
@marbre marbre closed this in #596 Nov 22, 2024
marbre added a commit that referenced this pull request Nov 22, 2024
This removes fastapi as a requirements as it pulls in pydantic which
depends on pydantic-core. The latter cannot be build for a free-threaded
Python and as part of the 0.27.1 release bumped PyO3 to 0.22.6 to
prevent accidental installs on free-threaded Python.

Supersedes and closes #595.
@marbre marbre deleted the shortfin-3.13t-pydantic-core branch November 22, 2024 19:37
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.

1 participant