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

Annex60.Fluid.Sensors.TraceSubstancesTwoPort outputs wrong value if allowFlowReversal=false #249

Closed
mwetter opened this issue May 22, 2015 · 0 comments · Fixed by #250
Closed

Annex60.Fluid.Sensors.TraceSubstancesTwoPort outputs wrong value if allowFlowReversal=false #249

mwetter opened this issue May 22, 2015 · 0 comments · Fixed by #250
Assignees
@mwetter

Comments

@mwetter
Copy link
Contributor

mwetter commented May 22, 2015

The assignment

CMed = s*inStream(port_b.C_outflow);

needs to be changed to

CMed = s*port_b.C_outflow;
@mwetter mwetter self-assigned this May 22, 2015
mwetter added a commit that referenced this issue May 22, 2015
@mwetter
Corrected issue #249 and added test case
ea6aab1
@mwetter mwetter mentioned this issue May 22, 2015
Merged
mwetter added a commit that referenced this issue May 26, 2015
@mwetter
Merge pull request #250 from iea-annex60/issue249_CSensor
3ff7698 
Corrected issue #249 and added test case
mwetter added a commit to lbl-srg/modelica-buildings that referenced this issue May 27, 2015
@mwetter
Merged Annex60
11d664d 
This includes bug fix ibpsa/modelica-ibpsa#249
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.

Corrected issue #249 and added test case ibpsa/modelica-ibpsa
1 participant
@mwetter