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

Pin Samtools version to 1.7 #354

Merged
merged 2 commits into from
Aug 12, 2020
Merged

Pin Samtools version to 1.7 #354

merged 2 commits into from
Aug 12, 2020

Conversation

hackdna
Copy link
Contributor

@hackdna hackdna commented Aug 4, 2020

To fix bcbio/bcbio-nextgen#3320 and toward bcbio/bcbio-nextgen#3318

I performed a clean bcbio install using the latest master and ended up with the following working configuration:

vagrant@bcbio:~$ which samtools
/home/vagrant/local/share/bcbio/anaconda/bin/samtools
vagrant@bcbio:~$ samtools --version
samtools 1.7
Using htslib 1.7
Copyright (C) 2018 Genome Research Ltd.
vagrant@bcbio:~$ conda list openssl
# packages in environment at /home/vagrant/local/share/bcbio/anaconda:
#
# Name                    Version                   Build  Channel
openssl                   1.1.1g               h516909a_1    conda-forge
pyopenssl                 19.1.0                   py36_0    conda-forge
r-openssl                 1.4.1             r35he5c4762_0    conda-forge
vagrant@bcbio:~$ conda list samtools
# packages in environment at /home/vagrant/local/share/bcbio/anaconda:
#
# Name                    Version                   Build  Channel
bioconductor-rsamtools    1.34.0           r351hf484d3e_0    bioconda
perl-bio-samtools         1.43            pl526h1341992_1    bioconda
samtools                  1.7                           1    bioconda

This update will ensure this configuration is reproducible. Tested using: bcbio_nextgen.py upgrade -u development --tools

@roryk
Copy link
Collaborator

roryk commented Aug 6, 2020

Thanks-- does getting 1.9 not work? 1.7 is super old.

@hackdna
Copy link
Contributor Author

hackdna commented Aug 6, 2020

Sure, this is just a quick fix to help folks with broken bcbio installations. According to bioconda/bioconda-recipes#12100, samtools 1.9 had issues since 2018. I will test more recent versions and report back.

@hackdna
Copy link
Contributor Author

hackdna commented Aug 6, 2020

Tested successfully by bcbio_nextgen.py upgrade -u development --tools with samtools=1.10 in tmpbcbio-install/cloudbiolinux/contrib/flavor/ngs_pipeline_minimal/packages-conda.yaml:

vagrant@bcbio:~$ which samtools
/home/vagrant/local/share/bcbio/anaconda/bin/samtools
vagrant@bcbio:~$ samtools --version
samtools 1.10
Using htslib 1.10.2
Copyright (C) 2019 Genome Research Ltd.
vagrant@bcbio:~$ conda list openssl
# packages in environment at /home/vagrant/local/share/bcbio/anaconda:
#
# Name                    Version                   Build  Channel
openssl                   1.1.1g               h516909a_1    conda-forge
pyopenssl                 19.1.0                   py36_0    conda-forge
r-openssl                 1.4.1             r35he5c4762_0    conda-forge

Installing from scratch may be a different story because by default mamba/conda resolves samtools to version 1.7. However, I don't know an easy way to test that because bcbio install always overwrites tmpbcbio-install/cloudbiolinux with a version from GitHub.

@roryk
Copy link
Collaborator

roryk commented Aug 6, 2020

Heya, you can skip the cloudbiolinux update by linking tmpbcbio-install/cloudbiolinux to your local one.

@hackdna
Copy link
Contributor Author

hackdna commented Aug 6, 2020

I did but this directory got overwritten with cloudbiolinux from GitHub when running bcbio_nextgen_install.py.

@roryk
Copy link
Collaborator

roryk commented Aug 12, 2020

Great, can you just push the 1.10 fix and we can see if it works, we can always revert it.

@roryk roryk merged commit 9e2ef24 into chapmanb:master Aug 12, 2020
@roryk
Copy link
Collaborator

roryk commented Aug 12, 2020

Thanks Ilya!

@hackdna hackdna deleted the patch-2 branch August 12, 2020 18:22
@hackdna
Copy link
Contributor Author

hackdna commented Aug 12, 2020

It worked for me:

vagrant@bcbio:~$ which samtools
/home/vagrant/local/share/bcbio/anaconda/bin/samtools
vagrant@bcbio:~$ samtools --version
samtools 1.10
Using htslib 1.10.2
Copyright (C) 2019 Genome Research Ltd.
vagrant@bcbio:~$ conda list openssl
# packages in environment at /home/vagrant/local/share/bcbio/anaconda:
#
# Name                    Version                   Build  Channel
openssl                   1.1.1g               h516909a_1    conda-forge
pyopenssl                 19.1.0                   py36_0    conda-forge
r-openssl                 1.4.1             r35he5c4762_0    conda-forge

@roryk
Copy link
Collaborator

roryk commented Aug 12, 2020

Awesome, thanks so much. Worked for me too.

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.

Problem installing hg38
2 participants