Migrate Viz to use Pydantic V2 - Githubissues

kedro-org / kedro-viz

Visualise your Kedro data and machine-learning pipelines and track your experiments.

https://demo.kedro.org

Apache License 2.0

662 stars 110 forks source link

Migrate Viz to use Pydantic V2 #1743

Closed ravi-kumar-pilla closed 6 months ago

ravi-kumar-pilla commented 6 months ago

Description

Resolves #1603

Development notes

Followed pydantic migration guide
Partially updated code using bump-pydantic tool
Manually updated @validators to @field_validators
Updated requirements.txt with new pydantic version
Updated test files as per lint suggestions

QA notes

All pytests should pass and kedro viz run should work as expected

Note: Observed flowchart node arrangements change (mirrored) due to the migration, though the data remains un-affected as below -

Before Migration:

current_placement

After Migration:

pydantic_v2_placement

Checklist

[x] Read the contributing guidelines
[x] Opened this PR as a 'Draft Pull Request' if it is work-in-progress
[ ] Updated the documentation to reflect the code changes
[x] Added new entries to the RELEASE.md file
[ ] Added tests to cover my changes

astrojuanlu commented 6 months ago

It's happeninggggggggg

MarkusSagen commented 6 months ago

Fantastic!!