Skip to content

Conversation

@lymereJ
Copy link
Collaborator

@lymereJ lymereJ commented Apr 17, 2025

Pull request overview

Add new inputs/outputs to model air leakage in parallel fan-powered terminal units.

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 structural output changes, add to output rules file and add OutputChange label

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

@lymereJ lymereJ added NewFeature Includes code to add a new feature to EnergyPlus IDDChange Code changes impact the IDD file (cannot be merged after IO freeze) OutputChange Code changes output in such a way that it cannot be merged after IO freeze labels Apr 17, 2025
@NREL NREL deleted a comment from github-actions bot Apr 17, 2025
@NREL NREL deleted a comment from github-actions bot Apr 28, 2025
@NREL NREL deleted a comment from github-actions bot Apr 28, 2025
@NREL NREL deleted a comment from github-actions bot Apr 28, 2025
@mjwitte
Copy link
Contributor

mjwitte commented Sep 25, 2025

@lymereJ Is this targeted for 25.2?

@lymereJ
Copy link
Collaborator Author

lymereJ commented Sep 26, 2025

@mjwitte - Ideally, yes, but it's not quite ready. I'm hoping to be able to get back to it sometime next week.

@lymereJ lymereJ added this to the EnergyPlus 25.2 IO Freeze milestone Sep 26, 2025
@lymereJ lymereJ force-pushed the parallel_piu_leakage branch from d7d35c6 to 4254b29 Compare October 10, 2025 18:15
@NREL NREL deleted a comment from nrel-bot-2c Oct 14, 2025
@NREL NREL deleted a comment from nrel-bot-2c Oct 14, 2025
@NREL NREL deleted a comment from nrel-bot-2c Oct 14, 2025
@NREL NREL deleted a comment from nrel-bot-2 Oct 14, 2025
@NREL NREL deleted a comment from github-actions bot Oct 14, 2025
@NREL NREL deleted a comment from github-actions bot Oct 14, 2025
@github-actions
Copy link

⚠️ Regressions detected on macos-14 for commit 002d49a

Regression Summary
  • Audit: 4
  • RDD: 4

@github-actions
Copy link

⚠️ Regressions detected on macos-14 for commit e0cfaf5

Regression Summary
  • Audit: 4
  • RDD: 4

@github-actions
Copy link

⚠️ Regressions detected on ubuntu-24.04 for commit 002d49a

Regression Summary
  • Audit: 4
  • RDD: 4

@github-actions
Copy link

⚠️ Regressions detected on ubuntu-24.04 for commit e0cfaf5

Regression Summary
  • Audit: 4
  • RDD: 4

@github-actions
Copy link

⚠️ Regressions detected on macos-14 for commit fe079e3

Regression Summary
  • Audit: 4
  • RDD: 4

@github-actions
Copy link

⚠️ Regressions detected on ubuntu-24.04 for commit fe079e3

Regression Summary
  • Audit: 4
  • RDD: 4

@lymereJ
Copy link
Collaborator Author

lymereJ commented Oct 15, 2025

The following plots show results generated using this branch for the new example file, for two cases: with and without leakage for the parallel PIU terminal serving "Space1-1". The terminal fan pulls air from "Space5-1", that is also the zone that is impacted by the leaks since they occur through the backdraft damper.

Heating and dead-band operations:

  • Air is leaking only when the terminal fan is off
  • During dead-band the flow to the zone (SPACE1-1 IN NODE) is equal to the primary air flow rate - the leaks
image **Cooling operations:**
  • During cooling operation, the primary air ramps up, so do the leaks (top, maroon)
  • The impact of the leaks on "Space5-1" are shown on the bottom panel

image image

These results for the new example file are still relevant.

I conducted additional tests on a model that uses an AirLoopHVAC:ZoneMixer instead of an AirLoopHVAC:ReturnPlenum as the return path. The results appear to be as expected. For example, when analyzing the return air, its temperature during simulations with leaks is lower compared to the case without leaks. This outcome aligns with expectations, as backdraft damper leaks only occur during dead-band and cooling operations, causing the return air to be cooler. In the figure below, the top panel illustrates the return air temperature (blue for leaks, orange for no leaks), while the bottom panel depicts the return air mass flow rate (maroon for leaks, dark green for no leaks).

image

I compared the temperature of the zone designated as the destination of the leaks for both configurations: using an AirLoopHVAC:ZoneMixer and an AirLoopHVAC:ReturnPlenum object. In both cases, the designated zone is a plenum, and the annual temperature profiles are almost identical. In the figure below, the orange line represents the temperature of the plenum zone without leaks, the maroon line corresponds to the case with leaks using an AirLoopHVAC:ReturnPlenum object, and the blue line represents the case with leaks using an AirLoopHVAC:ZoneMixer.

image

I also ran similar models in different climate zones. As expected, simulations with leaks have more impact in hotter climate zones.

Here are the additional models:

I think that this is ready for review.

@lymereJ lymereJ marked this pull request as ready for review October 15, 2025 05:18
@github-actions
Copy link

⚠️ Regressions detected on macos-14 for commit 6128e1d

Regression Summary
  • Audit: 4
  • RDD: 4

@github-actions
Copy link

⚠️ Regressions detected on ubuntu-24.04 for commit 6128e1d

Regression Summary
  • Audit: 4
  • RDD: 4

@mitchute mitchute removed this from the EnergyPlus 25.2 IO Freeze milestone Oct 21, 2025
@nrel-bot-2
Copy link

@lymereJ it has been 28 days since this pull request was last updated.

@lymereJ lymereJ added this to the EnergyPlus 26.1 milestone Nov 19, 2025
@github-actions
Copy link

⚠️ Regressions detected on macos-14 for commit e626463

Regression Summary
  • Audit: 4
  • RDD: 4

@github-actions
Copy link

⚠️ Regressions detected on ubuntu-24.04 for commit e626463

Regression Summary
  • Audit: 4
  • RDD: 4

@github-actions
Copy link

⚠️ Regressions detected on macos-14 for commit 9c0a991

Regression Summary
  • Audit: 4
  • RDD: 4

@github-actions
Copy link

⚠️ Regressions detected on macos-14 for commit 0091f9d

Regression Summary
  • Audit: 4
  • RDD: 4

@github-actions
Copy link

⚠️ Regressions detected on ubuntu-24.04 for commit 9c0a991

Regression Summary
  • Audit: 4
  • RDD: 4

@github-actions
Copy link

⚠️ Regressions detected on ubuntu-24.04 for commit 0091f9d

Regression Summary
  • Audit: 4
  • RDD: 4

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

IDDChange Code changes impact the IDD file (cannot be merged after IO freeze) NewFeature Includes code to add a new feature to EnergyPlus OutputChange Code changes output in such a way that it cannot be merged after IO freeze

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants