Skip to content
This repository has been archived by the owner on Sep 4, 2021. It is now read-only.

Add ability to read *.zip files #56

Open
bmcrae opened this issue Jan 12, 2014 · 2 comments
Open

Add ability to read *.zip files #56

bmcrae opened this issue Jan 12, 2014 · 2 comments

Comments

@bmcrae
Copy link
Contributor

bmcrae commented Jan 12, 2014

It would be nice to read *.zip files in addition to *.gz. The former is likely to be more commonly used by our user base.

@tanmaykm
Copy link
Member

Agree.

There is a difference between .gz and .zip though. While .gz always represents a single file, .zip is actually a container that can contain multiple files. How should we handle cases where we find more than one file in a .zip archive where we expect only one file? We can either take the first entry in the .zip file or throw an error. I think throwing an error is better than picking up the wrong file silently.

@bmcrae
Copy link
Contributor Author

bmcrae commented Jan 14, 2014

Agreed, I think throwing an error if there is anything but a single ascii
grid would be best.
On Jan 13, 2014 9:29 PM, "Tanmay Mohapatra" [email protected]
wrote:

Agree.

There is a difference between .gz and .zip though. While .gz always
represents a single file, .zip is actually a container that can contain
multiple files. How should we handle cases where we find more than one file
in a .zip archive where we expect only one file? We can either take the
first entry in the .zip file or throw an error. I think throwing an error
is better than picking up the wrong file silently.


Reply to this email directly or view it on GitHubhttps://github.com//issues/56#issuecomment-32239840
.

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

No branches or pull requests

2 participants