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.

Sign up for GitHub

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

Jump to bottom

Spawn models should raise an assertion of air pressure gets unreasonable high or low #3319

Open
mwetter opened this issue Mar 30, 2023 · 1 comment · May be fixed by #3320
Open

Spawn models should raise an assertion of air pressure gets unreasonable high or low #3319

mwetter opened this issue Mar 30, 2023 · 1 comment · May be fixed by #3320
Assignees
@mwetter
Labels
spawn Development for Spawn of EnergyPlus
Milestone
Release 11.0

Comments

@mwetter
Copy link
Member

mwetter commented Mar 30, 2023

If in room models that connect to EnergyPlus only infiltration is modeled but no exhaust air, then the pressure raises indefinite. Such a misconfiguration could be detected by monitoring the fluid port pressure, ideally when the model is already time sampled to avoid a zero crossing function.

Such an problem is likely the cause of https://groups.google.com/g/modelica-buildings/c/7IIKG_DjkpQ/m/WoTdNEx-AgAJ?utm_medium=email&utm_source=footer
@mwetter mwetter added the spawn Development for Spawn of EnergyPlus label Mar 30, 2023
@mwetter mwetter added this to the Release 10.0 milestone Mar 30, 2023
@mwetter mwetter self-assigned this Mar 30, 2023
@mwetter mwetter linked a pull request Mar 30, 2023 that will close this issue
Open
@mwetter
Copy link
Member Author

mwetter commented Apr 3, 2023

The failed CI test has been reported to Modelon, issue #2023040339000309

@mwetter mwetter modified the milestones: Release 10.0, Release 11.0 Sep 5, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@mwetter mwetter

Labels
spawn Development for Spawn of EnergyPlus
Projects
None yet
Milestone
Release 11.0
Development

Successfully merging a pull request may close this issue.

Added check for air pressure to be within reasonable limits lbl-srg/modelica-buildings
1 participant
@mwetter