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

Index upgrade and probability feature #17

Merged
merged 4 commits into from
Jan 9, 2021

Conversation

MatthewRalston
Copy link
Owner

Fixes the kmerdb index subcommand substantially by fixing the kmerdb.index submodule. This is made use of by the kmerdb probability subcommand, and the new kmerdb.probability submodule, which are implemented in this branch as well.

… what we have right now. This could be a very sticky bugfix with the index issues that we are seeing related to the sparse feature. The first thing to do is to check the log for a sparse first k-mer with the probability function 'kmerdb probability test/data/sample.fa test/data/Cacetobutylicum_ATCC824.8.kdb' and grep if it is in the kdb file. Then we'll see what it has next to it in the kdbi.
…I'll explain in full in comments on the pull request from this issue.
@MatthewRalston MatthewRalston added bug Something isn't working enhancement New feature or request labels Jan 9, 2021
@MatthewRalston MatthewRalston self-assigned this Jan 9, 2021
@MatthewRalston MatthewRalston merged commit 97ba630 into master Jan 9, 2021
@MatthewRalston MatthewRalston deleted the index_upgrade_and_probability_feature branch January 9, 2021 18:07
MatthewRalston added a commit that referenced this pull request Jan 14, 2021
…ity_feature

Index upgrade and probability feature
MatthewRalston added a commit that referenced this pull request May 1, 2022
…ity_feature

Index upgrade and probability feature
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working enhancement New feature or request
Projects
Bugs
  
Awaiting triage
Features v1.0
  
Awaiting triage
Development

Successfully merging this pull request may close these issues.

None yet

1 participant