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

WFS 'filter - select extent on canvas' : unexpected type 'QgsCoordinateReferenceSystem' #58

Closed
sgrellet opened this issue Feb 5, 2018 · 7 comments
Assignees
Labels
bug Houston, we've a problem!

Comments

@sgrellet
Copy link
Member

sgrellet commented Feb 5, 2018

connected to https://wfspoc.brgm-rec.fr/geoserver/ows
-> choosing Feature type 'ef:EnvironmentalMonitoringFacility'
-> Filter by extent with select extent on canvas that generates that BBOX : 7.51633453933924,8.00596800433564,48.35317290336555,48.804760524798034,EPSG:4326
-> 'Download to': a temporary file or an explicit file path
-> then 'Download'

: 2018-01-16T10:52:48 WARNING Traceback (most recent call last):
File "C:/Users/grellets/AppData/Roaming/QGIS/QGIS3\profiles\default/python/plugins\gml_application_schema_toolbox\gui\load_panel.py", line 213, in on_downloadButton_clicked
out = self.download()
File "C:/Users/grellets/AppData/Roaming/QGIS/QGIS3\profiles\default/python/plugins\gml_application_schema_toolbox\gui\load_panel.py", line 282, in download
params['bbox'] = self._get_bbox(wfs)
File "C:/Users/grellets/AppData/Roaming/QGIS/QGIS3\profiles\default/python/plugins\gml_application_schema_toolbox\gui\load_panel.py", line 251, in _get_bbox
transform = QgsCoordinateTransform(self.bboxWidget.crs(), default_crs)
TypeError: QgsCoordinateTransform(): arguments did not match any overloaded call:
overload 1: too many arguments
overload 2: not enough arguments
overload 3: not enough arguments
overload 4: not enough arguments
overload 5: argument 1 has unexpected type 'QgsCoordinateReferenceSystem'

		  Version de Python : 3.6.0 (v3.6.0:41df79263a11, Dec 23 2016, 08:06:12) [MSC v.1900 64 bit (AMD64)] 
		  Version de QGIS : 2.99.0-Master Master, b4be39f98b 
@sgrellet sgrellet added the bug Houston, we've a problem! label Feb 5, 2018
@sgrellet sgrellet added this to Bugs in GMLAS toolbox 1.2.0 Feb 5, 2018
@sgrellet
Copy link
Member Author

sgrellet commented Mar 1, 2018

Tested ok on v1.2.0-rc2 using QGIS 3.0.0-Girona code revision 001c80b0c3

@sgrellet sgrellet closed this as completed Mar 1, 2018
@sgrellet sgrellet moved this from Bugs to Done in GMLAS toolbox 1.2.0 Mar 1, 2018
@sgrellet sgrellet reopened this Mar 1, 2018
@sgrellet
Copy link
Member Author

sgrellet commented Mar 1, 2018

@sgrellet sgrellet moved this from Done to Bugs in GMLAS toolbox 1.2.0 Mar 1, 2018
@sgrellet
Copy link
Member Author

sgrellet commented Mar 5, 2018

Moreover,
Testing on that WFS 1.1.0 (http://services.sandre.eaufrance.fr/geo/eth_FXX?) with the following BBOX (1.7652995014001838,2.510791612594774,47.66389334744894,48.081820743118634,EPSG:4326), open XML Mode and 'Load from WFS Layer':
-> I get rivers from the bottom half of France eventhough the BBOX was just around Orléans (linked to #57 (comment))

@mhugo
Copy link
Collaborator

mhugo commented Apr 11, 2018

looking at the toolbox logs it seems that, even if the 'Limit to' is unticket the WFS request still contains the count parameter (see below)

Fixed now in de58d3e

@mhugo
Copy link
Collaborator

mhugo commented Apr 11, 2018

On BRGM server with the bbox, I can't reproduce the issue, and get the correct result.

On Sandre server, the returned features are indeed wrong. The two servers declare the same default CRS (urn:ogc:def:crs:EPSG::4326). So it seems the problem is server side ?

@mhugo
Copy link
Collaborator

mhugo commented Apr 12, 2018

I was wrong, bbox parameters order was wrong, fixed with #57

@mhugo mhugo closed this as completed Apr 12, 2018
@sgrellet
Copy link
Member Author

fixed under
GMLAS toolbox plugin : v1.2.0-rc4
Python : 3.6.0 (v3.6.0:41df79263a11, Dec 23 2016, 08:06:12) [MSC v.1900 64 bit (AMD64)]
QGIS : 3.0.1-Girona Girona, a86bec25ebw

@sgrellet sgrellet moved this from Bugs to Done in GMLAS toolbox 1.2.0 Apr 13, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Houston, we've a problem!
Projects
No open projects
Development

No branches or pull requests

2 participants