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

TDD diffuser and dome inside and outside surface heat balance calculations not consistent with EngRef #8526

Closed
1 of 3 tasks
xuanluo113 opened this issue Feb 10, 2021 · 0 comments · Fixed by #8540
Closed
1 of 3 tasks
Assignees
Labels
Defect Includes code to repair a defect in EnergyPlus

Comments

@xuanluo113
Copy link
Contributor

xuanluo113 commented Feb 10, 2021

Issue overview

According to the Engineering Reference,

For conductive and convective heat gain, TDDs are treated as one entity with an effective thermal resistance (i.e. R-value) between the outside and inside surfaces. The outside face temperature of the dome and the inside face temperature of the diffuser are calculated as usual by the outside and inside heat balances respectively. The temperatures are then copied to the inside face of the dome and the outside face of the diffuser.

However, in EnergyPlus inside and outside surface heat balance calculations...

First, the surface outside temperature calculation is bypassed for TDD surface as their HeatTransferAlgorithm is assigned as HeatTransferModel_TDD. The TDD branch never gets accessed. These temperature values are initialized and remain static.
Then the values are copied to the TDD diffuser's outside temperature.

Second, the inside surf temperature of the diffuser is copied to the dome in the diffuser zone loop (e.g. Zone(1)), but later in the dome surface zone loop (e.g. Zone(2)), the inside surf temperature of the dome is recalculated and assigned to the reported value, overwriting the previously copied value.

Here is a snapshot of the summer design day results for the testfile DaylightingDeviceTubular.idf. The outside temperature of the diffuser and dome, and the inside temperature of the dome are not calculated correctly.

Picture1

Checklist

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

  • Defect file added (list location of defect file here)
  • 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)
@xuanluo113 xuanluo113 added the Defect Includes code to repair a defect in EnergyPlus label Feb 10, 2021
@xuanluo113 xuanluo113 self-assigned this Feb 10, 2021
@xuanluo113 xuanluo113 added this to the EnergyPlus 9.5.0 milestone Feb 10, 2021
@xuanluo113 xuanluo113 linked a pull request Feb 25, 2021 that will close this issue
20 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Defect Includes code to repair a defect in EnergyPlus
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant