Skip to content

Commit

Permalink
change: comment out optional pkgs message
Browse files Browse the repository at this point in the history
  • Loading branch information
jiajic committed May 24, 2024
1 parent d618598 commit eef5283
Showing 1 changed file with 24 additions and 24 deletions.
48 changes: 24 additions & 24 deletions R/create.R
Original file line number Diff line number Diff line change
Expand Up @@ -152,30 +152,30 @@ createGiottoObject <- function(expression,
## data.table vars
cell_ID <- feat_ID <- NULL

## check if all optional packages are installed
# TODO: update at the end
# TODO: extract from suggest field of DESCRIPTION
extra_packages <- c(
"scran", "MAST", "png", "tiff", "biomaRt",
"trendsceek", "multinet", "RTriangle", "FactoMineR"
)

pack_index <- extra_packages %in% rownames(utils::installed.packages())
extra_installed_packages <- extra_packages[pack_index]
extra_not_installed_packages <- extra_packages[!pack_index]

if (any(pack_index == FALSE) == TRUE) {
wrap_msg(
"Consider to install these (optional) packages to run all possible",
"Giotto commands for spatial analyses: ",
extra_not_installed_packages
)
wrap_msg(
"Giotto does not automatically install all these packages as they",
"are not absolutely required and this reduces the number of
dependencies"
)
}
# ## check if all optional packages are installed
# # TODO: update at the end
# # TODO: extract from suggest field of DESCRIPTION
# extra_packages <- c(
# "scran", "MAST", "png", "tiff", "biomaRt",
# "trendsceek", "multinet", "RTriangle", "FactoMineR"
# )
#
# pack_index <- extra_packages %in% rownames(utils::installed.packages())
# extra_installed_packages <- extra_packages[pack_index]
# extra_not_installed_packages <- extra_packages[!pack_index]
#
# if (any(pack_index == FALSE) == TRUE) {
# wrap_msg(
# "Consider to install these (optional) packages to run all possible",
# "Giotto commands for spatial analyses: ",
# extra_not_installed_packages
# )
# wrap_msg(
# "Giotto does not automatically install all these packages as they",
# "are not absolutely required and this reduces the number of
# dependencies"
# )
# }


## if cores is not set, then set number of cores automatically, but with
Expand Down

0 comments on commit eef5283

Please sign in to comment.