-
Notifications
You must be signed in to change notification settings - Fork 66
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
Demote sp and raster to suggests #332
Comments
Bumped up priority list. |
OK going to work on this after mentioning as a priority in #362 |
Work on this: #481 |
Some insight into dependencies in v1.8.7, x marks ones targetted for removal:
dg = miniCRAN::makeDepGraph(pkg = "stplanr", includeBasePkgs = FALSE, suggests = FALSE)
plot(dg) # mass of pkgs dg_betweenness = igraph::betweenness(dg)
summary(dg_betweenness)
#> Min. 1st Qu. Median Mean 3rd Qu. Max.
#> 0.000 0.000 0.000 1.879 2.792 19.500
class(dg_betweenness)
#> [1] "numeric"
dg_many = dg_betweenness > 1
# dg_small = igraph::delete_vertices(graph = dg, v = dg_many)
dg_small = igraph::induced_subgraph(graph = dg, which(dg_many))
plot(dg_small) Created on 2022-04-03 by the reprex package (v2.0.1) |
|
A benefit of this refactoring work is that the package will be lighter. Latest version of the zips: https://github.com/ropensci/stplanr/archive/refs/heads/master.zip the package is 7.1 MB, a bit on the big side... |
Promising result, only 2 NOTEs to fix on win devel test: https://win-builder.r-project.org/KN7sTaJ8LeFF/00check.log |
Will speed up install times, could lead to future depreciation.
The text was updated successfully, but these errors were encountered: