Ticket #2311 (closed enhancement: fixed)

Opened 2 years ago

Last modified 10 months ago

GML support to read posList and exterior/interior available in gml 3.1 SFL0

Reported by: assefa Assigned to: warmerdam
Priority: normal Milestone: 1.6.0
Component: OGR_SF Version: unspecified
Severity: normal Keywords: GML
Cc: tomkralidis

Description

Upgrade to support the posList element found in gml3.1.x and also the possiblity to read exterior and interior elements for a polygon.

I will attach a patch for this. Please review and apply if appropriate.

Attachments

gml2ogrgeometry.diff (3.9 kB) - added by assefa on 04/03/08 10:44:35.
ogr_gml_geom.diff (1.8 kB) - added by assefa on 04/03/08 10:47:42.
autotest for gml

Change History

04/03/08 10:44:35 changed by assefa

  • attachment gml2ogrgeometry.diff added.

04/03/08 10:47:42 changed by assefa

  • attachment ogr_gml_geom.diff added.

autotest for gml

04/03/08 11:00:00 changed by tomkralidis

  • cc set to tomkralidis.

FYI this originates from MapServer?'s implementation of OGC Filter 1.1.0 (http://trac.osgeo.org/mapserver/ticket/2111)

06/18/08 10:07:40 changed by assefa

Is there a major reason not to apply this patch?

I am also adding in this bug a thread from the mapserver list that references this bug:

"There is a patch submitted in http://trac.osgeo.org/gdal/ticket/2311 that could help in this case. The patch was intended for the reading of gml3 profile 1.

Couple of issues I had when proposing the patch is that the support for things like gml:posList are limited. According to the specs for gml3.1/level 1 gml:posList does not have any attribute but for gml3.1, it could have attributes such as a dimension (<gml:posList srsDimension="3">. The patch assumes a 2 dimension geometry. It is also my understanding that currently the ogr gml parser strips out any attribute information from the gml file, and things like "srsDimension" are not available in gml2ogrgeometry.cpp. I think any upgrade to the reader would have to address this.

Note that using this patch and modifying it a bit to assume a 3d geometries for posList elements, I was able to read polygon/line/points coming out of the wfs server (http://frameworkwfs.usgs.gov/) and display it in Mapserver. "

06/25/08 15:30:47 changed by warmerdam

  • keywords set to GML.
  • status changed from new to assigned.
  • milestone set to 1.6.0.

Assefa,

I've dragged my heels on this because there is another stream of GML3 supporting work going on for the NAS format. I had been thinking I would incorporate that at some point. However, scanning the actual change you propose it seems quite harmless, if perhaps not a comprehensive implementation of GML3 support. So I'll apply it.

Thanks!

(follow-up: ↓ 6 ) 06/25/08 15:40:32 changed by warmerdam

  • status changed from assigned to closed.
  • resolution set to fixed.

The changes have been committed in trunk (r14758).

06/25/08 16:14:27 changed by assefa

Thanks.

(in reply to: ↑ 4 ) 05/10/09 13:15:19 changed by chaitanya

Replying to warmerdam:

The changes have been committed in trunk (r14758).

Typo. It's actually r14768

05/30/09 16:07:19 changed by rouault

In r17155 (1.7.0dev) : Add support for gml3.1.1 srsDimension attribute, to deal with 3D geometries (#2311). Test added in r17156