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

OGR GML driver uses bbox instead of actual feature geometry #7925

Closed
pathmapper opened this issue Jun 12, 2023 · 3 comments · Fixed by #7927
Closed

OGR GML driver uses bbox instead of actual feature geometry #7925

pathmapper opened this issue Jun 12, 2023 · 3 comments · Fixed by #7927
Assignees

Comments

@pathmapper
Copy link
Contributor

pathmapper commented Jun 12, 2023

Not entirely sure if this is a GDAL issue, so please close here if it's QGIS related and I'll file it there.

Expected behavior

OGR GML driver uses actual feature geometry.

Actual behavior

OGR GML driver uses bbox instead of actual geometry, GFS from GDAL 3.7.0 includes the following compared to the GFS from GDAL 3.6.2:

grafik

Steps to reproduce the problem.

Drag'n'drop the attached test.gml in QGIS. GFS from GDAL 3.6.2 and 3.7.0 also attached.

test_data.zip

Result after drag'n'drop in QGIS:
grafik

Operating system

Windows 10 Version 2009

GDAL version and provenance

3.7.0 installed with QGIS standalone installer 3.30.3

@jratike80
Copy link
Collaborator

That data seem to have a very complex schema. In addition, the schema is hard to find because this location is no more valid xmlns:xplan="http://www.xplanung.de/xplangml/5/2" and it redirects into an URL that gives an error "page not found" https://xleitstelle.de/xplangml/5/2

I managed to find where the schema is and the advanced GMLAS driver is able to resolve it.

ogrinfo gmlas:test.gml -oo XSD="https://repository.gdi-de.org/schemas/de.xleitstelle.xplanung/5.2/XPlanGML.xsd" -oo remove_unused_layers=yes -oo remove_unused_fields=yes

I tested one layer and GMLAS driver finds linestrings:

ogrinfo gmlas:test.gml -oo XSD="https://repository.gdi-de.org/schemas/de.xleitstelle.xplanung/5.2/XPlanGML.xsd" -oo remove_unused_layers=yes -oo remove_unused_fields=yes bp_baugrenze

Layer bp_plan has more complicated geometries like MULTISURFACE (CURVEPOLYGON (COMPOUNDCURVE (CIRCULARSTRING ....

@pathmapper
Copy link
Contributor Author

pathmapper commented Jun 12, 2023

Thanks @jratike80 !

it redirects into an URL that gives an error "page not found"

I'll report that to XLeitstelle.

Any idea what could have changed between GDAL 3.6.2 and 3.7.0 what could cause this issue with the GML driver?

Edit: True, the schema is very complex...

@rouault
Copy link
Member

rouault commented Jun 12, 2023

Definitely a regression. This is related to the #7133 change. I'm investigating

@rouault rouault self-assigned this Jun 12, 2023
rouault added a commit to rouault/gdal that referenced this issue Jun 12, 2023
…se features have gml:boundedBy elements and the geometry element name is different between the different layers (fixes OSGeo#7925)
rouault added a commit to rouault/gdal that referenced this issue Jun 12, 2023
…se features have gml:boundedBy elements and the geometry element name is different between the different layers (fixes OSGeo#7925)
rouault added a commit that referenced this issue Jun 13, 2023
…se features have gml:boundedBy elements and the geometry element name is different between the different layers (fixes #7925)
pathmapper added a commit to kreis-viersen/xplan-reader that referenced this issue Jun 15, 2023
jeffypooo pushed a commit to IgnisTechnologies/gdal that referenced this issue Aug 9, 2023
…se features have gml:boundedBy elements and the geometry element name is different between the different layers (fixes OSGeo#7925)
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 a pull request may close this issue.

3 participants