search

KipCrossing / geotiff

A noGDAL tool for reading and writing geotiff files
GNU Lesser General Public License v2.1
213 stars 23 forks source link

Update requirements.txt #50

Closed wlbksy closed 2 years ago

wlbksy commented 2 years ago

fix requirements

Zeitsperre commented 2 years ago

@wlbksy

This dependency is not currently needed for geotiff; Do you have plans to integrate functionality based on it here? I'm not sure what we would gain from this.

Also, I'm looking at imagecodecs, and it seems like it's a project in an alpha state and that the newer versions have dropped Python3.7. We would also need to drop that version of Python 3.7 on this PR.

wlbksy commented 2 years ago

Currently I can't acquire band data without this dependency using geotiff package. It would complain lacking imagecodecs on my macOS miniconda. I've no idea why this is dependent, but installing this fixed the command line complaining.

KipCrossing commented 2 years ago

It looks like the tifffile package wants the user to take care of installing imagecodecs

see here

So perhaps it's now up to us to ensure this via this PR. I'll look into it a bit more.

KipCrossing commented 2 years ago

Look's like the import bug was introduced to tifffile in version 2022.4.28 where pretty much all the import attempts fail. We can restrict the version of tifffile to an earlier version

KipCrossing commented 2 years ago

Closed by #51