-
Notifications
You must be signed in to change notification settings - Fork 32
Open
Description
I understand that this isn't the point of this PR but it illuminated to me that we have some weird logic in the
plant_config. I vote that we potentially make an issue about it.
I think it's ambiguous that it says "tech_to_dispatch_connections" but excludes the open-loop controllers. I vote either we update this name to be more clear that this isn't used for open loop dispatch or we add all dispatch to the "tech_to_dispatch_connections". We also need to spell this out in documentation. It's currently not included.
Originally posted by @kbrunik in #615 (comment)
I also think that this is partially related to the differences between the Heuristic Pyomo controller and the Optimized Pyomo controller.
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
No labels