-
Notifications
You must be signed in to change notification settings - Fork 0
feat: Adding a script to generate schema #198
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
base: main
Are you sure you want to change the base?
Conversation
This reverts commit 5d7af1b.
jafranc
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I left a proposal for :
- simplify update of the schema files by fetching the schema on the GEOS repo and the commit numbers
- get the old-school regex to change all SomeThingType to SomeThing but the ProblemType due to its root-nature
Lmk if it is good for you and working.
Tested on multiple old commit, worked at least from 5368ee - Sept 2024.
Cannot go prior introduction of VtkWells
| cd geos-trame | ||
| python -m venv pydantic-venv | ||
| source pydantic-venv/bin/activate | ||
| pip install -e . | ||
| pip install "xsdata[cli]" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This sounds like a workaround trick to avoid ruff inconsistent call in xsdata <=24.6.0. I'd rather enforce the use of xsdata>24.6.1 and it is working.
This PR adds a script to generate easily the pydantic classes that depends on the GEOS schema.xsd file. As this schema may be slightly modified regularly, this process allows to have a dependable way to update it.