Grouper's additional dimensions are non-optional #1739
Labels
bug
Something isn't working
help wanted
Extra attention is needed
sdba
Issues concerning the sdba submodule.
Generic Issue
Description
Related to #1736 and discussion #1735. The
Grouper
add_dims
was intended to allow the training of adjustments across dimensions. For example, train a method by mixing all realizations of a simulation. Of course, in such a case, the extra dimensions doesn't exist on the reference data. But the current code expects them to exist in many places.What I Did
What I Received
This specific example fails because:
sdba.utils.map_cdf
passes the group'sdims
without filtering them toapply_ufunc
group.apply('mean', ref)
tries to average the extra dim.Surely, there are more spots where this happens. But fixing those two fixes the issue for LOCI.
Code of Conduct
The text was updated successfully, but these errors were encountered: