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

mash dist with -l: "ERROR: Did not find fasta records" #55

Closed
snayfach opened this issue Aug 13, 2017 · 4 comments
Closed

mash dist with -l: "ERROR: Did not find fasta records" #55

snayfach opened this issue Aug 13, 2017 · 4 comments

Comments

@snayfach
Copy link

I've encountered a possible bug in mash version 1.1.1 using the following command:
mash dist -p 5 sketch.msh -l genome_list.txt

where, sketch.msh is a sketch containing 129k genomes and genome_list.txt contains the path to those same genomes

That command produces the error:
ERROR: Did not find fasta records in "/path/to/fna/1281156.3.fna".

Which is weird because there ARE fasta records in that file. Further, this command works fine:

echo /path/to/fna/1281156.3.fna > genome_path.txt
mash dist -p 5 sketch.msh -l genome_path.txt

as does this one:
mash dist -p 5 sketch.msh /path/to/1281156.3.fna

I thought the issue might be related to the # of threads (-p 5). I tried re-running the first command without specifying the number of threads:
mash dist sketch.msh -l genome_list.txt

this seems to maybe be working? It was still running after about 30 minutes and didn't produce any error or output. The previous commands took only a minute or two to produce an error or output.

I've gotten around this issue for the time being by running mash dist over a loop, but a solution using a single mash command would be preferred. Any ideas?

@ondovb
Copy link
Member

ondovb commented Sep 11, 2017

Looks like the same issue as #41. Fix is in master; release coming soon!

@ondovb
Copy link
Member

ondovb commented Sep 23, 2017

Fixed in v2.0. Please reopen if issues persist.

@ondovb ondovb closed this as completed Sep 23, 2017
@balags1
Copy link

balags1 commented May 23, 2021

This issue seems to persist in Mash v2.3, just noted.

@balags1
Copy link

balags1 commented May 27, 2021

Noted may be in error, my apologies, there was a file format issue.

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

No branches or pull requests

3 participants