NatLabRockies / NatLabRockies/EnergyPlus

Add to warning of negative zone volume the azimuths and tilts for all surfaces in zone

Open
#6,392 4 comments 0 reactions 1 assignee View on GitHub

@JasonGlazer is already working on this.

Since Nov 14, 2017.

Defect
Dominant language
C++
Stars
1.6k
Forks
490
Avg merge
6d 21h
Merged PRs (30d)
22

Description

When a negative zone volume is detected but the zone is fully enclosed, it is likely that the remaining issue is that not all the surfaces coordinates are in the correct order. To help users diagnose this issue, when a negative zone volume is detected the azimuths, tilts and areas for all the surfaces of the zone should be reported. Perhaps the report should only occur when DisplayExtraWarnings is used.

See https://unmethours.com/question/27850/why-a-negative-zone-volume/

To reproduce this problem reverse the order of coordinates for a few of the surfaces for a zone in a test file.

Also added a defect file in DevSupport

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.