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

Air System Outdoor Air Limiting Factor can be wrong for simple OA controls due to tiny differences #8802

Closed
3 tasks
mjwitte opened this issue Jun 3, 2021 · 0 comments · Fixed by #9947
Closed
3 tasks
Assignees

Comments

@mjwitte
Copy link
Contributor

mjwitte commented Jun 3, 2021

Issue overview

Air System Outdoor Air Limiting Factor uses several < or > tests to determine if the OA flow rate has been constrained by a particular limiting factor (e.g. economizer, OA schedule, etc.). For simple OA controllers with no economizer, no min OA schedule, and no other constraints, this should always be zero. But occasional 1s show up due to tiny differences in comparing two values which should be equal, but due to different calculation paths for the two values, they differ out in the 10th decimal or so. Probably need to add a tolerance to one or more of these comparisons, and review the entire logic for this output.

Also, this output is reset to zero every time step inside CalcOAController which I'm pretty sure is not called if an airloop is off. Need a test file to prove this.

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 (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)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
1 participant