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

Outdoor air summary multipliers #8528

Closed
1 of 5 tasks
mjwitte opened this issue Feb 11, 2021 · 0 comments · Fixed by #9519
Closed
1 of 5 tasks

Outdoor air summary multipliers #8528

mjwitte opened this issue Feb 11, 2021 · 0 comments · Fixed by #9519
Assignees

Comments

@mjwitte
Copy link
Contributor

mjwitte commented Feb 11, 2021

Issue overview

  • In the old Outdoor Air Summary - the AFN infiltration values are being multiplied when the should not be. See code here
    The multipliers were added in Revised html reporting for zone list multipliers #5192, so need to test this with and without multipliers, but it looks wrong here based on code inspection.

  • In the LEED Summary Report, the row order in the "EAp2-6. Energy Use Summary" and "EAp2-7. Energy Cost Summary" sub-tables is impacted if another table calls PreDefTableEntry for a row labeled "Total" in FillRemainingPredefinedEntries, because that becomes the first use of "Total" and it comes before any use of "Electricity" or "Gas" or other fuel type which first occur in WriteBEPS. It would be good to avoid this order dependency somehow. Perhaps prepopulate the unique label list or call FillRemainingPredefinedEntries after WriteBEPS or ???

Details

Some additional details for this issue (if relevant):

Checklist

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

  • Defect file added in EnergyPlusDevSupport\DefectFiles
  • Ticket added to Pivotal for defect (development team task)
  • Pull request created (the pull request will have additional tasks related to reviewing changes that fix this defect)
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 a pull request may close this issue.

1 participant