Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update HVAC models per RESNET HVAC addendum #1879

Draft
wants to merge 152 commits into
base: master
Choose a base branch
from
Draft

Conversation

yzhou601
Copy link
Collaborator

@yzhou601 yzhou601 commented Nov 9, 2024

Pull Request Description

closes #1871

Checklist

Not all may apply:

  • Schematron validator (EPvalidator.xml) 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

@yzhou601 yzhou601 self-assigned this Nov 12, 2024
HPXMLtoOpenStudio/resources/defaults.rb Outdated Show resolved Hide resolved
HPXMLtoOpenStudio/resources/defaults.rb Outdated Show resolved Hide resolved
HPXMLtoOpenStudio/resources/defaults.rb Show resolved Hide resolved
HPXMLtoOpenStudio/resources/hvac.rb Show resolved Hide resolved
HPXMLtoOpenStudio/resources/hvac.rb Show resolved Hide resolved
HPXMLtoOpenStudio/resources/hvac.rb Show resolved Hide resolved
@yzhou601
Copy link
Collaborator Author

CI is complaining about the BuildResidentialHPXML test: test_workflows at the extra-second-heating-system-fireplace-to-heat-pump.xml. I checked the error and generated hpxml file, it's because the HeatingCapacity17F input is not consistent with 17F nominal datapoint in detailed performance data. But I can't find the place where the HeatingCapacity17F is set. @joseph-robertson @shorowit can you help point out the place?

@shorowit
Copy link
Contributor

CI is complaining about the BuildResidentialHPXML test: test_workflows at the extra-second-heating-system-fireplace-to-heat-pump.xml. I checked the error and generated hpxml file, it's because the HeatingCapacity17F input is not consistent with 17F nominal datapoint in detailed performance data. But I can't find the place where the HeatingCapacity17F is set. @joseph-robertson @shorowit can you help point out the place?

Just fixed it. HeatingCapacity17F was being defaulted, but it had more decimal places than the detailed performance data defaulted capacities.

yzhou601 and others added 9 commits February 13, 2025 11:21
…o resnet_heat_pump

# Conflicts:
#	Changelog.md
#	HPXMLtoOpenStudio/measure.xml
#	workflow/tests/base_results/results_simulations_bills.csv
#	workflow/tests/base_results/results_simulations_energy.csv
#	workflow/tests/base_results/results_simulations_loads.csv
#	workflow/tests/base_results/results_simulations_misc.csv
@shorowit shorowit mentioned this pull request Feb 15, 2025
7 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
Status: In progress
Development

Successfully merging this pull request may close these issues.

Update HP models per RESNET HVAC addendum
2 participants