roocs / clisops

Climate Simulation Operations
https://clisops.readthedocs.io/en/latest/
Other
21 stars 9 forks source link

define the "inner_mask" for locstream case #292

Closed vindelico closed 1 year ago

vindelico commented 1 year ago

Define the inner_mask in subset_shape() for the locstream case. subset_shape() would return all lat/lon inside inner_mask which is equivalent to the bounding box of the polygon, not the area inside the polygon.

Pull Request Checklist:

coveralls commented 1 year ago

Pull Request Test Coverage Report for Build 5662181592


Totals Coverage Status
Change from base Build 5579360884: 0.06%
Covered Lines: 1111
Relevant Lines: 1415

💛 - Coveralls
Zeitsperre commented 1 year ago

@vindelico Do you have any free time to work on/finalize this PR? I have use cases that could benefit from your changes!

vindelico commented 1 year ago

@vindelico Do you have any free time to work on/finalize this PR? I have use cases that could benefit from your changes!

Can you check HISTORY.rst and AUTHORS.rst, plz? - otherwise I don't have additional changes.