Skip to content

Fix setpoint sensors#2193

Draft
yzhou601 wants to merge 3 commits intomasterfrom
thermostat_sensors
Draft

Fix setpoint sensors#2193
yzhou601 wants to merge 3 commits intomasterfrom
thermostat_sensors

Conversation

@yzhou601
Copy link
Copy Markdown
Collaborator

Pull Request Description

Output variables "Zone Thermostat Heating/Cooling Setpoint Temperature" can be changed by E+ when there's an on-off thermostat, EMS program should use setpoint "Schedule Value" instead to be safe.

Checklist

Not all may apply:

  • Schematron validator (EPvalidator.sch) has been updated
  • Sample files have been added/updated (openstudio tasks.rb update_hpxmls)
  • Tests have been added/updated (e.g., HPXMLtoOpenStudio/tests/test*.rb and/or workflow/tests/test*.rb)
  • Documentation has been updated
  • Changelog has been updated
  • openstudio tasks.rb update_measures has been run
  • No unexpected changes to simulation results of sample files

supp_staging_program.addLine(" Set #{supp_coil_avail_act.name} = 0")
supp_staging_program.addLine('Else') # global variable = 1
supp_staging_program.addLine(" Set #{supp_coil_avail_act.name} = 1")
supp_staging_program.addLine(" If (supp_htg_rate_1_ago > 0) && (#{htg_sp_ss.name} + #{living_temp_ss.name} > 0.01)")
Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

"+" here doesn't make sense to me, should be "-" instead, likely a typo, likely by me. :(

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Triage

Development

Successfully merging this pull request may close these issues.

1 participant