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

Incorrect accounting for zone multipliers in the Outdoor Air Details report #10799

Open
1 of 3 tasks
mdahlhausen opened this issue Oct 22, 2024 · 2 comments
Open
1 of 3 tasks

Comments

@mdahlhausen
Copy link
Contributor

Issue overview

When zone multipliers are used with zone multipliers, incorrect values show up in the Mechanical Ventilation Parameters by Zone table in the Outdoor Air Details Report. This may only be occurring for some, not all system types.

In this example model DOAS_WSHP_multipliers.zip, CORE_MID ZN (and other MID zones) have a zone multiplier of 10. CORE_MID ZN has the same load assumptions as CORE_BOTTOM ZN and CORE_TOP ZN. CORE_BOTTOM ZN and CORE_TOP ZN have a zone multiplier of 1.

image

Average Number of Occupants,Nominal Number of Occupants,Zone Volume [m3], and Zone Area are the same for all CORE zones. Design Zone Outdoor Airflow - Voz [m3/s] and Minimum Dynamic Target Ventilation - Voz-dyn-min [m3/s] are different, apparently already accounting for the zone multiplier. In the Total Facility line, the MID zone multipliers are applied again to calculate the total, giving incorrect totals for Design Zone Outdoor Airflow - Voz [m3/s] and Minimum Dynamic Target Ventilation - Voz-dyn-min [m3/s]. The code here fails to account for the zone multipliers already being included in the total.

Details

  • EnergyPlus 24.1

Checklist

Add to this list or remove from it as applicable. This is a simple templated set of guidelines.

  • Defect file added (defect file here).
  • Ticket added to EnergyPlus Defect Complexity (Github Project)
  • Pull request created (the pull request will have additional tasks related to reviewing changes that fix this defect)
@mjwitte
Copy link
Contributor

mjwitte commented Oct 23, 2024

@mdahlhausen It might be useful to add a column for zone multiplier to help keep things clear. Would you prefer to see the last two columns report the values with or without multipliers?

@mdahlhausen
Copy link
Contributor Author

mdahlhausen commented Oct 23, 2024

The tables in the Outdoor Air Summary Report has a footnote on the tables: "Values shown for a single zone without multipliers". For the Outdoor Air Details tables I think a good option would be to show the values without zone multipliers, add the footnote "Values shown for a single zone without multipliers", and then add a column of zone multipliers.

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

No branches or pull requests

2 participants