GeoscienceAustralia / PyRate

A Python tool for estimating velocity and time-series from Interferometric Synthetic Aperture Radar (InSAR) data.
https://geoscienceaustralia.github.io/PyRate/
Apache License 2.0
200 stars 70 forks source link

Remove Pillow dependency #296

Closed mcgarth closed 4 years ago

mcgarth commented 4 years ago

Pillow is a required python dependency in PyRate but is only used in one function pyrate.core.gdal_pyrate.crop, which itself is not used in the main workflow. It is only used as a comparison tool in a unit test.

glob2 is a requirement but not used anywhere.

I propose we remove these package dependencies and the functions/tests that require them

codecov-commenter commented 4 years ago

Codecov Report

Merging #296 into develop will decrease coverage by 0.03%. The diff coverage is n/a.

Impacted file tree graph

@@             Coverage Diff             @@
##           develop     #296      +/-   ##
===========================================
- Coverage    83.16%   83.13%   -0.04%     
===========================================
  Files           26       26              
  Lines         3498     3462      -36     
  Branches       547      543       -4     
===========================================
- Hits          2909     2878      -31     
+ Misses         491      489       -2     
+ Partials        98       95       -3     
Impacted Files Coverage Δ
pyrate/core/gdal_python.py 84.89% <ø> (-0.26%) :arrow_down:

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update cf679ee...30d6882. Read the comment docs.