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

migrate pydantic from v1 to v2 #2531

Merged
merged 3 commits into from
Jan 22, 2024

Conversation

joennlae
Copy link
Contributor

Following this guide here: https://docs.pydantic.dev/dev/migration/

Previous attempts: #2468 #2322

closes #1396
closes #2313
closes #2322

It also includes a fix to error code propagation for the OpenAI-compatible server.

@joennlae
Copy link
Contributor Author

I was just fixing this also in parallel :-)

@simon-mo simon-mo merged commit 71d63ed into vllm-project:main Jan 22, 2024
14 of 16 checks passed
@simon-mo
Copy link
Collaborator

Thank you for the contribution!

@Lvjinhong
Copy link

Why does it still show ERROR: pip's dependency resolver does not currently take into account all the packages that are installed. This behavior is the source of the following dependency conflicts.
vllm 0.2.7 requires pydantic==1.10.13, but you have pydantic 2.5.3 which is incompatible.

@simon-mo
Copy link
Collaborator

This commit will be part of up coming vLLM 0.2.8, which will be releaesed in <1 week

@yippp
Copy link

yippp commented Jan 28, 2024

Seems pydantic v2 leads to a problem meet in #2619

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.

Outdated Pydantic Version bump pydantic to v2
4 participants