Fix GHP experimental model rated conditions by yzhou601 · Pull Request #2204 · NatLabRockies/OpenStudio-HPXML

yzhou601 · 2026-04-30T18:15:44Z

Pull Request Description

GHP experimental models now directly use user-provided COP/EER in E+ coil objects, however, E+ coil rated conditions are different from GLHP rated conditions. We should convert user-provided GLHP COP/EER/capacities before adding coil inputs.

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

…o GHP_experimental_rated_conditions

…m/NREL/OpenStudio-HPXML into GHP_experimental_rated_conditions

…o GHP_experimental_rated_conditions

…m/NREL/OpenStudio-HPXML into GHP_experimental_rated_conditions

…o GHP_experimental_rated_conditions

…m/NREL/OpenStudio-HPXML into GHP_experimental_rated_conditions

…e unit tests to make sure two models are consistent in rated inputs

yzhou601 · 2026-05-23T00:56:18Z

          # E+ Capacity and EIR as function of temperature curves(bi-quadratic) generated using E+ HVACCurveFitTool
          # See: https://bigladdersoftware.com/epx/docs/24-2/auxiliary-programs/hvac-performance-curve-fit-tool.html#hvac-performance-curve-fit-tool
-          # Catalog data from : https://files.climatemaster.com/Genesis-GS-Series-Product-Catalog.pdf, p180
+          # Catalog data from : https://www.climatemaster.com/download/18.274be999165850ccd5b5b73/1535543867815/lc377-climatemaster-commercial-tranquility-20-single-stage-ts-series-water-source-heat-pump-submittal-set.pdf


Update outdated product links

yzhou601 · 2026-05-23T00:57:53Z

-  GroundSourceCoolRatedWET = 85.0 # degF, Rated water entering temperature for ground-source systems, cooling
  GroundSourceCoolRatedIDB = 80.0 # degF, Rated indoor drybulb for ground-source systems, cooling
  GroundSourceCoolRatedIWB = 67.0 # degF, Rated indoor wetbulb for ground-source systems, cooling
+  GroundSourceHeatGLHPRatedEWT = 32.0 # degF, Rated water entering temperature for ground-source systems, heating


These variables are used to convert E+ rated conditions to GLHP rated conditions

yzhou601 · 2026-05-23T01:00:33Z

        )
        speed = OpenStudio::Model::CoilCoolingWaterToAirHeatPumpVariableSpeedEquationFitSpeedData.new(model, cap_ft_curve, cap_faf_curve, cap_fwf_curve, eir_ft_curve, eir_faf_curve, eir_fwf_curve, waste_heat_ft)
+        # convert GLHP rated COPs/capacities to E+ rated
+		rated_capacity_eplus = UnitConversions.convert(heat_pump.cooling_capacity, 'Btu/hr', 'W') / get_experimental_ghp_rated_condition_conversion(:clg, hp_ap.cool_cap_ft_spec[i]) * hp_ap.cool_capacity_ratios[i]


Convert HPXML rated inputs (GLHP rated conditions) to E+ rated inputs

yzhou601 · 2026-05-23T01:13:20Z

+	    # Need to adjust the capacity to GLHP rated conditions
+        total_cap_curve_value_design = MathTools.biquadratic(UnitConversions.convert(mj.cool_indoor_wetbulb, 'F', 'C'), UnitConversions.convert(entering_temp, 'F', 'C'), clg_ap.cool_cap_ft_spec[hvac_cooling_speed])
+        total_cap_curve_value_glhp_rated = MathTools.biquadratic(UnitConversions.convert(HVAC::GroundSourceCoolRatedIWB, 'F', 'C'), UnitConversions.convert(HVAC::GroundSourceCoolGLHPRatedEWT, 'F', 'C'), clg_ap.cool_cap_ft_spec[hvac_cooling_speed])
+		total_cap_curve_value = total_cap_curve_value_design / total_cap_curve_value_glhp_rated


Uses E+ performance curves to convert design condition to GLHP rated conditions (not E+ rated condition, so here we need to do curve calculation twice).

This will make the heat pumps sized at GLHP rated conditions, consistent with what user provides. Then the capacities will be converted to E+ rated inputs later when we creates E+ models.

yzhou601 · 2026-05-23T01:15:48Z

+    cool_eir_ft_spec = [1.1828664909, -0.0450835550, 0.0009273315, 0.0056194113, 0.0006683467, -0.0007256237]
+	cool_capacity_curve_value = MathTools.biquadratic(UnitConversions.convert(HVAC::GroundSourceCoolRatedIWB, 'F', 'C'), UnitConversions.convert(HVAC::GroundSourceCoolGLHPRatedEWT, 'F', 'C'), cool_cap_ft_spec)
+	cool_eir_curve_value = MathTools.biquadratic(UnitConversions.convert(HVAC::GroundSourceCoolRatedIWB, 'F', 'C'), UnitConversions.convert(HVAC::GroundSourceCoolGLHPRatedEWT, 'F', 'C'), cool_eir_ft_spec)
+	expected_clg_capacity = standard_clg_capacity / cool_capacity_curve_value


Unit test to make sure the rated conditions are converted correctly.
The experimental model should have the consistent rated inputs as standard models before conversion.

Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>

…m/NREL/OpenStudio-HPXML into GHP_experimental_rated_conditions

…o GHP_experimental_rated_conditions

… can directly apply at interim speeds

Copilot

Pull request overview

Copilot reviewed 11 out of 11 changed files in this pull request and generated 10 comments.

indentation fix by AI Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>

Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>

…m/NREL/OpenStudio-HPXML into GHP_experimental_rated_conditions

Copilot

Pull request overview

Copilot reviewed 12 out of 12 changed files in this pull request and generated 1 comment.

Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>

…o GHP_experimental_rated_conditions

yzhou601 · 2026-06-11T23:15:17Z

@jmaguire1 will check experimental model airflow/waterflow curves to make sure they're expected.

…o GHP_experimental_rated_conditions

yzhou601 added 2 commits April 30, 2026 11:41

convert user provided cops

55ae9a1

hvac sizing

f270e68

github-project-automation Bot added this to OpenStudio-HPXML Apr 30, 2026

github-project-automation Bot moved this to Triage in OpenStudio-HPXML Apr 30, 2026

yzhou601 self-assigned this Apr 30, 2026

yzhou601 added 2 commits April 30, 2026 12:48

capacity, refactor

ae7df53

one more cleanup

33aca65

shorowit added this to the 1.13.0 milestone May 5, 2026

yzhou601 and others added 7 commits May 11, 2026 14:33

temporarily disabled experimental unit tests

dff73ee

Merge branch 'master' of https://github.com/NREL/OpenStudio-HPXML int…

4c0cb87

…o GHP_experimental_rated_conditions

changed capacity ratios and cop ratios based on rated data

76c1ee7

Latest results. [skip ci]

0f6fe3c

update comments

b48d2b3

Merge branch 'GHP_experimental_rated_conditions' of https://github.co…

9b34f98

…m/NREL/OpenStudio-HPXML into GHP_experimental_rated_conditions

Merge branch 'master' of https://github.com/NREL/OpenStudio-HPXML int…

c7b6331

…o GHP_experimental_rated_conditions

shorowit moved this from Triage to In progress in OpenStudio-HPXML May 12, 2026

GitHub Action and others added 9 commits May 12, 2026 17:21

Latest results. [skip ci]

2a5d00a

Merge branch 'master' of https://github.com/NREL/OpenStudio-HPXML int…

d2c2cf4

…o GHP_experimental_rated_conditions

Merge branch 'GHP_experimental_rated_conditions' of https://github.co…

f1cd56e

…m/NREL/OpenStudio-HPXML into GHP_experimental_rated_conditions

Latest results. [skip ci]

2fea4e1

Merge branch 'master' of https://github.com/NREL/OpenStudio-HPXML int…

63ae249

…o GHP_experimental_rated_conditions

Merge branch 'GHP_experimental_rated_conditions' of https://github.co…

732765a

…m/NREL/OpenStudio-HPXML into GHP_experimental_rated_conditions

update unit tests

30931f3

revert experimental assumptions to stick to E+ rated conditions, wrot…

f23eff8

…e unit tests to make sure two models are consistent in rated inputs

changelog

b95bc5f

yzhou601 commented May 23, 2026

View reviewed changes

yzhou601 marked this pull request as ready for review May 23, 2026 01:16

Latest results. [skip ci]

1733f91

yzhou601 requested a review from Copilot May 28, 2026 17:37

Copilot started reviewing on behalf of yzhou601 May 28, 2026 17:37 View session

yzhou601 and others added 8 commits May 28, 2026 12:09

typo fix for unit test 2

b543e9c

Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>

Apply suggestions from code review

b3f6a27

Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>

change coil reference capacity too, address method comment

625d0cd

Merge branch 'GHP_experimental_rated_conditions' of https://github.co…

9ea8fb8

…m/NREL/OpenStudio-HPXML into GHP_experimental_rated_conditions

Merge branch 'master' of https://github.com/NREL/OpenStudio-HPXML int…

62b062b

…o GHP_experimental_rated_conditions

fix index, the capacity ratios are calculated at E+ rated conditions,…

981d5e4

… can directly apply at interim speeds

exclude a warning

88b955d

Latest results. [skip ci]

e0d9e44

yzhou601 requested a review from Copilot June 1, 2026 19:47

Copilot started reviewing on behalf of yzhou601 June 1, 2026 19:47 View session

Copilot AI reviewed Jun 1, 2026

View reviewed changes

yzhou601 and others added 5 commits June 1, 2026 15:11

Apply suggestions from code review

d13b701

indentation fix by AI Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>

Potential fix for pull request finding

d4f7866

Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>

Apply suggestions from code review - reuse methods

bb53c39

Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>

update_measures finally ran!

2344d0f

Merge branch 'GHP_experimental_rated_conditions' of https://github.co…

f4fb5be

…m/NREL/OpenStudio-HPXML into GHP_experimental_rated_conditions

yzhou601 requested a review from Copilot June 1, 2026 23:03

Copilot started reviewing on behalf of yzhou601 June 1, 2026 23:03 View session

Copilot AI reviewed Jun 1, 2026

View reviewed changes

Comment thread HPXMLtoOpenStudio/resources/hvac.rb Outdated

remove duplicate method explanation

6a6237b

Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>

yzhou601 requested a review from jmaguire1 June 1, 2026 23:47

yzhou601 added 2 commits June 10, 2026 14:05

Merge branch 'master' of https://github.com/NREL/OpenStudio-HPXML int…

ab60212

…o GHP_experimental_rated_conditions

updated airflow coefficients to fix an excel error at curve generation

d02d202

GitHub Action and others added 6 commits June 12, 2026 03:19

Latest results. [skip ci]

c160f82

Merge branch 'master' of https://github.com/NREL/OpenStudio-HPXML int…

cf98335

…o GHP_experimental_rated_conditions

Update curves

e337db3

update measures

076bfb1

Merge branch 'master' of https://github.com/NREL/OpenStudio-HPXML int…

ca0f413

…o GHP_experimental_rated_conditions

Latest results. [skip ci]

4cc8669

Uh oh!

Conversation

yzhou601 commented Apr 30, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Pull Request Description

Checklist

Uh oh!

yzhou601 May 23, 2026

Choose a reason for hiding this comment

Uh oh!

yzhou601 May 23, 2026

Choose a reason for hiding this comment

Uh oh!

yzhou601 May 23, 2026

Choose a reason for hiding this comment

Uh oh!

yzhou601 May 23, 2026

Choose a reason for hiding this comment

Uh oh!

yzhou601 May 23, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Uh oh!

yzhou601 commented Jun 11, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

yzhou601 commented Apr 30, 2026 •

edited

Loading