ropensci / geojsonio

Convert many data formats to & from GeoJSON & TopoJSON
https://docs.ropensci.org/geojsonio
Other
150 stars 59 forks source link

fix for stringsAsFactors for R v4 #166

Closed sckott closed 4 years ago

sckott commented 4 years ago

Specifically, see the problems shown for the r-devel Debian checks.

These can be reproduced by checking with --as-cran using a very current r-devel (r77865 or later), which makes data.frame() and read.table() use a stringsAsFactors = FALSE default, which is planned to become the new default for the upcoming R 4.0.0.

Please see https://developer.r-project.org/Blog/public/2020/02/16/stringsasfactors/index.html for more information about this change.

Can you please fix your package to work with both the old and new default? In principle, this can easily be achieved by adding stringsAsFactors = TRUE to the relevant calls to data.frame() or read.table() [or other read.* function calling read.table()], but please only do this if the sort order used in the string to factor conversion really does not matter (see the blog post about the locale dependence of the conversion). Otherwise, please change to create the factors with explicitly given levels.

The new problems may be from code in a package you depend on: in this case, please let me know, and get in touch with the maintainer of that package.

Please correct before 2020-03-20 to safely retain your package on CRAN.

github-actions[bot] commented 2 years ago

This issue has been automatically locked. If you believe you have found a related problem, please file a new issue (with a reprex: https://reprex.tidyverse.org) and link to this issue.