-
Notifications
You must be signed in to change notification settings - Fork 4
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
Layer that is a data.frame #62
Comments
@annakrystalli for a change would you like to try running this one? 😅 |
Same error for me in I think it's because the package does some post processing to the crs that fails if the response is not an |
the layer type in the layer medata from wfs info was sf, should it indicate something else? |
I see "sf" is hard-coded, maybe I can get it from the web service 👀 |
I modified ows4R code to I get
|
I wonder whether we should expect more metadata before the download. |
And when we do get a data.frame, the layers can obviously NOT be reduced. |
So, TODOS
|
That's ok because the function only tries to reduce and if it can't it returns a list |
https://geo.vliz.be/geoserver/Dataportal/wfs?SERVICE=WFS&REQUEST=DescribeFeatureType&VERSION=2.0.0 does not have info that it's a table |
Related eblondel/ows4R#65 |
For any layer if we get its description and see no element is a geometry, we can know it will be a data.frame rather than an sf object. |
I think @maelle 's approach in eblondel/ows4R#65 (comment) would be great. There are layers in there with no geometry as they are required or used in the infrastructure of EMODnet-Biology, so we cannot really get rid of them. In any case I will check why some layers don't have a geometry. For instance Dataportal:eurobis-obisenv_basic. Probably someone just forgot as the important fields when exporting to csv are decimalLongitude and decimalLatitude. |
@salvafern #83 is meant to tackle this. |
@salvafern My main problem in #83 is the size of fixtures. So if you can have a look at the functionality, that'd be great, I can keep thinking about the testing. |
Created on 2022-03-02 by the reprex package (v2.0.1)
The text was updated successfully, but these errors were encountered: