Skip to content

Conversation

@chipbarnaby
Copy link
Contributor

Description

Relief-vent scheme (AIRNETEXT with overridden exterior conditions produced bad results.

With this PR, AIRNETEXT exterior pressure is now computed with ambient (outdoor) air density. This can be thought of as meaning the relief vents are connect to a zone with outdoor pressure -- that is, a zone with a large vent area to ambient.

Additional changes

  • increased the threshold for 'unreasonable zone pressure" messages with the intent that messages will occur only for truly malformed networks.
  • coded out obsolete AIRNET_COMPARE conditional code

One regression test uses overridden exterior conditions. Associated reference result file updated.

No documentation changes.

@chipbarnaby chipbarnaby marked this pull request as draft October 9, 2023 21:18
@chipbarnaby chipbarnaby requested a review from nealkruis October 10, 2023 14:51
@chipbarnaby chipbarnaby marked this pull request as ready for review October 10, 2023 14:55
@nealkruis nealkruis merged commit 980a4cd into main Oct 10, 2023
@nealkruis nealkruis deleted the airnet-fixes branch October 10, 2023 15:38
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 this pull request may close these issues.

3 participants