r-spatial / evolution

Preparing CRAN for the retirement of rgdal, rgeos and maptools
MIT License
26 stars 3 forks source link

request to include mssql driver in sf/terra #11

Closed cmundy closed 1 year ago

cmundy commented 1 year ago

Hi,

For many years I have taken advantage of the sp/rgdal/rgeos suite of packages for my spatial needs. I have been watching the progression of sf/terra and updating all my scripts as can. The one hiccup I have encountered so far is that the drive for mssql is not included in sf a this point (mssqlSpatial is still supported by gdal/ogr).

Is it possible to flag the inclusion of the ogr drive for mssql spatial please?

Thanks Craig

ps and many thanks for the r-spatial packages over the past 15 years!

rsbivand commented 1 year ago

The driver list depends only on the GDAL build. Please show the session.info() of the platform where it is present and where you now think it is absent. I guess there may be a difference between R 4.1 and R 4.2 on Windows, or between x86_64 and aarch on macOS, but you do not state your platforms.

cmundy commented 1 year ago

My apologies Roger,

sessionInfo() R version 4.2.2 (2022-10-31 ucrt) Platform: x86_64-w64-mingw32/x64 (64-bit) Running under: Windows 10 x64 (build 19045)

Matrix products: default

locale: [1] LC_COLLATE=English_Australia.utf8 LC_CTYPE=English_Australia.utf8 [3] LC_MONETARY=English_Australia.utf8 LC_NUMERIC=C [5] LC_TIME=English_Australia.utf8

attached base packages: [1] stats graphics grDevices utils datasets methods base

loaded via a namespace (and not attached): [1] compiler_4.2.2 tools_4.2.2

library(sf) Linking to GEOS 3.9.3, GDAL 3.5.2, PROJ 8.2.1; sf_use_s2() is TRUE library(terra) terra 1.7.3

Session info below. Yes I do think it’s a platform issue.

At the following link the MSSQLSpatial driver appears in the list.

Complete list of drivers from my machine using st_drivers(what = "vector")

name AVCBin AVCE00 AmigoCloud BAG CAD CSV CSW Carto DGN DXF EDIGEO EEDA ESRI Shapefile ESRIC ESRIJSON Elasticsearch FlatGeobuf GML GPKG GPSBabel GPX GeoJSON GeoJSONSeq GeoRSS Geoconcept HTTP Idrisi JML JP2OpenJPEG KML LVBAG MBTiles MVT MapInfo File MapML Memory MySQL NGW OAPIF ODS OGCAPI OGR_GMT OGR_PDS OGR_SDTS OGR_VRT OSM OpenFileGDB PDF PDS4 PGDUMP PLSCENES S57 SQLite SVG SXF Selafin TIGER TopoJSON UK .NTF VDV VFK VICAR WAsP WFS XLS XLSX netCDF

From: Roger Bivand @.> Sent: Tuesday, February 14, 2023 7:20 PM To: r-spatial/evolution @.> Cc: Craig Mundy @.>; Author @.> Subject: Re: [r-spatial/evolution] request to include mssql driver in sf/terra (Issue #11)

The driver list depends only on the GDAL build. Please show the session.info() of the platform where it is present and where you now think it is absent. I guess there may be a difference between R 4.1 and R 4.2 on Windows, or between x86_64 and aarch on macOS, but you do not state your platforms. — Reply to this email directly, https://github.com/r-spatial/evolution/issues/11#issuecomment-1429311871, or https://github.com/notifications/unsubscribe-auth/ACRUTC5MHHBSV77VO2YSGADWXM53HANCNFSM6AAAAAAU3CTSQ4. You are receiving this because you authored the thread.Message ID: @.***>

This email is confidential, and is for the intended recipient only. Access, disclosure, copying, distribution, or reliance on any of it by anyone outside the intended recipient organisation is prohibited and may be a criminal offence. Please delete if obtained in error and email confirmation to the sender. The views expressed in this email are not necessarily the views of the University of Tasmania, unless clearly intended otherwise.

cmundy commented 1 year ago

Apologies,

End of a long day.

Mssqlspatial appears in this list https://r-spatial.github.io/sf/reference/st_drivers.html

From: Roger Bivand @.> Sent: Tuesday, February 14, 2023 7:20 PM To: r-spatial/evolution @.> Cc: Craig Mundy @.>; Author @.> Subject: Re: [r-spatial/evolution] request to include mssql driver in sf/terra (Issue #11)

The driver list depends only on the GDAL build. Please show the session.info() of the platform where it is present and where you now think it is absent. I guess there may be a difference between R 4.1 and R 4.2 on Windows, or between x86_64 and aarch on macOS, but you do not state your platforms.

— Reply to this email directly, view it on GitHubhttps://github.com/r-spatial/evolution/issues/11#issuecomment-1429311871, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ACRUTC5MHHBSV77VO2YSGADWXM53HANCNFSM6AAAAAAU3CTSQ4. You are receiving this because you authored the thread.Message ID: @.**@.>>

This email is confidential, and is for the intended recipient only. Access, disclosure, copying, distribution, or reliance on any of it by anyone outside the intended recipient organisation is prohibited and may be a criminal offence. Please delete if obtained in error and email confirmation to the sender. The views expressed in this email are not necessarily the views of the University of Tasmania, unless clearly intended otherwise.

edzer commented 1 year ago

See also https://github.com/r-spatial/sf/issues/2088

rsbivand commented 1 year ago

"MSSQLSpatial" driver is in Rtools43, so is in packages built for forthcoming R 4.3 on Windows.

cmundy commented 1 year ago

Thanks all for your help – much appreciated.

And thanks again for all your work in R-spatial.

Craig

From: Roger Bivand @.> Sent: Tuesday, February 14, 2023 11:46 PM To: r-spatial/evolution @.> Cc: Craig Mundy @.>; Author @.> Subject: Re: [r-spatial/evolution] request to include mssql driver in sf/terra (Issue #11)

Closed #11https://github.com/r-spatial/evolution/issues/11 as completed.

— Reply to this email directly, view it on GitHubhttps://github.com/r-spatial/evolution/issues/11#event-8515094158, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ACRUTC36G6WSYTYTZMQ5A6TWXN5AVANCNFSM6AAAAAAU3CTSQ4. You are receiving this because you authored the thread.Message ID: @.**@.>>

This email is confidential, and is for the intended recipient only. Access, disclosure, copying, distribution, or reliance on any of it by anyone outside the intended recipient organisation is prohibited and may be a criminal offence. Please delete if obtained in error and email confirmation to the sender. The views expressed in this email are not necessarily the views of the University of Tasmania, unless clearly intended otherwise.