Please undo plan() changes + consider parallelly::supportsMulticore()

[HenrikBengtsson]

Hello. You're using

onemapsgapi/R/geocode_onemap.R

Line 49 in 5077c98

if (Sys.info()[["sysname"]] == "Windows") {plan(multisession)} else {plan(multicore)}

without undoing those future plan changes. Note that the user might have already set another plan. See https://future.futureverse.org/articles/future-7-for-package-developers.html#avoid-changing-the-future-backend for how to undo these changes automatically, and why it's important.

Also, you may want to use parallelly::supportsMulticore() instead of looking at Sys.info()[["sysname"]] == "Windows". See https://parallelly.futureverse.org/reference/supportsMulticore.html for details.

In summary, please consider updating to:

#' @importFrom parallelly supportsMulticore
with(if (supportsMulticore()) { plan(multicore) } else { plan(multisession) }, local = TRUE)