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

Kmer Count along a sequence (kmer-plot) #87

Merged
merged 6 commits into from Dec 19, 2014
Merged

Conversation

jgurtowski
Copy link

For a particular input sequence, output the kmer counts from one or more bwt's.

Useful if you want to see the kmer coverage over a specific sequence.

In spirit of Amos kmer-cov-plot

@jts
Copy link
Owner

jts commented Dec 10, 2014

Thanks for this James, it looks great. I'll try to review and merge it in the next day or two.

@jts
Copy link
Owner

jts commented Dec 10, 2014

I didn't notice that the traversal algorithm was removed in the last commit so I've removed the comments specific to this code.

I have some nit-picky formatting comments - the file currently switches between in-line bracing and newline + brace. Can you standardize this? Probably best to use newline + brace since this is what the rest of SGA uses and I'd like to be consistent. Thanks!

@jgurtowski
Copy link
Author

Kmer-count now can take a sequence file (kmer-plot functionality has been merged).

jts added a commit that referenced this pull request Dec 19, 2014
Kmer Count along a sequence (kmer-plot)
@jts jts merged commit dd74001 into jts:master Dec 19, 2014
@jts
Copy link
Owner

jts commented Dec 19, 2014

Merged, thanks James!

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

Successfully merging this pull request may close these issues.

None yet

2 participants