-
-
Notifications
You must be signed in to change notification settings - Fork 1
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Installation instructions #54
Comments
let's also follow the approach of the bioc-project and bioc-rnaseq modules, and refer for platform-specific R and RStudio installation instructions to the installation instructions provided on the landing page of the bioc-intro module as opposed to having a copy of them on here. |
|
For the end user using "satijalab/seurat-data" should be fine and likely better to use here, but note that there have been issues in the past with repo string ("seurat-data") not matching the package name ("SeuratData") in GHA installations as part of the github page build. "lgeistlinger/SeuratData" circumvents this problem.
If we use personal setups for the course then it needs to be communicated that the latest release version of R, Bioconductor, and individual packages should be used. Otherwise instantiated cloud setups as in workshop.bioconductor.org serve such instances to the participants which should minimize such issues. |
Okay, I addressed these items in #55 |
For the installation instructions under https://ccb-hms.github.io/osca-workbench/, consider the following:
BiocManager::install
command can be condensed based on the dependency structure between packages. For example, the following packages are dependencies ofscater
and are thus installed at the timescater
is installed and thus do not need to be listed here:SingleCellExperiment
,ggplot2
,scuttle
,BiocNeighbors
,BiocParallel
,BiocSingular
,pheatmap
.SeuratData
is only available from github, change tolgeistlinger/SeuratData
Ncpus = 4
argument in the recommended/default installation command seems problematic as many users in less privileged parts of the world might stumble over this one. My suggestion would be to provide the installation command without theNcpus
argument and have a note below that points out that the installation process can be expedited on multi-core machines with theNcpus
argument.The text was updated successfully, but these errors were encountered: