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

python bitshuffle/bitshuffle.py < file should work #55

Open
jyn514 opened this issue Mar 2, 2018 · 1 comment
Open

python bitshuffle/bitshuffle.py < file should work #55

jyn514 opened this issue Mar 2, 2018 · 1 comment
Labels
easy enhancement improvement to existing feature good first issue waiting on Needs different issue to be closed first

Comments

@jyn514
Copy link
Collaborator

jyn514 commented Mar 2, 2018

Would involve changing infer_mode. Should be fairly easy to change.

Jeremy:

I expected python bitshuffle/bitshuffle.py < file to [encode the input file]
if I put in an encoded file I would expect it to decode

Me:

for that we'd have to look through the file before we decode it
grep ((<<.*>>)) isn't terribly hard though

@jyn514 jyn514 added enhancement improvement to existing feature good first issue easy labels Mar 2, 2018
@jyn514 jyn514 self-assigned this Mar 2, 2018
@jyn514
Copy link
Collaborator Author

jyn514 commented Mar 4, 2018

Looking at this again, it's easy to to always encode but significantly harder to infer (since input may not always be grepable. Think I'll wait on smart inferrence until we handle #16 and #54

@jyn514 jyn514 added the waiting on Needs different issue to be closed first label Mar 4, 2018
@jyn514 jyn514 removed their assignment Jul 13, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
easy enhancement improvement to existing feature good first issue waiting on Needs different issue to be closed first
Projects
None yet
Development

No branches or pull requests

1 participant