Circuitscape.py icon indicating copy to clipboard operation
Circuitscape.py copied to clipboard

Add ability to read *.zip files

Open bmcrae opened this issue 11 years ago • 2 comments

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.

bmcrae avatar Jan 12 '14 03:01 bmcrae

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.

tanmaykm avatar Jan 14 '14 05:01 tanmaykm

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/Circuitscape/Circuitscape/issues/56#issuecomment-32239840 .

bmcrae avatar Jan 14 '14 06:01 bmcrae