Default to 'serializedSize' [#736]

HenrikBengtsson · Jul 28, 2024 · 21fe894 · 21fe894
1 parent af7055d
commit 21fe894
Show file tree

Hide file tree

Showing 3 changed files with 2 additions and 8 deletions.
diff --git a/DESCRIPTION b/DESCRIPTION
@@ -1,5 +1,5 @@
 Package: future
-Version: 1.33.2-9010
+Version: 1.33.2-9011
 Title: Unified Parallel and Distributed Processing in R for Everyone
 Imports:
     digest,

diff --git a/R/options.R b/R/options.R
@@ -361,11 +361,5 @@ update_package_options <- function(debug = FALSE) {
 
   ## future 1.34.0:
   update_package_option("future.globals.objectSize.method", mode = "character", debug = debug)
-  if (is.null(getOption("future.globals.objectSize.method"))) {
-   if (exists("serializedSize", envir = getNamespace("parallelly"), inherits = FALSE)) {
-      options(future.globals.objectSize.method = "serializedSize")
-    }
-  }
-
   update_package_option("future.plan.cleanup.legacy", mode = "logical", debug = debug)
 }
diff --git a/R/utils-objectSize.R b/R/utils-objectSize.R
@@ -7,7 +7,7 @@ objectSize <- function(x, depth = 3L, enclosure = getOption("future.globals.obje
   if (isNamespace(x)) return(0)
   if (depth <= 0) return(0)
 
-  method <- getOption("future.globals.objectSize.method", "objectSize")
+  method <- getOption("future.globals.objectSize.method", "serializedSize")
   if (method == "serializedSize") {
     size <- serializedSize(x)
     return(size)