NCAS-CMS / cf-python

A CF-compliant Earth Science data analysis library
http://ncas-cms.github.io/cf-python
MIT License
119 stars 19 forks source link

`cf.aggregate` sometimes needlessly puts in a "transpose" layer into the Dask graph #754

Closed davidhassell closed 5 months ago

davidhassell commented 5 months ago

In v3.16.2, cf.aggregate sometimes needlessly puts in a "transpose" layer into the Dask graph that is a null operation, i.e. it doesn't change the axis order. This can have performance implications (it is slower to inspect and optimise a Dask graph with more layers), and prevents the possibility of active storage operations (see #501).

PR to follow