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

gml:boundedBy issue with BBOX definition #21

Open
ninsbl opened this issue Aug 18, 2017 · 6 comments
Open

gml:boundedBy issue with BBOX definition #21

ninsbl opened this issue Aug 18, 2017 · 6 comments

Comments

@ninsbl
Copy link

ninsbl commented Aug 18, 2017

When we (me and @pesekon2) tried using the sos4R client (https://cran.r-project.org/web/packages/sos4R/index.html) to fetch data from istSOS we ran into issues with spatial data of the response.
In particular, bounding box of the offerings was returned as NULL.

When looking at the response here:
http://istsos.org/istsos/demo?request=getCapabilities&section=contents&service=SOS
it seems that the bounding box is given differently from GML definition of envelope (www.datypic.com/sc/niem21/e-gml32_boundedBy.html). In fact, the response does not seem to be correct as only one coordinate pair is given (not both upper and lower corner). A second point and maybe the proper XML tags seem to be missing...

In comparison, http://sensorweb.demo.52north.org/52n-sos-webapp/service?service=SOS&request=GetCapabilities gives the BBOX exactly as in the GML definition... Maybe a bug?

@ninsbl
Copy link
Author

ninsbl commented Aug 20, 2017

Now I tested also with OWSlib, and OWSlib also returns neither BoundingBox nor CRS...

@ninsbl
Copy link
Author

ninsbl commented Aug 23, 2017

Seems to affect also the QGIS SOS client, which does not use OWSlib I think...

@ninsbl
Copy link
Author

ninsbl commented Aug 25, 2017

Seems to affect only SOS 1.0.0

r += " " + str(ofl.boundedBy) + "\n"

as the bpoundedBy definition for SOS 2.0 appears to be correct
r += " <gml:Envelope srsName=\"http://www.opengis.net/def/crs/EPSG/0/%s\">\n" % sosConfig.istsosepsg

@pesekon2 do you think you could work on this and create a PR?

@ninsbl
Copy link
Author

ninsbl commented Aug 27, 2017

The bug can also be here:

def BuildOffEnvelope(pgdb,id,sosConfig):

@ninsbl
Copy link
Author

ninsbl commented Aug 27, 2017

Actually, in:

sql = "SELECT ST_asgml(Box2D(u.geom)) as ext FROM"

a full GML 3 Envelope could be build like this:
SELECT ST_AsGML(3, ST_Collect(u.geom), 15, 32) as ext

@pesekon2
Copy link
Contributor

pesekon2 commented Aug 28, 2017

The problem is that ofl in GCresponseRender has just a method boundedBy and boundedBy nor ofl doesn't contain anything like lowerX.

For me, the best way seems to involve here also this part from version 2.0.0 somehow. But before I start something like this, I would like to know from istSOS developers whether is it acceptable way and whether it can't destroy something.

PS: Is there any way to activate the debug level when quering through html request?

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

No branches or pull requests

2 participants