New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Suppressed performance warning for many small chunks in reorder(...) #79

Merged

rogerkuou merged 4 commits into main from 73_performance_warning_reorder

Jun 25, 2024

Contributor

vanlankveldthijs commented Jun 20, 2024

Resolves #73

Suppressed all warnings while calling xarray.sortby in stm.reorder.
Added a note that only PerformanceWarning should be suppressed, but the type could not be properly determined.
Indicated in reorder function description that this can be a costly operation.

thijsvl added 4 commits

June 20, 2024 15:05


          Suppressed performance warning for many small chunks in reorder(...)

824799c


          Reduced length of single long comment line.

3afe3dd


          Removed unused import from stm.py

8f5a22c


          Sorted stm.py imports.

5fb59fb

rogerkuou approved these changes

View reviewed changes

Member

rogerkuou left a comment

Thx @vanlankveldthijs! Please go ahead and merge this.

rogerkuou merged commit 2475621 into main

16 checks passed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet