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

accept lowercase bases #8

Closed
JWDebler opened this issue Jun 16, 2021 · 3 comments
Closed

accept lowercase bases #8

JWDebler opened this issue Jun 16, 2021 · 3 comments
Labels
enhancement New feature or request

Comments

@JWDebler
Copy link

Hi there,
I just had rust-mdbg crashing on me with this error:

Non-ACGTN nucleotide encountered!Non-ACGTN nucleotide encountered!

Since CANU had no problems with my intput files I was surprised.
Took me a while to figure out that the problem was that the bases in my fastq files were in lowercase.

Any chance you could allow lowercase bases as input?
Cheers

@rchikhi
Copy link
Collaborator

rchikhi commented Jun 16, 2021

hi, yes we'll fix the parser eventually, for now you can prepare your input using: seqtk seq -AU [yourfile.fasta/yourfile.fastq.gz]

@rchikhi rchikhi closed this as completed Jun 16, 2021
@ekimb ekimb added the enhancement New feature or request label Jun 16, 2021
@luizirber
Copy link

I added an issue to nthash to discuss how to solve this without needing to preprocess the input (and maybe avoid having to uppercase the sequence?)

@rchikhi
Copy link
Collaborator

rchikhi commented Jun 16, 2021

good idea, thanks!

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

No branches or pull requests

4 participants