Add limit_output option to XArray bilinear resampler#701
Add limit_output option to XArray bilinear resampler#701Zaczero wants to merge 1 commit intopytroll:mainfrom
Conversation
| """Initialize xarray bilinear resampler.""" | ||
| super().__init__(source_geo_def, target_geo_def, radius_of_influence, | ||
| neighbours=neighbours, epsilon=epsilon, reduce_data=reduce_data) | ||
| self._limit_output = limit_output |
There was a problem hiding this comment.
I realize that it doesn't seem that the existing kwargs are documented (correct me if I'm wrong @pnuu), but could we get a description of the limit_output keyword argument added? I suppose that means documenting all the kwargs though doesn't it?
There was a problem hiding this comment.
The params are documented in the base class: https://github.com/pytroll/pyresample/blob/main/pyresample/bilinear/_base.py#L55-L70
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #701 +/- ##
==========================================
+ Coverage 93.67% 93.68% +0.01%
==========================================
Files 89 89
Lines 13621 13654 +33
==========================================
+ Hits 12759 12792 +33
Misses 862 862
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
In my case it's dead code, and disabling it reduces runtime by 35%.