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

Replace Hard-Coded Sensible Capacity Curves with ADP/BF #1707

Merged
merged 70 commits into from
Sep 20, 2024

Conversation

joseph-robertson
Copy link
Collaborator

@joseph-robertson joseph-robertson commented May 8, 2024

Pull Request Description

[improves the design airflow/design SHR calculation in apply_hvac_equipment_adjustments within hvac_sizing.rb for central-ducted air systems used for cooling]

Checklist

PR Author: Check these when they're done. Not all may apply. strikethrough and check any that do not apply.

PR Reviewer: Verify each has been completed.

  • Schematron validator (EPvalidator.xml) has been updated
  • Sample files have been added/updated (openstudio tasks.rb update_hpxmls)
  • Tests have been added/updated (HPXMLtoOpenStudio/tests/test_hvac_sizing.rb and/or workflow/tests/test*.rb)
  • Documentation has been updated (no change necessary)
  • Changelog has been updated
  • openstudio tasks.rb update_measures has been run
  • No unexpected changes to simulation results of sample files. TODO: merge and test w/ diverse set of buildings/climates in ResStock house 237.xml (Florida) edge case was investigated, and the ADP/BF approach for this case produces more reasonable equipment capacities (see attached excel, ppt).

ADP_BF_ResStock Testing Results.pptx

testing_ResStock_datapoint_237.xlsx

Sashadf1 added 6 commits May 30, 2024 17:33
…ties to calculate design SHR. Cool Cap Design throwing undefined variable error in Psychrometrics.CalculateSHR
…sn't numerically fail. hvac_cooling_ap.rated_shr and hvac_cooling_shr showing 0.05 discrepancy. ask team if this is a problem?
…etter approach to calculating the design SHR and design airflow through the cooling coil for the following OS-HPXML equipment types: Central AC, A2A HP used for cooling, Mini-Split AC, and ducted MSHP used for cooling
@Sashadf1 Sashadf1 changed the title Replace Sensible Capacity Curves with ADP/BF Replace Hard-Coded Sensible Capacity Curves with ADP/BF Jun 20, 2024
Sashadf1 and others added 5 commits June 21, 2024 14:33
…o adp_bf_method_design_shr

# Conflicts:
#	HPXMLtoOpenStudio/measure.rb
#	HPXMLtoOpenStudio/resources/hvac.rb
#	HPXMLtoOpenStudio/resources/hvac_sizing.rb
#	HPXMLtoOpenStudio/resources/psychrometrics.rb
…o adp_bf_method_design_shr

# Conflicts:
#	HPXMLtoOpenStudio/measure.xml
shorowit and others added 9 commits August 21, 2024 14:09
…o adp_bf_method_design_shr

# Conflicts:
#	HPXMLtoOpenStudio/measure.xml
#	HPXMLtoOpenStudio/resources/hvac_sizing.rb
#	workflow/tests/base_results/results_acca_hvac.csv
#	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
#	workflow/tests/base_results/results_sizing.csv
@Sashadf1 Sashadf1 requested a review from shorowit September 17, 2024 21:19
@Sashadf1
Copy link
Collaborator

@shorowit to resolve merge conflicts and coordinate w/ residential team to test w/ diverse buildstock run

…o adp_bf_method_design_shr

# Conflicts:
#	HPXMLtoOpenStudio/measure.rb
#	HPXMLtoOpenStudio/measure.xml
@shorowit shorowit mentioned this pull request Sep 18, 2024
6 tasks
GitHub Action and others added 3 commits September 18, 2024 04:31
Copy link
Contributor

@shorowit shorowit left a comment

Choose a reason for hiding this comment

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

We have pounded the tires on this enough. Thanks for the good work here, @Sashadf1! I'll merge this in once the CI comes back happy.

@shorowit shorowit merged commit 4143ec9 into master Sep 20, 2024
7 checks passed
@shorowit shorowit deleted the adp_bf_method_design_shr branch September 20, 2024 21:35
@joseph-robertson joseph-robertson mentioned this pull request Sep 23, 2024
26 tasks
@Sashadf1 Sashadf1 mentioned this pull request Oct 7, 2024
3 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

5 participants