-
Notifications
You must be signed in to change notification settings - Fork 294
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
.n_occupied() was implemented differently in hashbits and hashtable #562
Milestone
Comments
Thanks @qingpeng |
Any progress? |
Upon further review I see that we only count unique k-mers in hashbits. n_occupied merely tells you the number of bins occupied in the first table. I am going to implement a CountingHash.n_kmers() function and update the scripts to use it instead of n_occupied(). |
This was referenced Nov 6, 2014
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
as discussed in #429 and #491
Also,in load-graph.py, '--report-total-kmers' option will print .n_occupied() as "Total number of k-mers".
In .info file, .n_unique_kmers() is printed as number of "unique k-mers".
This is confusing. I will look into this.
The text was updated successfully, but these errors were encountered: