-
Notifications
You must be signed in to change notification settings - Fork 45
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
Old-style crs message from sf when running getbb #241
Comments
Thanks @agila5, and yes, that is an problem caused by and internal to |
Previously fixed in #218, but obviously not in all necessary places. Need to link the problem lines identified above to the code used there. |
mpadge
added a commit
that referenced
this issue
Sep 6, 2021
Thanks @agila5 !! library (osmdata)
#> Data (c) OpenStreetMap contributors, ODbL 1.0. https://www.openstreetmap.org/copyright
library (sf)
#> Linking to GEOS 3.9.1, GDAL 3.3.1, PROJ 8.0.1
packageVersion ("sf")
#> [1] '1.0.2'
packageVersion ("osmdata")
#> [1] '0.1.6.7'
test <- getbb("Lecco, Italy", format_out = "sf_polygon")
print (test)
#> Simple feature collection with 2 features and 0 fields
#> Geometry type: POLYGON
#> Dimension: XY
#> Bounding box: xmin: 9.243734 ymin: 45.64938 xmax: 9.541106 ymax: 46.15172
#> Geodetic CRS: WGS 84
#> geometry
#> 1 POLYGON ((9.243734 45.76648...
#> 2 POLYGON ((9.366765 45.87497... Created on 2021-09-06 by the reprex package (v2.0.0.9000) |
Hi @mpadge and thanks for the quick fix 🎉 |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Hi @mpadge, sorry to bother you. I just wanted to report a slightly problematic interaction between
osmdata
andsf
(from v1.0.2). Reprex:Created on 2021-09-04 by the reprex package (v2.0.1)
Session info
A few notes:
I run some quick debug functions, and I think the problem is the following lines of code:
osmdata/R/getbb.R
Lines 391 to 393 in d229226
(probably the same for multipolygon).
I'm not 100% sure this is really a problem in
osmdata
, but I thought that maybe you still want to check it. Moreover, according to sf news, that same message will become a warning in the future.The text was updated successfully, but these errors were encountered: