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

IncludeThreeWayTable Valve #1254

Closed
alexanderAKU opened this issue Nov 15, 2019 · 2 comments · Fixed by #1256
Closed

IncludeThreeWayTable Valve #1254

alexanderAKU opened this issue Nov 15, 2019 · 2 comments · Fixed by #1256

Comments

@alexanderAKU
Copy link
Contributor

This issue is the extension of issue #1227:

Currently, the library contains a table based two way valve. However, a table based three way valve is not included yet.

The model new model ThreeWayTable will extend from the PartialThreeWayValve and replace the partial models res1 and res3 by the model TwoWayTable

@alexanderAKU
Copy link
Contributor Author

@mwetter could you create a branch with the name issue1254_includeThreeWayTableValve so that I can push my changes to the repository?

@mwetter
Copy link
Contributor

mwetter commented Nov 18, 2019

alexanderAKU pushed a commit to RWTH-EBC/modelica-ibpsa that referenced this issue Nov 27, 2019
alexanderAKU pushed a commit to RWTH-EBC/modelica-ibpsa that referenced this issue Nov 27, 2019
alexanderAKU pushed a commit to RWTH-EBC/modelica-ibpsa that referenced this issue Dec 17, 2019
…Valves example and delete example Valves.Examples.ThreeWayTable
alexanderAKU pushed a commit to RWTH-EBC/modelica-ibpsa that referenced this issue Dec 17, 2019
alexanderAKU pushed a commit to RWTH-EBC/modelica-ibpsa that referenced this issue Dec 17, 2019
…le.mos deleted and script ThreeWayValve modified
alexanderAKU pushed a commit to RWTH-EBC/modelica-ibpsa that referenced this issue Dec 17, 2019
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 a pull request may close this issue.

2 participants