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

Change output of st_as_text() for MULTIPOINT to include inner parens. #1219 #1221

Merged
merged 4 commits into from
Dec 20, 2019

Conversation

ateucher
Copy link
Contributor

If merged, closes #1219.

As stated in the linked issue, the output of lwgeom::st_astext() will continue to omit the inner parens, which will cause an inconsistency unless PostGIS decides to go the same way.

I know you're working on other higher priority issues (i.e., the proj stuff), so no big rush on this.

I have run revdepcheck::revdep_check() on all 214 reverse dependencies and found no issues:

library(revdepcheck)
revdep_report_summary()
#> # Platform
#> 
#> field      value                        
#> ---------  -----------------------------
#> version    R version 3.6.2 (2019-12-12) 
#> os         macOS Mojave 10.14.6         
#> system     x86_64, darwin15.6.0         
#> ui         X11                          
#> language   (EN)                         
#> collate    en_CA.UTF-8                  
#> ctype      en_CA.UTF-8                  
#> tz         America/Vancouver            
#> date       2019-12-19                   
#> 
#> # Dependencies
#> 
#> package    old     new       Δ  
#> ---------  ------  --------  ---
#> sf         0.8-0   0.8-999   *  
#> classInt   0.4-2   0.4-2        
#> DBI        1.1.0   1.1.0        
#> e1071      1.7-3   1.7-3        
#> magrittr   1.5     1.5          
#> Rcpp       1.0.3   1.0.3        
#> units      0.6-5   0.6-5        
#> 
#> # Revdeps
revdep_summary()
#> ✓ adklakedata 0.6.1                      ── E: 0     | W: 0     | N: 0    
#> ✓ amt 0.0.7                              ── E: 0     | W: 0     | N: 0    
#> ✓ arcos 0.8.2                            ── E: 0     | W: 0     | N: 0    
#> ✓ areal 0.1.5                            ── E: 0     | W: 0     | N: 0    
#> ✓ bcdata 0.1.2                           ── E: 0     | W: 0     | N: 0    
#> ✓ bcmaps 0.18.0                          ── E: 0     | W: 0     | N: 0    
#> ✓ bcmaps.rdata 0.1.5                     ── E: 0     | W: 0     | N: 0    
#> ✓ bcmapsdata 0.3.1                       ── E: 0     | W: 0     | N: 0    
#> ✓ BIOMASS 2.1.1                          ── E: 0     | W: 0     | N: 0    
#> ✓ biscale 0.1.2                          ── E: 0     | W: 0     | N: 0    
#> ✓ bnspatial 1.1                          ── E: 0     | W: 0     | N: 0    
#> ✓ brazilmaps 0.1.0                       ── E: 0     | W: 0     | N: 0    
#> ✓ btb 0.1.30                             ── E: 0     | W: 0     | N: 0    
#> ✓ c14bazAAR 1.0.3                        ── E: 0     | W: 0     | N: 0    
#> ✓ cancensus 0.2.0                        ── E: 0     | W: 0     | N: 0    
#> ✓ capm 0.14.0                            ── E: 0     | W: 0     | N: 0    
#> ✓ cartogram 0.2.0                        ── E: 0     | W: 0     | N: 0    
#> ✓ cartography 2.2.1                      ── E: 0     | W: 0     | N: 0    
#> ✓ cdcfluview 0.9.0                       ── E: 0     | W: 0     | N: 0    
#> ✓ censusxy 0.1.2                         ── E: 0     | W: 0     | N: 0    
#> ✓ ckanr 0.4.0                            ── E: 0     | W: 0     | N: 0    
#> ✓ compstatr 0.2.0                        ── E: 0     | W: 0     | N: 0    
#> ✓ concaveman 1.0.0                       ── E: 0     | W: 0     | N: 0    
#> ✓ contact 1.0.1                          ── E: 0     | W: 0     | N: 0    
#> ✓ crawl 2.2.1                            ── E: 0     | W: 0     | N: 0    
#> ✓ crimedata 0.2.0                        ── E: 0     | W: 0     | N: 0    
#> ✓ csodata 0.1.5.0                        ── E: 0     | W: 0     | N: 0    
#> ✓ cyclestreets 0.1.5                     ── E: 0     | W: 0     | N: 0    
#> ✓ DeclareDesign 0.20.0                   ── E: 0     | W: 0     | N: 0    
#> ✓ diffman 0.1.0                          ── E: 0     | W: 0     | N: 0    
#> ✓ dodgr 0.2.5                            ── E: 0     | W: 0     | N: 0    
#> ✓ dssd 0.1.0                             ── E: 0     | W: 0     | N: 0    
#> ✓ ebirdst 0.1.0                          ── E: 0     | W: 0     | N: 0    
#> ✓ echor 0.1.3                            ── E: 0     | W: 0     | N: 0    
#> ✓ EcoIndR 1.6                            ── E: 0     | W: 0     | N: 0    
#> ✓ eddi 0.0.1                             ── E: 0     | W: 0     | N: 0    
#> ✓ eixport 0.4.0                          ── E: 0     | W: 0     | N: 0    
#> ✓ elevatr 0.2.0                          ── E: 0     | W: 0     | N: 0    
#> ✓ EmissV 0.665.1.0                       ── E: 0     | W: 0     | N: 0    
#> ✓ eRTG3D 0.6.2                           ── E: 0     | W: 0     | N: 0    
#> ✓ eSDM 0.3.2                             ── E: 0     | W: 0     | N: 0    
#> ✓ eurostat 3.3.55                        ── E: 0     | W: 0     | N: 0    
#> ✓ exactextractr 0.1.1                    ── E: 0     | W: 0     | N: 0    
#> ✓ fasterize 1.0.0                        ── E: 0     | W: 0     | N: 0    
#> ✓ FedData 2.5.7                          ── E: 0     | W: 0     | N: 0    
#> ✓ fgdr 1.0.0                             ── E: 0     | W: 0     | N: 0    
#> ✓ finbif 0.1.0                           ── E: 0     | W: 0     | N: 0    
#> ✓ fingertipscharts 0.0.10                ── E: 0     | W: 0     | N: 0    
#> ✓ foieGras 0.4.0                         ── E: 0     | W: 0     | N: 0    
#> ✓ GADMTools 3.7-2                        ── E: 0     | W: 0     | N: 0    
#> ✓ gdalUtilities 1.0.0                    ── E: 0     | W: 0     | N: 0    
#> ✓ geobr 1.1                              ── E: 0     | W: 0     | N: 0    
#> ✓ geogrid 0.1.1                          ── E: 0     | W: 0     | N: 0    
#> ✓ geohashTools 0.3.0                     ── E: 0     | W: 0     | N: 0    
#> ✓ geojson 0.3.2                          ── E: 0     | W: 0     | N: 0    
#> ✓ geojsonio 0.8.0                        ── E: 0     | W: 0     | N: 0    
#> ✓ geometa 0.6-1                          ── E: 0     | W: 0     | N: 0    
#> ✓ geometr 0.1.1                          ── E: 0     | W: 0     | N: 0    
#> ✓ geonetwork 0.3                         ── E: 0     | W: 0     | N: 0    
#> ✓ geosample 0.2.1                        ── E: 0     | W: 0     | N: 0    
#> ✓ geoviz 0.2.1                           ── E: 0     | W: 0     | N: 0    
#> ✓ ggformula 0.9.2                        ── E: 0     | W: 0     | N: 0    
#> ✓ ggiraph 0.7.0                          ── E: 0     | W: 0     | N: 0    
#> ✓ ggplot2 3.2.1                          ── E: 0     | W: 0     | N: 0    
#> ✓ ggsn 0.5.0                             ── E: 0     | W: 0     | N: 0    
#> ✓ ggspatial 1.0.3                        ── E: 0     | W: 0     | N: 0    
#> ✓ ggVennDiagram 0.3                      ── E: 0     | W: 0     | N: 0    
#> ✓ googlePolylines 0.7.2                  ── E: 0     | W: 0     | N: 0    
#> ✓ graph4lg 0.2.0                         ── E: 0     | W: 0     | N: 0    
#> ✓ GSODR 2.0.0                            ── E: 0     | W: 0     | N: 0    
#> ✓ gstat 2.0-3                            ── E: 0     | W: 0     | N: 0    
#> ✓ gtfsrouter 0.0.1                       ── E: 0     | W: 0     | N: 0    
#> ✓ GWSDAT 3.0.3                           ── E: 0     | W: 0     | N: 0    
#> ✓ hereR 0.2.1                            ── E: 0     | W: 0     | N: 0    
#> ✓ hydrolinks 0.10.0                      ── E: 0     | W: 0     | N: 0    
#> ✓ ipumsr 0.4.2                           ── E: 0     | W: 0     | N: 0    
#> ✓ isoband 0.2.0                          ── E: 0     | W: 0     | N: 0    
#> ✓ janitor 1.2.0                          ── E: 0     | W: 0     | N: 0    
#> ✓ jpmesh 1.1.3                           ── E: 0     | W: 0     | N: 0    
#> ✓ jpndistrict 0.3.4                      ── E: 0     | W: 0     | N: 0    
#> ✓ kokudosuuchi 0.4.2                     ── E: 0     | W: 0     | N: 0    
#> ✓ LAGOSNE 2.0.1                          ── E: 0     | W: 0     | N: 0    
#> ✓ landsepi 0.0.8                         ── E: 0     | W: 0     | N: 0    
#> ✓ lconnect 0.1.0                         ── E: 0     | W: 0     | N: 0    
#> ✓ leafem 0.0.1                           ── E: 0     | W: 0     | N: 0    
#> ✓ leaflet 2.0.3                          ── E: 0     | W: 0     | N: 0    
#> ✓ leafpm 0.1.0                           ── E: 0     | W: 0     | N: 0    
#> ✓ leafpop 0.0.5                          ── E: 0     | W: 0     | N: 0    
#> ✓ leri 0.0.1                             ── E: 0     | W: 0     | N: 0    
#> ✓ lidR 2.1.4                             ── E: 0     | W: 0     | N: 0    
#> ✓ linemap 0.1.0                          ── E: 0     | W: 0     | N: 0    
#> ✓ link2GI 0.4.1                          ── E: 0     | W: 0     | N: 0    
#> ✓ lutz 0.3.1                             ── E: 0     | W: 0     | N: 0    
#> ✓ lwgeom 0.1-7                           ── E: 0     | W: 0     | N: 0    
#> ✓ macleish 0.3.4                         ── E: 0     | W: 0     | N: 0    
#> ✓ mapdeck 0.2.1                          ── E: 0     | W: 0     | N: 0    
#> ✓ mapedit 0.5.0                          ── E: 0     | W: 0     | N: 0    
#> ✓ mapi 1.0.1                             ── E: 0     | W: 0     | N: 0    
#> ✓ mapsapi 0.4.2                          ── E: 0     | W: 0     | N: 0    
#> ✓ mapview 2.7.0                          ── E: 0     | W: 0     | N: 0    
#> ✓ mlr 2.16.0                             ── E: 0     | W: 0     | N: 0    
#> ✓ MODIS 1.1.6                            ── E: 0     | W: 0     | N: 0    
#> ✓ MODISTools 1.1.0                       ── E: 0     | W: 0     | N: 0    
#> ✓ MODIStsp 1.3.9                         ── E: 0     | W: 0     | N: 0    
#> ✓ moveVis 0.10.3                         ── E: 0     | W: 0     | N: 0    
#> ✓ MTA 0.2.0                              ── E: 0     | W: 0     | N: 0    
#> ✓ nasapower 1.1.3                        ── E: 0     | W: 0     | N: 0    
#> ✓ ncdfgeom 1.1.0                         ── E: 0     | W: 0     | N: 0    
#> ✓ NetLogoR 0.3.6                         ── E: 0     | W: 0     | N: 0    
#> ✓ nhdplusTools 0.3.11                    ── E: 0     | W: 0     | N: 0    
#> ✓ nhdR 0.5.2                             ── E: 0     | W: 0     | N: 0    
#> ✓ NipponMap 0.2                          ── E: 0     | W: 0     | N: 0    
#> ✓ nlaR 0.4.0                             ── E: 0     | W: 0     | N: 0    
#> ✓ nlgeocoder 0.1.3                       ── E: 0     | W: 0     | N: 0    
#> ✓ NLMR 0.4.2                             ── E: 0     | W: 0     | N: 0    
#> ✓ nlrx 0.4.0                             ── E: 0     | W: 0     | N: 0    
#> ✓ nngeo 0.3.0                            ── E: 0     | W: 0     | N: 0    
#> ✓ oceanis 1.0.5                          ── E: 0     | W: 0     | N: 0    
#> ✓ opendatatoronto 0.1.1                  ── E: 0     | W: 0     | N: 0    
#> ✓ openSTARS 1.1.0                        ── E: 0     | W: 0     | N: 0    
#> ✓ Orcs 1.2.0                             ── E: 0     | W: 0     | N: 0    
#> ✓ osmdata 0.1.2                          ── E: 0     | W: 0     | N: 0    
#> ✓ osrm 3.3.2                             ── E: 0     | W: 0     | N: 0    
#> ✓ ows4R 0.1-4                            ── E: 0     | W: 0     | N: 0    
#> ✓ parlitools 0.3.4                       ── E: 0     | W: 0     | N: 0    
#> ✓ pct 0.2.7                              ── E: 0     | W: 0     | N: 0    
#> ✓ pinochet 0.1.0                         ── E: 0     | W: 0     | N: 0    
#> ✓ plotdap 0.0.4                          ── E: 0     | W: 0     | N: 0    
#> ✓ plotly 4.9.1                           ── E: 0     | W: 0     | N: 0    
#> ✓ protolite 2.0                          ── E: 0     | W: 0     | N: 0    
#> ✓ PWFSLSmoke 1.2.100                     ── E: 0     | W: 0     | N: 0    
#> ✓ qualmap 0.1.1                          ── E: 0     | W: 0     | N: 0    
#> ✓ quickmapr 0.3.0                        ── E: 0     | W: 0     | N: 0    
#> ✓ raceland 1.0.3                         ── E: 0     | W: 0     | N: 0    
#> ✓ raster 3.0-7                           ── E: 0     | W: 0     | N: 0    
#> ✓ rasterVis 0.47                         ── E: 0     | W: 0     | N: 0    
#> ✓ rcartocolor 2.0.0                      ── E: 0     | W: 0     | N: 0    
#> ✓ RCzechia 1.4.3                         ── E: 0     | W: 0     | N: 0    
#> ✓ readwritesqlite 0.0.2                  ── E: 0     | W: 0     | N: 0    
#> ✓ reproducible 0.2.11                    ── E: 0     | W: 0     | N: 0    
#> ✓ rerddapXtracto 0.4.5                   ── E: 0     | W: 0     | N: 0    
#> ✓ rFIA 0.1.1                             ── E: 0     | W: 0     | N: 0    
#> ✓ rfishnet2 0.1.0                        ── E: 0     | W: 0     | N: 0    
#> ✓ rgbif 1.4.0                            ── E: 0     | W: 0     | N: 0    
#> ✓ rgeopat2 0.2.6                         ── E: 0     | W: 0     | N: 0    
#> ✓ rgrass7 0.2-1                          ── E: 0     | W: 0     | N: 0    
#> ✓ rmangal 2.0.0                          ── E: 0     | W: 0     | N: 0    
#> ✓ rmapshaper 0.4.1                       ── E: 0     | W: 0     | N: 0    
#> ✓ rmapzen 0.4.2                          ── E: 0     | W: 0     | N: 0    
#> ✓ rnaturalearth 0.1.0                    ── E: 0     | W: 0     | N: 0    
#> ✓ rnoaa 0.9.5                            ── E: 0     | W: 0     | N: 0    
#> ✓ rpostgisLT 0.6.0                       ── E: 0     | W: 0     | N: 0    
#> ✓ RPyGeo 1.0.0                           ── E: 0     | W: 0     | N: 0    
#> ✓ RQGIS 1.0.4                            ── E: 0     | W: 0     | N: 0    
#> ✓ Rsagacmd 0.0.2                         ── E: 0     | W: 0     | N: 0    
#> ✓ rSymbiota 1.0.0                        ── E: 0     | W: 0     | N: 0    
#> ✓ sabre 0.3.2                            ── E: 0     | W: 0     | N: 0    
#> ✓ sdcSpatial 0.1.1                       ── E: 0     | W: 0     | N: 0    
#> ✓ sen2r 1.2.1                            ── E: 0     | W: 0     | N: 0    
#> ✓ sfdct 0.0.6                            ── E: 0     | W: 0     | N: 0    
#> ✓ siland 1.4.5                           ── E: 0     | W: 0     | N: 0    
#> ✓ skimr 2.0.2                            ── E: 0     | W: 0     | N: 0    
#> ✓ slga 1.1.0                             ── E: 0     | W: 0     | N: 0    
#> ✓ SMITIDstruct 0.0.5                     ── E: 0     | W: 0     | N: 0    
#> ✓ smoothr 0.1.1                          ── E: 0     | W: 0     | N: 0    
#> ✓ sociome 1.3.0                          ── E: 0     | W: 0     | N: 0    
#> ✓ SpaDES.core 0.2.7                      ── E: 0     | W: 0     | N: 0    
#> ✓ SpaDES.tools 0.3.4                     ── E: 0     | W: 0     | N: 0    
#> ✓ spatialEco 1.2-1                       ── E: 0     | W: 0     | N: 0    
#> ✓ SpatialKDE 0.5.0                       ── E: 0     | W: 0     | N: 0    
#> ✓ SpatialPosition 2.0.0                  ── E: 0     | W: 0     | N: 0    
#> ✓ spatialreg 1.1-5                       ── E: 0     | W: 0     | N: 0    
#> ✓ spatialrisk 0.6.5                      ── E: 0     | W: 0     | N: 0    
#> ✓ spatialwidget 0.2                      ── E: 0     | W: 0     | N: 0    
#> ✓ spbabel 0.5.0                          ── E: 0     | W: 0     | N: 0    
#> ✓ spData 0.3.2                           ── E: 0     | W: 0     | N: 0    
#> ✓ spdep 1.1-3                            ── E: 0     | W: 0     | N: 0    
#> ✓ spsurvey 4.1.1                         ── E: 0     | W: 0     | N: 0    
#> ✓ stars 0.4-0                            ── E: 0     | W: 0     | N: 0    
#> ✓ stats19 1.1.0                          ── E: 0     | W: 0     | N: 0    
#> ✓ stcos 0.2.1                            ── E: 0     | W: 0     | N: 0    
#> ✓ stlcsb 0.1.2                           ── E: 0     | W: 0     | N: 0    
#> ✓ stormwindmodel 0.1.1                   ── E: 0     | W: 0     | N: 0    
#> ✓ stplanr 0.4.1                          ── E: 0     | W: 0     | N: 0    
#> ✓ streamDepletr 0.1.0                    ── E: 0     | W: 0     | N: 0    
#> ✓ sugarbag 0.1.1                         ── E: 0     | W: 0     | N: 0    
#> ✓ swmmr 0.9.0                            ── E: 0     | W: 0     | N: 0    
#> ✓ tabularaster 0.5.0                     ── E: 0     | W: 0     | N: 0    
#> ✓ tanaka 0.1.1                           ── E: 0     | W: 0     | N: 0    
#> ✓ TargomoR 0.2.0                         ── E: 0     | W: 0     | N: 0    
#> ✓ tidycensus 0.9.2                       ── E: 0     | W: 0     | N: 0    
#> ✓ tidyRSS 1.2.11                         ── E: 0     | W: 0     | N: 0    
#> ✓ tidytransit 0.6.1                      ── E: 0     | W: 0     | N: 0    
#> ✓ tidyUSDA 0.2.4                         ── E: 0     | W: 0     | N: 0    
#> ✓ tigris 0.8.2                           ── E: 0     | W: 0     | N: 0    
#> ✓ tilegramsR 0.2.0                       ── E: 0     | W: 0     | N: 0    
#> ✓ tmap 2.3-1                             ── E: 0     | W: 0     | N: 0    
#> ✓ tmaptools 2.0-2                        ── E: 0     | W: 0     | N: 0    
#> ✓ trackeRapp 1.0                         ── E: 0     | W: 0     | N: 0    
#> ✓ transformr 0.1.1                       ── E: 0     | W: 0     | N: 0    
#> ✓ tricolore 1.2.1                        ── E: 0     | W: 0     | N: 0    
#> ✓ trigpoints 1.0.0                       ── E: 0     | W: 0     | N: 0    
#> ✓ uavRmp 0.5.4                           ── E: 0     | W: 0     | N: 0    
#> ✓ USAboundaries 0.3.1                    ── E: 0     | W: 0     | N: 0    
#> ✓ VancouvR 0.1.1                         ── E: 0     | W: 0     | N: 0    
#> ✓ vein 0.8.0                             ── E: 0     | W: 0     | N: 0    
#> ✓ velociraptr 1.1.0                      ── E: 0     | W: 0     | N: 0    
#> ✓ velox 0.2.0                            ── E: 0     | W: 0     | N: 0    
#> ✓ WaveSampling 0.1.0                     ── E: 0     | W: 0     | N: 0    
#> ✓ wdpar 1.0.0                            ── E: 0     | W: 0     | N: 0    
#> ✓ weathercan 0.3.1                       ── E: 0     | W: 0     | N: 0    
#> ✓ webTRISr 0.2.0                         ── E: 0     | W: 0     | N: 0    
#> ✓ windAC 1.2.0                           ── E: 0     | W: 0     | N: 0    
#> ✓ windfarmGA 2.2.3                       ── E: 0     | W: 0     | N: 0

Created on 2019-12-19 by the reprex package (v0.3.0)

@edzer edzer merged commit 322fa43 into r-spatial:master Dec 20, 2019
@edzer
Copy link
Member

edzer commented Dec 20, 2019

Great work - thanks!

ateucher added a commit to ateucher/sf that referenced this pull request Dec 20, 2019
ateucher added a commit to ateucher/sf that referenced this pull request Dec 20, 2019
edzer added a commit that referenced this pull request Dec 20, 2019
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

Successfully merging this pull request may close these issues.

st_as_text: format of MULTIPOINT
2 participants