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

Add options for outputting gzipped/bzip2ed sequence #505

Closed
ctb opened this issue Jun 19, 2014 · 8 comments
Closed

Add options for outputting gzipped/bzip2ed sequence #505

ctb opened this issue Jun 19, 2014 · 8 comments
Milestone

Comments

@ctb
Copy link
Member

ctb commented Jun 19, 2014

It'd be nice if normalize-by-median etc could compress files on output.

@mr-c
Copy link
Contributor

mr-c commented Jun 20, 2014

Do you see this being a screed feature or a khmer/oxli feature?

@ctb
Copy link
Member Author

ctb commented Jun 22, 2014

On Fri, Jun 20, 2014 at 11:52:27AM -0700, Michael R. Crusoe wrote:

Do you see this being a screed feature or a khmer/oxli feature?

Both, I think.

@mr-c mr-c added this to the unscheduled milestone Sep 1, 2014
@mr-c mr-c modified the milestones: 1.3+, unscheduled Jan 16, 2015
@mr-c mr-c self-assigned this Jan 16, 2015
@mr-c
Copy link
Contributor

mr-c commented Jan 16, 2015

Huh, I can't unassign myself. @b-wyss has claimed this

@mr-c mr-c assigned b-wyss and unassigned mr-c Jan 20, 2015
@mr-c
Copy link
Contributor

mr-c commented Jan 27, 2015

Must be done with a switch to not change existing behavior

@bocajnotnef
Copy link
Contributor

re lessened animosity towards breaking API: Do we still need a switch?

@mr-c
Copy link
Contributor

mr-c commented May 16, 2015

We need sane defaults and override switches.

If we generate the filename then use the same compression as the
corresponding input, naming the output accordingly.

If the user specifies the filename then infer the compression type that
way.

Support --no-compression and --bzip2, --gzip to override the above.

On Sat, May 16, 2015 at 11:20 AM Jake Fenton notifications@github.com
wrote:

re lessened animosity towards breaking API: Do we still need a switch?


Reply to this email directly or view it on GitHub
#505 (comment).

@ctb
Copy link
Member Author

ctb commented May 17, 2015 via email

@bocajnotnef
Copy link
Contributor

Added in #747, closing #505

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants