We discussed this in a code standup on 6/9/26.
Original comment from @jaredthomas68: "I thought we had decided to have the SLC take in demand and output set_point. The tech control would take in set_point and output control_command or something like that." #751 (comment)_
We landed on:
- SLC takes in demand and outputs set-points
- tech-level controllers take in set-points and output command_values
We can and should update these names to be more in-line with conventional control terminology.
Changing these variables in H2I is more complex than a simple find-and-replace due to subtleties with what the variables mean in different contexts.
Some models may need updating to include command_value.
I spent a few hours working through these changes but hit many speed bumps.
Because we've been waiting on SLC capabilities to come in, I've opened this issue for future work.
We discussed this in a code standup on 6/9/26.
Original comment from @jaredthomas68: "I thought we had decided to have the SLC take in
demandand outputset_point. The tech control would take inset_pointand outputcontrol_commandor something like that." #751 (comment)_We landed on:
We can and should update these names to be more in-line with conventional control terminology.
Changing these variables in H2I is more complex than a simple find-and-replace due to subtleties with what the variables mean in different contexts.
Some models may need updating to include
command_value.I spent a few hours working through these changes but hit many speed bumps.
Because we've been waiting on SLC capabilities to come in, I've opened this issue for future work.