You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
In the function "theo_del" the ggplot2 function fortify is used to convert the data of a SptialPolygon to a data frame.
con <- ggplot2::fortify(sp_con)
According to the documentation the function fortify may be deprecated in the future.Mmaybe it should be replaced with the appropiate function from the 'broom' package as described by ggplot2 documentation. This works with warnings and 'group' is chr instead of factor but as only order is used, that is not a problem.
BUT: In the broom package documentation it is stated that development of sp tidiers is halted and may be deprecated in the future. They propose changing to sf instead of sp. That would effect the whole package here...
The text was updated successfully, but these errors were encountered:
Updating all functions to sf instead of sp is probably a good idea! Best to start right now, or? End of this month I have some extra time to work on such things.
While I also think that updating to sf, where already possible, is the best way for the future, the question is, if we shouldn't first finish the documentation of the package as it is now. But if you have time at the end of the month, we can certainly talk about it in detail! :)
In the function "theo_del" the ggplot2 function fortify is used to convert the data of a SptialPolygon to a data frame.
con <- ggplot2::fortify(sp_con)
According to the documentation the function fortify may be deprecated in the future.Mmaybe it should be replaced with the appropiate function from the 'broom' package as described by ggplot2 documentation. This works with warnings and 'group' is chr instead of factor but as only order is used, that is not a problem.
BUT: In the broom package documentation it is stated that development of sp tidiers is halted and may be deprecated in the future. They propose changing to sf instead of sp. That would effect the whole package here...
The text was updated successfully, but these errors were encountered: