-
Notifications
You must be signed in to change notification settings - Fork 7
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
Please remove dependencies on **rgdal**, **rgeos**, and/or **maptools** #18
Comments
Thanks Roger. Will do so. When the Related to this: paleolimbot/ggspatial#110 |
Does this mean that the use of rgeos in |
Yes, I forgot rgeos. I'd need to rewrite many functions. Fixing that has to wait some time. |
This package has strong dependence on at least one of |
Thanks, will fix before October. |
Thanks! If you can manage June (when we switch |
Please also see https://r-spatial.org/r/2023/04/10/evolution3.html, fix best by June, latest October 2023. |
Ok, understood. I have a lot to do at work these days but will manage to uncouple sp, rgdal, rgeos and raster latest by October 2023. |
You may safely leave sp and raster, only rgdal, rgeos and maptools are retiring. https://r-spatial.org/r/2023/04/10/evolution3.html has lots of hints on work-arounds. |
Should be done now: https://github.com/mikkovihtakari/ggoceanmaps/tree/sf-version It goes to CRAN once I have controlled that everything works. |
Looks good to go here, sp status 2 and without retiring packages on the search path; thanks! |
@rsbivand, any idea why ggOceanMaps still imports sp, raster and rgeos even though these are not mentioned anywhere in DESCRIPTION nor NAMESPACE? Here is the entire branch: https://github.com/MikkoVihtakari/ggOceanMaps/tree/sf-version library(ggOceanMaps)
#> Loading required package: ggplot2
#> ggOceanMaps: Setting data download folder to a temporary folder
#> /var/folders/9v/b70pd53x04d3jjmlrbcgp4_w0000gv/T//RtmpcEjorI. This
#> means that any downloaded map data need to be downloaded again when you
#> restart R. To avoid this problem, change the default path to a
#> permanent folder on your computer. Add following lines to your
#> .Rprofile file: {.ggOceanMapsenv <- new.env(); .ggOceanMapsenv$datapath
#> <- 'YourCustomPath'}. You can use usethis::edit_r_profile() to edit the
#> file.'~/Documents/ggOceanMapsLargeData'would make it in a writable
#> folder on most operating systems.
pack <- available.packages()
pack["ggOceanMaps","Imports"]
#> [1] "sp, raster, sf, rgeos, methods, utils, stars, smoothr, units,\ndplyr, parallel" Created on 2023-07-04 with reprex v2.0.2 Here is session info in case it helps library(ggOceanMaps)
#> Loading required package: ggplot2
#> ggOceanMaps: Setting data download folder to a temporary folder
#> /var/folders/9v/b70pd53x04d3jjmlrbcgp4_w0000gv/T//RtmpzXoXAz. This
#> means that any downloaded map data need to be downloaded again when you
#> restart R. To avoid this problem, change the default path to a
#> permanent folder on your computer. Add following lines to your
#> .Rprofile file: {.ggOceanMapsenv <- new.env(); .ggOceanMapsenv$datapath
#> <- 'YourCustomPath'}. You can use usethis::edit_r_profile() to edit the
#> file.'~/Documents/ggOceanMapsLargeData'would make it in a writable
#> folder on most operating systems.
sessionInfo()
#> R version 4.3.1 (2023-06-16)
#> Platform: x86_64-apple-darwin20 (64-bit)
#> Running under: macOS Ventura 13.3.1
#>
#> Matrix products: default
#> BLAS: /Library/Frameworks/R.framework/Versions/4.3-x86_64/Resources/lib/libRblas.0.dylib
#> LAPACK: /Library/Frameworks/R.framework/Versions/4.3-x86_64/Resources/lib/libRlapack.dylib; LAPACK version 3.11.0
#>
#> locale:
#> [1] en_US.UTF-8/en_US.UTF-8/en_US.UTF-8/C/en_US.UTF-8/en_US.UTF-8
#>
#> time zone: Europe/Oslo
#> tzcode source: internal
#>
#> attached base packages:
#> [1] stats graphics grDevices utils datasets methods base
#>
#> other attached packages:
#> [1] ggOceanMaps_2.0.0 ggplot2_3.4.2
#>
#> loaded via a namespace (and not attached):
#> [1] lwgeom_0.2-13 gtable_0.3.3 dplyr_1.1.2 compiler_4.3.1
#> [5] reprex_2.0.2 tidyselect_1.2.0 Rcpp_1.0.10 parallel_4.3.1
#> [9] scales_1.2.1 yaml_2.3.7 fastmap_1.1.1 R6_2.5.1
#> [13] generics_0.1.3 classInt_0.4-9 sf_1.0-13 knitr_1.43
#> [17] tibble_3.2.1 stars_0.6-1 units_0.8-2 munsell_0.5.0
#> [21] DBI_1.1.3 pillar_1.9.0 rlang_1.1.1 utf8_1.2.3
#> [25] xfun_0.39 fs_1.6.2 cli_3.6.1 withr_2.5.0
#> [29] magrittr_2.0.3 class_7.3-22 digest_0.6.32 grid_4.3.1
#> [33] rstudioapi_0.14 lifecycle_1.0.3 vctrs_0.6.3 KernSmooth_2.23-21
#> [37] proxy_0.4-27 evaluate_0.21 glue_1.6.2 abind_1.4-5
#> [41] fansi_1.0.4 e1071_1.7-13 colorspace_2.1-0 rmarkdown_2.23
#> [45] tools_4.3.1 pkgconfig_2.0.3 htmltools_0.5.5 Created on 2023-07-04 with reprex v2.0.2 |
I'd expect the databases to synch within about 24 hours. |
Thanks for the clarification :) Well, it should only use sf and stars now. I hope you can retire soon :) |
Yes, just 206 packages left to help ... thanks for your help! |
This package depends on (depends, imports or suggests) raster and one or more of the retiring packages rgdal, rgeos or maptools (https://r-spatial.org/r/2022/04/12/evolution.html, https://r-spatial.org/r/2022/12/14/evolution2.html). Since raster
3.6.3
, all use of external FOSS library functionality has been transferred to terra, making the retiring packages very likely redundant. It would help greatly if you could remove dependencies on the retiring packages as soon as possible.Maybe related to #5
The text was updated successfully, but these errors were encountered: