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

Can't use k of 39 (must be <= 31) #160

Open
ailtonpcf opened this issue Jan 9, 2024 · 1 comment
Open

Can't use k of 39 (must be <= 31) #160

ailtonpcf opened this issue Jan 9, 2024 · 1 comment

Comments

@ailtonpcf
Copy link

Hey,
I would like to try Krakenuniq with kmers longer than 31, however I get an error:
Can't use k of 39 (must be <= 31)

Is that really true that I can't use bigger kmer values, or a bug?

Thanks in advance,
Ailton.

@salzberg
Copy link
Collaborator

salzberg commented Jan 9, 2024

yes that's correct - Krakenuniq uses an array that can only hold values up to 2^31 and there are places in the code that would break if you used longer k-mers. (And yes, I know that there are 4^31 possible DNA sequences of length 31.) Changing this would require a major code rewrite. Btw with longer k-mers you lose sensitivity rather quickly.

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

No branches or pull requests

2 participants