You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
{{ message }}
Repository navigation
Fix FanPerformance:NightVentilation across fan types - #11883
Removes "Maximum Flow Rate" from FanPerformance:NightVentilation, eliminating its VAV-only flow cap and potential fan/inlet mismatch. The transition warns when a nonblank value is discarded.
Fan:ConstantVolume: Air loop behavior is unchanged; zone equipment fans use their component’s night ventilation manager status.
Fan:VariableVolume: Uses alternate night pressure and efficiency without the normal part load curve; power is based on actual airflow, including at low flow.
Fan:OnOff: Applies alternate night settings without normal speed ratio curves.
Fan:ZoneExhaust: Applies night settings only when coupled to a ventilating serving air loop.
Fan:SystemModel: Uses its night pressure when specified, otherwise falls back to design pressure. Continuous mode night power scales with actual flow rather than design flow.
Fouled-filter airflow reduction is calculated against rated fan pressure, while the selected night pressure is retained for operating calculations.
Scopes night ventilation state to the relevant equipment, including first-call air-terminal handling; updates the IDD, transition rules, documentation, examples, and regression tests.
Description of the purpose of this PR
Pull Request Author
Title of PR should be user-synopsis style (clearly understandable in a standalone changelog context)
Label the PR with at least one of: Defect, Refactoring, NewFeature, Performance, and/or DoNoPublish
Pull requests that impact EnergyPlus code must also include unit tests to cover enhancement or defect repair
Author should provide a "walkthrough" of relevant code changes using a GitHub code review comment process
If any diffs are expected, author must demonstrate they are justified using plots and descriptions
If changes fix a defect, the fix should be demonstrated in plots and descriptions
If any defect files are updated to a more recent version, upload new versions here or on DevSupport
If IDD requires transition, transition source, rules, ExpandObjects, and IDFs must be updated, and add IDDChange label
If structural output changes, add to output rules file and add OutputChange label
If adding/removing any LaTeX docs or figures, update that document's CMakeLists file dependencies
If adding/removing any output files (e.g., eplustbl.*)
Update ..\scripts\Epl-run.bat
Update ..\scripts\RunEPlus.bat
Update ..\src\EPLaunch\ MainModule.bas, epl-ui.frm, and epl.vbp (VersionComments)
Update ...github\workflows\energyplus.py
Reviewer
Perform a Code Review on GitHub
If branch is behind develop, merge develop and build locally to check for side effects of the merge
If defect, verify by running develop branch and reproducing defect, then running PR and reproducing fix
If feature, test running new feature, try creative ways to break it
CI status: all green or justified
Check that performance is not impacted (CI Linux results include performance check)
Run Unit Test(s) locally
Check any new function arguments for performance impacts
Verify IDF naming conventions and styles, memos and notes and defaults
If new idf included, locally check the err file and other outputs
Addresses #11808 by making night-ventilation fan performance consistent across supported fan types and scoping night-mode state to the relevant equipment.
Changes:
Removes the night-ventilation maximum-flow field and adds input transition support.
Updates fan calculations and equipment-specific night-mode handling.
Adds regression coverage and updates documentation and example inputs.
The reason will be displayed to describe this comment to others. Learn more.
🔵 Needs a closer look
Cross-equipment state handling and input migration need verified simulation and transition results, with unresolved low-flow power and regression-coverage concerns.
Add end-to-end regression for zone fan night performance selection
src/EnergyPlus/SystemAvailabilityManager.cc:283
The zone-fan regressions assign availManagers(1).availStatus manually, so they would still pass if this production assignment were removed and zone equipment stopped selecting night performance. Add a regression that runs ManageSystemAvailability through actual manager evaluation, changes night ventilation from CycleOn to NoAction, and checks the stored component-specific status and resulting fan performance.
Use the fixed-speed night power multiplier in both low-flow heat-safeguard branches instead of reapplying the normal fan power curve. Add a cubic-curve regression that verifies expected night power at very low flow.
Evaluate zone NightVentilation managers through ManageSystemAvailability instead of manually assigning CycleOn and NoAction statuses. Verify component-specific manager and aggregate statuses transition from CycleOn to NoAction and that fan power returns to its design value, while retaining stale-global-state coverage.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
DefectIncludes code to repair a defect in EnergyPlusIDDChangeCode changes impact the IDD file (cannot be merged after IO freeze)
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
joseph-robertson
added
Defect
Pull request overview
FanPerformance:NightVentilationhandling across fan types #11808FanPerformance:NightVentilation, eliminating its VAV-only flow cap and potential fan/inlet mismatch. The transition warns when a nonblank value is discarded.Fan:ConstantVolume: Air loop behavior is unchanged; zone equipment fans use their component’s night ventilation manager status.Fan:VariableVolume: Uses alternate night pressure and efficiency without the normal part load curve; power is based on actual airflow, including at low flow.Fan:OnOff: Applies alternate night settings without normal speed ratio curves.Fan:ZoneExhaust: Applies night settings only when coupled to a ventilating serving air loop.Fan:SystemModel: Uses its night pressure when specified, otherwise falls back to design pressure. Continuous mode night power scales with actual flow rather than design flow.Description of the purpose of this PR
Pull Request Author
Reviewer