Skip to content
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

Swap out fishpond for eds #772

Merged
merged 5 commits into from
Jun 6, 2024
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion .github/workflows/build-docker.yml
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
name: Build Docker Image and Push to Dockerhub
name: Build Docker Image

# Every time something is merged to master or a tag is pushed, we build and push
# the Docker image
Expand Down
10 changes: 8 additions & 2 deletions Dockerfile
Original file line number Diff line number Diff line change
Expand Up @@ -68,12 +68,18 @@ RUN DEBIAN_FRONTEND=noninteractive apt-get install -y --no-install-recommends \
COPY requirements.txt requirements.txt
RUN pip install -r requirements.txt



# Use renv for R packages
WORKDIR /usr/local/renv
ENV RENV_CONFIG_CACHE_ENABLED=FALSE
RUN Rscript -e "install.packages('renv')"

# Temporary fix for broken(?) RSamtools package
RUN Rscript -e "install.packages('BiocManager'); BiocManager::install('Rsamtools')"
Comment on lines +78 to +79
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ooof what happened to RSamtools?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm not sure... something in an interaction between the Bioc docker image, renv and Rsamtools means that the package it downloads is corrupted and fails to untar properly. Ally ran into it in her Docker image a week or so ago. I'm expecting it to be resolved at some point when things rebuild somewhere.


COPY renv.lock renv.lock
RUN R -e "install.packages('renv')"
RUN R -e "renv::restore()" \
RUN Rscript -e "renv::restore()" \
rm -rf ~/.cache/R/renv && \
rm -rf /tmp/downloaded_packages && \
rm -rf /tmp/Rtmp*
Expand Down
2 changes: 1 addition & 1 deletion components/dependencies.R
Original file line number Diff line number Diff line change
Expand Up @@ -7,7 +7,7 @@ library(markdown)
library(magick)

# Recommended by tximport
library(fishpond)
library(eds)

# Required for DESeq2::lfcShrink()
library(apeglm)
Expand Down
50 changes: 11 additions & 39 deletions renv.lock
Original file line number Diff line number Diff line change
Expand Up @@ -2378,6 +2378,17 @@
],
"Hash": "20783ecb7e7cea3f351e8b21153e3eb8"
},
"eds": {
"Package": "eds",
"Version": "1.6.0",
"Source": "Bioconductor",
"Repository": "Bioconductor 3.19",
"Requirements": [
"Matrix",
"Rcpp"
],
"Hash": "ab19d02b3418e44d6f32ffb8060427d2"
},
"ellipsis": {
"Package": "ellipsis",
"Version": "0.3.2",
Expand Down Expand Up @@ -2631,31 +2642,6 @@
],
"Hash": "192053c276525c8495ccfd523aa8f2d1"
},
"fishpond": {
"Package": "fishpond",
"Version": "2.10.0",
"Source": "Bioconductor",
"Repository": "Bioconductor 3.19",
"Requirements": [
"GenomicRanges",
"IRanges",
"Matrix",
"S4Vectors",
"SingleCellExperiment",
"SummarizedExperiment",
"abind",
"graphics",
"gtools",
"jsonlite",
"matrixStats",
"methods",
"qvalue",
"stats",
"svMisc",
"utils"
],
"Hash": "9091a296b5cdb86da7c4c35f958c67f6"
},
"flexmix": {
"Package": "flexmix",
"Version": "2.3-19",
Expand Down Expand Up @@ -5008,20 +4994,6 @@
],
"Hash": "184d7799bca4ba8c3be72ea396f4b9a3"
},
"svMisc": {
"Package": "svMisc",
"Version": "1.2.3",
"Source": "Repository",
"Repository": "CRAN",
"Requirements": [
"R",
"methods",
"stats",
"tools",
"utils"
],
"Hash": "b13f5680860f67c32ccb006ad38f24f4"
},
"sys": {
"Package": "sys",
"Version": "3.4.2",
Expand Down
Loading