Skip to content

Add mechanical ventilation warnings#813

Merged
shorowit merged 1 commit into
masterfrom
mechvent_warnings
May 22, 2026
Merged

Add mechanical ventilation warnings#813
shorowit merged 1 commit into
masterfrom
mechvent_warnings

Conversation

@shorowit
Copy link
Copy Markdown
Collaborator

Pull Request Description

Adds some warnings related to mechanical ventilation systems with unmeasured airflow.

For example, if this is in the HPXML:

            <VentilationFan>
              <SystemIdentifier id='VentilationFan1'/>
              <FanType>energy recovery ventilator</FanType>
              <RatedFlowRate dataSource>75.8</RatedFlowRate>
              <HoursInOperation>24.0</HoursInOperation>
              <UsedForWholeBuildingVentilation>true</UsedForWholeBuildingVentilation>
              <IsSharedSystem>false</IsSharedSystem>
              <TotalRecoveryEfficiency>0.6</TotalRecoveryEfficiency>
              <SensibleRecoveryEfficiency>0.7</SensibleRecoveryEfficiency>
              <FanPower>66.1</FanPower>
              <extension>
                <FlowRateNotTested>true</FlowRateNotTested>
              </extension>
            </VentilationFan>

you will now get:

Warning: Both RatedFlowRate and extension/FlowRateNotTested="true" are provided; the ventilation system is assumed to have unmeasured airflow and RatedFlowRate will not be used.

Checklist

Not all may apply:

  • OS-HPXML git subtree has been pulled
  • 301validator.sch has been updated (reference EPvalidator.sch)
  • 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

@shorowit shorowit self-assigned this May 21, 2026
@shorowit shorowit merged commit 58b7d82 into master May 22, 2026
9 checks passed
@shorowit shorowit deleted the mechvent_warnings branch May 22, 2026 23:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant