Skip to content

Conversation

@mrwyattii
Copy link
Contributor

@mrwyattii mrwyattii commented Feb 21, 2024

Pydantic v2 has been out for some time now. We have been relying on using the v1 API available in v2 until now. This is a refresh of #3902 to bring proper v2 support to DeepSpeed.

Corresponding DeepSpeed-MII PR here.

@loadams

@loadams
Copy link
Collaborator

loadams commented Feb 27, 2024

Do you want to do what we did on the previous pydantic update of updating the nv-mii test to use your branch rather than main so that it passes here?

@mrwyattii
Copy link
Contributor Author

CI with deepspeedai/DeepSpeed-MII#423 for nv-a6000 and nv-mii

@loadams
Copy link
Collaborator

loadams commented Jul 1, 2024

CI triggered using updates in MII

nv-a6000
nv-mii

Both tests now passing.

@loadams
Copy link
Collaborator

loadams commented Aug 20, 2024

Tested in #6382.

@loadams loadams merged commit 0a4457c into master Aug 22, 2024
weiji14 added a commit to regro-cf-autotick-bot/deepspeed-feedstock that referenced this pull request Aug 23, 2024
weiji14 added a commit to conda-forge/deepspeed-feedstock that referenced this pull request Aug 29, 2024
* updated v0.15.0

* MNT: Re-rendered with conda-build 24.7.1, conda-smithy 3.38.0, and conda-forge-pinning 2024.08.23.00.41.48

* Remove ninja as runtime dependency

Xref #1

* Remove note about pydantic<2.0.0 pin

Since deepspeedai/DeepSpeed#5167 is merged already.

---------

Co-authored-by: Wei Ji <[email protected]>
mauryaavinash95 pushed a commit to DataStates/DeepSpeed that referenced this pull request Mar 20, 2025
Pydantic v2 has been out for some time now. We have been relying on
using the v1 API available in v2 until now. This is a refresh of deepspeedai#3902
to bring proper v2 support to DeepSpeed.

Corresponding DeepSpeed-MII PR
[here](deepspeedai/DeepSpeed-MII#423).

@loadams

---------

Co-authored-by: Logan Adams <[email protected]>
Co-authored-by: Logan Adams <[email protected]>
Co-authored-by: Olatunji Ruwase <[email protected]>
Co-authored-by: Abhishek Kulkarni <[email protected]>
Co-authored-by: Abhishek Kulkarni <[email protected]>
Co-authored-by: Lev Kurilenko <[email protected]>
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.

6 participants