pytroll / pyresample

Geospatial image resampling in Python
http://pyresample.readthedocs.org
GNU Lesser General Public License v3.0
346 stars 94 forks source link

[Future Spherical Class] Add SExtent #466

Open ghiggi opened 1 year ago

ghiggi commented 1 year ago

This PR adds the SExtent class of the future spherical geometry interface.

codecov[bot] commented 1 year ago

Codecov Report

Merging #466 (b640432) into main (5f53ecd) will increase coverage by 0.02%. The diff coverage is 94.57%.

:exclamation: Current head b640432 differs from pull request most recent head 485bd48. Consider uploading reports for the commit 485bd48 to get more accurate results

@@            Coverage Diff             @@
##             main     #466      +/-   ##
==========================================
+ Coverage   94.27%   94.29%   +0.02%     
==========================================
  Files          69       74       +5     
  Lines       12394    12929     +535     
==========================================
+ Hits        11685    12192     +507     
- Misses        709      737      +28     
Flag Coverage Δ
unittests 94.29% <94.57%> (+0.02%) :arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
pyresample/future/spherical/extent.py 79.28% <79.28%> (ø)
pyresample/future/spherical/__init__.py 100.00% <100.00%> (ø)
pyresample/test/test_sgeom/__init__.py 100.00% <100.00%> (ø)
pyresample/test/test_sgeom/test_extent.py 100.00% <100.00%> (ø)
pyresample/test/test_sgeom/test_shapely.py 100.00% <100.00%> (ø)
pyresample/_multi_proc.py 83.01% <0.00%> (+1.88%) :arrow_up:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

coveralls commented 1 year ago

Coverage Status

Coverage remained the same at 93.823% when pulling b64043266a6e082f63b3cbbdc2f90dd6747a6157 on ghiggi:feature-sextent into 5f53ecdfa56c3d75f79be56770f1871d38873aad on pytroll:main.

ghiggi commented 1 year ago

@djhoese @mraspaud this is ready for review. Tomorrow I have faculty meetings, but I can finalize the last details on Friday if you manage to review it tomorrow. This PR is required for the future SPolygon class PR.

ghiggi commented 1 year ago

Prioritize the review of https://github.com/pytroll/pyresample/pull/465, https://github.com/pytroll/pyresample/pull/473 and https://github.com/pytroll/pyresample/pull/475 first ;) This will only be needed to speed up the SPolygon computations