Skip to content

Conversation

@paloma-martinez
Copy link
Collaborator

@paloma-martinez paloma-martinez commented Dec 5, 2025

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.

@paloma-martinez paloma-martinez self-assigned this Dec 5, 2025
@paloma-martinez paloma-martinez changed the title feature: Adding a script to generate schema feat: Adding a script to generate schema Dec 5, 2025
@paloma-martinez paloma-martinez marked this pull request as ready for review December 12, 2025 12:38
Copy link
Collaborator

@jafranc jafranc left a 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

Comment on lines 44 to 48
cd geos-trame
python -m venv pydantic-venv
source pydantic-venv/bin/activate
pip install -e .
pip install "xsdata[cli]"
Copy link
Collaborator

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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants