czbiohub-sf / nf-predictorthologs

*de novo* orthologous gene predictions from bam + bed or fasta/fastq data
MIT License
4 stars 2 forks source link

Add option to use sourmash to search instead of diamond #35

Closed olgabot closed 4 years ago

olgabot commented 4 years ago

don't convert between hashes <> kmers <> seqs, but instead search for hashes directly

PR checklist

Learn more about contributing: CONTRIBUTING.md

pranathivemuri commented 4 years ago

seems like tests are getting canceled - https://github.com/czbiohub/nf-predictorthologs/pull/35/checks?check_run_id=600390407#step:5:37

olgabot commented 4 years ago

I am completely unable to test this on my home machine due to a bizarre docker/nextflow issue of directories not being mounted (nextflow google group message).. I have no idea why it's happening. But if all the tests are working here, including test_diff_hash_sourmash then I'm happy to merge!