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

[WIP] pydantic2 updates #29

Merged
merged 4 commits into from
Dec 20, 2023
Merged

[WIP] pydantic2 updates #29

merged 4 commits into from
Dec 20, 2023

Conversation

gpetretto
Copy link
Contributor

Updates to make jobflow_remote compatible with pydantic version >=2.
In principle I tested and it seems to work correctly with the latest versions of jobflow and maggma. However, I prefer not to merge right away, as this will be incompatible with pydantic<2 and it would require updating the whole set of MP related packages. I started to open this PR in case anybody needs it or wants to review/test.

@ml-evs
Copy link
Member

ml-evs commented Nov 17, 2023

Is there a blocker for this @gpetretto? Now atomate2 has updated (and is now adding many new workflows each release) I guess we need to catch up... I'll test this out and see if I run into any issues.

@davidwaroquiers
Copy link
Member

Is there a blocker for this @gpetretto? Now atomate2 has updated (and is now adding many new workflows each release) I guess we need to catch up... I'll test this out and see if I run into any issues.

These changes are included in the #37 I think. Do you think it would be good to merge it before anyway ?

@gpetretto
Copy link
Contributor Author

Indeed these changes are also included in the other PR.
I merged there the commits from this branch, so in principle it should not be a problem to merge this one as well.

@davidwaroquiers davidwaroquiers merged commit a60d5c5 into develop Dec 20, 2023
4 checks passed
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