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
most_recent_beliefs_only=True seems like a sensible default for Sensor.search_beliefs. I'd propose to keep most_recent_beliefs_only=False for TimedBelief.search.
I like our starting point (from timely beliefs) in that the defaults for all search parameters have the same connotation: "If you don't set any, you get everything". Here we're discussing to deviate from that logic to: "If you don't set any, you get the most recent beliefs about all events". I guess there's something to be said for keeping the defaults of both (FlexMeasures) functions in sync.
most_recent_beliefs_only=True
seems like a sensible default forSensor.search_beliefs
. I'd propose to keepmost_recent_beliefs_only=False
forTimedBelief.search
.(Inspired by discussion in #307.)
The text was updated successfully, but these errors were encountered: