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

Add 2 packages to existing container #3249

Merged
merged 11 commits into from
Aug 27, 2024

Conversation

LaurenceKuhl
Copy link
Contributor

No description provided.

@@ -603,4 +603,4 @@ lumpy-sv=0.3.1 quay.io/bioconda/base-glibc-busybox-bash:3.1 3
bwameth=0.2.7,bwa-mem2=2.2.1
pasta=1.9.0,pigz=2.8
longphase=1.7.3,htslib=1.20
r-base,r-igraph,r-dplyr
r-base,r-igraph,r-dplyr,r-tidyr,r-readr,r-ggplot2
Copy link
Member

Choose a reason for hiding this comment

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

Please add versions

combinations/hash.tsv Outdated Show resolved Hide resolved
combinations/hash.tsv Outdated Show resolved Hide resolved
@LaurenceKuhl
Copy link
Contributor Author

hi sorry asked for a review too early :(

Co-authored-by: LaurenceKuhl <laurence.kuhlburger@gmail.com>
@bgruening
Copy link
Member

No worries, but there are missing still a few versions.

combinations/hash.tsv Outdated Show resolved Hide resolved
@LaurenceKuhl
Copy link
Contributor Author

hi @bgruening i'm trying to figure out the versions so create locally a conda environment with all packages i needed and then manually checked which versions of my R packages were installed there and then appended them to this commit, so i'm not sure why it's failing in the container? Is there any other way for me to figure out matching versions?
Thanks a lot for any link towards any doc in case i missed something :)

@bgruening bgruening merged commit 3608e6b into BioContainers:master Aug 27, 2024
1 check passed
@bgruening
Copy link
Member

I did the same than you did conda create -n gasdf --dry-run r-base r-igraph r-dplyr r-tidyr r-readr r-ggplot2 and failed to write down the correct deps. As soon as I was able to write down what my eyes where seeing it worked :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants