You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
The gray longwave component in climt had a tendency which did not define its dims. Since this property is available in the input_properties of the component, one would imagine that this is ok. However,
since the code for output_properties in TendencyStepper does not pass the input properties to combine_component_properties, we get an InvalidPropertyDictError.
This is not a proper fix, because there is still an underlying bug in TendencyStepper. You shouldn't need to specify dims on a tendency when that quantity is present in the inputs.
The gray longwave component in climt had a tendency which did not define its dims. Since this property is available in the input_properties of the component, one would imagine that this is ok. However,
since the code for output_properties in TendencyStepper does not pass the input properties to
combine_component_properties
, we get an InvalidPropertyDictError.This is in the line
sympl/sympl/_core/tendencystepper.py
Line 84 in 3277b7b
This occurs when I try to create a timestepper with gray gas radiation alone.
I have fixed the issue by adding a dims property to the component. However, I wanted to check if this is expected behaviour.
The text was updated successfully, but these errors were encountered: