Skip to content

more efficient owned records, and use a fork of flate2 that supports mutli-part gzip #1

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

Closed
wants to merge 4 commits into from

Conversation

pmarks
Copy link
Contributor

@pmarks pmarks commented Mar 7, 2017

thanks for the great lib!

FYI the required change in flate2 has been merged but not released:
rust-lang/flate2-rs#43

It may just be worth waiting for the updated version of flate2, but wanted to make you aware of it.

@aseyboldt
Copy link
Owner

Thanks for the pull request.
The MultiIGz change look good, but as you said I'd rather wait for a release of flate2. Do you know anything about when that might happen? Seems this has been sitting there for a month without change.
About the OwnedRecord: The original idea behind using individual vecs, was that this makes it possible to modify the (length of) sequences easily – for example when trimming. But thinking about it now, I guess this could still be achieved with this faster version. Just out of interest, what are you using the OwnedRecord for?

@pmarks
Copy link
Contributor Author

pmarks commented Jun 17, 2017

@aseyboldt - want to take a look at these updates?

  • switched over to release flate2 to get the MultiGzDecoder
  • use rust-bio instead of parasailors for examples -- parasailors doesn't currently build on my mac. safer to use a pure Rust library.

@aseyboldt
Copy link
Owner

Sorry for not replying for so long.
Thanks for the PR, the changes to Buffer.clean and MulitGzDecoder look good, but I'd rather keep the current OwnedRecord. It makes it possible to modify the record, and other than that I can't think of use cases for the owned record anyway (I might be missing something of course).
Also, lz4 is important, a fast parser doesn't help much with gzip :-)
I also prefer parasailors to rust-bio for alignments, it is way faster. It compiles fine for me, do you still have issues with it?

@pmarks pmarks closed this Apr 26, 2018
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.

2 participants