Skip to content

Conversation

bpark1327
Copy link
Collaborator

@bpark1327 bpark1327 commented Mar 28, 2025

Pull Request Description

Correct the airtightness used for infiltration rate calculation is based on test results but not less than 0.3 ACHnatural when Aext < 0.5 and the mechanical ventilation system is exhaust-only,

Checklist

Not all may apply:

  • OS-HPXML git subtree has been pulled
  • 301validator.xml has been updated (reference EPvalidator.xml)
  • Sample files have been added/updated (openstudio tasks.rb update_hpxmls)
  • Tests have been added/updated (e.g., rulesets\tests 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 on CI

@bpark1327
Copy link
Collaborator Author

Modeled a middle interior apartment unit located in Climate Zone 6 with an exhaust-only mechanical ventilation system.
The unit was modeled with an infiltration rate of 0.27 CFM50/ft² of enclosure area. It has 3,730 ft² of compartmentalization boundary area and an infiltration volume of 10,200 ft³.
The table below illustrates the impact of modifying the rated home's infiltration calculation on the model's results.

  Before After
ERI 53.56 50.16
Infiltration Rate (ACH50) 8.98 1.83
Mech Vent Fan Tested Flow Rate (CFM) 34.5 57
Mech Vent Fan Power (W) 12.321 20.357

MFNCv13_CZ6_VT_elec_middle_interior-exhaust.xml.txt

@bpark1327 bpark1327 marked this pull request as ready for review April 1, 2025 20:15
@bpark1327 bpark1327 requested a review from shorowit April 1, 2025 21:03
Copy link
Collaborator

@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.

This all looks good to me. Since I only want a changelog update, I'll just go ahead and make that change and then merge this in. Thanks, @bpark1327!

end
end

ach50 *= a_ext
Copy link
Collaborator

Choose a reason for hiding this comment

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

👍

_test_ruleset(hpxml_name, 'latest').each do |(_run_type, calc_type), hpxml_bldg|
if [CalcType::RatedHome].include? calc_type
_check_infiltration(hpxml_bldg, ach50: 10.1, height: 8.0, volume: 7200.0)
_check_infiltration(hpxml_bldg, ach50: 2.49, height: 8.0, volume: 7200.0)
Copy link
Collaborator

Choose a reason for hiding this comment

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

Looks right.

@shorowit shorowit merged commit ffef901 into master Apr 1, 2025
@shorowit shorowit deleted the infil_rate_calc_correction branch April 1, 2025 21:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants