R reports unset environment variables as empty string

bbimber · bbimber · commit 3bd837a0de47 · 2021-12-30T15:12:05.000-08:00
diff --git a/singlecell/resources/chunks/Functions.R b/singlecell/resources/chunks/Functions.R
@@ -93,7 +93,7 @@ options(future.globals.maxSize = Inf)
 
 options('Seurat.memsafe' = TRUE)
 
-if (!is.null(Sys.getenv('SEURAT_MAX_THREADS'))) {
+if (Sys.getenv('SEURAT_MAX_THREADS') != '') {
     print(paste0('Setting future::plan workers to: ', Sys.getenv('SEURAT_MAX_THREADS')))
-    future::plan(strategy='multiprocess', workers=Sys.getenv('SEURAT_MAX_THREADS'))
+    future::plan(strategy='multisession', workers=Sys.getenv('SEURAT_MAX_THREADS'))
 }

Original file line number	Diff line number	Diff line change
`@@ -93,7 +93,7 @@ options(future.globals.maxSize = Inf)`
`93`	`93`
`94`	`94`	`options('Seurat.memsafe' = TRUE)`
`95`	`95`
`96`		`-if (!is.null(Sys.getenv('SEURAT_MAX_THREADS'))) {`
	`96`	`+if (Sys.getenv('SEURAT_MAX_THREADS') != '') {`
`97`	`97`	`print(paste0('Setting future::plan workers to: ', Sys.getenv('SEURAT_MAX_THREADS')))`
`98`		`- future::plan(strategy='multiprocess', workers=Sys.getenv('SEURAT_MAX_THREADS'))`
	`98`	`+ future::plan(strategy='multisession', workers=Sys.getenv('SEURAT_MAX_THREADS'))`
`99`	`99`	`}`