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

MixingVolumeTraceSubstanceReverseFlow has negative C_outflow #613

Closed
mwetter opened this issue Dec 8, 2016 · 0 comments · Fixed by #614
Closed

MixingVolumeTraceSubstanceReverseFlow has negative C_outflow #613

mwetter opened this issue Dec 8, 2016 · 0 comments · Fixed by #614
Assignees
@mwetter

Comments

@mwetter
Copy link
Contributor

mwetter commented Dec 8, 2016

The boundary condition in this example is set up in such a way that during reverse flow, port_a.C_outflow is negative. This should be fixed to be physically correct, and to avoid that the min attribute is violated.
@mwetter mwetter self-assigned this Dec 8, 2016
mwetter added a commit that referenced this issue Dec 8, 2016
@mwetter
Changed boundary condition for #613
bbb6ae7
@mwetter mwetter mentioned this issue Dec 8, 2016
Merged
Mathadon added a commit that referenced this issue Dec 8, 2016
@Mathadon
Merge pull request #614 from iea-annex60/issue613_C_outflow
8541f0c 
Changed boundary condition for #613
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@mwetter mwetter

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Changed boundary condition for #613 ibpsa/modelica-ibpsa
1 participant
@mwetter