StaPH-B / docker-builds

:package: :whale: Dockerfiles and documentation on tools for public health bioinformatics
GNU General Public License v3.0
182 stars 116 forks source link

adding parsnp 2.0.5 #980

Closed erinyoung closed 3 months ago

erinyoung commented 3 months ago

There's a new version of parsnp!

According to the release notes:

Parsnp now outputs a MAF file in addition to the typical XMFA file (can turn this off with (--no-maf). Parsnp now uses the partition mode by default when there are more than 100 input sequences.

All I did was copy the Dockerfile from version 2.0.4 and update the software arg.

The full diff:

$ diff parsnp/2*/Dockerfile
1c1
< ARG PARSNP_VER="2.0.4"
---
> ARG PARSNP_VER="2.0.5"

Pull Request (PR) checklist:

Kincekara commented 3 months ago

@erinyoung I realized that parsnp searches FastTreeMP which can improve overall performance. Have you tried to install FastTreeMP instead of Fasttree?

#26 72.27 19:59:06 - INFO - FastTreeMP not found. Trying fasttree...
#26 72.27 19:59:06 - DEBUG - fasttree -nt -quote -gamma -slow -boot 100 outdir_parsnp_fasttree/parsnp.snps.mblocks
erinyoung commented 3 months ago

I'm going to pass installing FastTreeMp (http://www.microbesonline.org/fasttree/#OpenMP) because it involves installing fasttree from source instead of just downloading the executable.

Kincekara commented 3 months ago

@erinyoung precompiled executable is available here: http://www.microbesonline.org/fasttree/FastTreeMP I am merging this now. We can check FastTreeMP in the next iteration.

Kincekara commented 3 months ago

Thank you for the update! Deploy link: https://github.com/StaPH-B/docker-builds/actions/runs/9191921039.