-
Notifications
You must be signed in to change notification settings - Fork 2
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
Incorporate the high-memory BWT implementation from fmlrc2 #9
Comments
It's no bother, though I am curious as to why the sudden interest in the crate. Anyways, incorporating the exact implementation is likely tricky in the crate's current state; the main issue being that k-mer caching isn't really built into
That's the high level process I would use. I just haven't had a need for it yet. |
It's not so sudden interest for me :) This crate come to my interest in the last 4 months. I have tens of long and short reads transcriptomes and genomes to analyze and correct. It is really useful to have to type only one command to build the mswbt compared to the old version which is OS-dependent and more tedious in my opinion. I had some free hours since yesterday so I tried to invest in the crate for my future works and hope to have at least help a bit. I don't know if I will have much free time in the coming days but we'll see how things are going. For the incorporation of the high-memory BWT, I'll try to do my best when I can. Thanks. P.S. I'm also the person behind the Rust for Bioinformatics twitter profile and the community love your works! |
Yea, that was initially the intent was to make building a bit more user friendly; granted it's also not as efficient as the As for the high-mem impl., I'm happy to help review/iterate on that if you do decide to work on it! Also, good to know the person behind the twitter handle haha |
Hi @holtjma,
Sorry to bother you, and sorry if it is more issues and PR from me than from anyone in the recent days.
I believe this tool will greatly benefit from this addition.
I would like to try to see if we can incorporate the high-memory BWT implementation from fmlrc2 here. Before starting I prefer to have your recommendations and direction so that it will not be a waste of time for me and you. And if it is not something you would like for mswbt2, we can just leave it as it is.
Thanks!
The text was updated successfully, but these errors were encountered: