-
Notifications
You must be signed in to change notification settings - Fork 82
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
1367 geothermalheatpump example fails in openmodelica #1441
1367 geothermalheatpump example fails in openmodelica #1441
Conversation
…lheatpump-example-fails-in-openmodelica
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please change the name and check if the new connection is required. If the line is better now, you can leave it.
AixLib/Fluid/Examples/GeothermalHeatPump/BaseClasses/GeothermalHeatPumpBase.mo
Show resolved
Hide resolved
AixLib/Fluid/Examples/GeothermalHeatPump/BaseClasses/GeothermalHeatPumpBase.mo
Outdated
Show resolved
Hide resolved
Errors in regression test. Compare the results on the following page |
…ion reference files.Please pull the new files before push again. Plottet Results https://ebc.pages.rwth-aachen.de/EBC_all/github_ci/AixLib/1367-geothermalheatpump-example-fails-in-openmodelica/plots/
The previous pipelines failed due to another, not already solved, problem with the |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Adressed requested changes. See comments.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good, thanks!
Closes #1367
The flow direction from the cold storage to the evaporator was going in the wrong direction. The heating coil orientation was changed to down->up and the port connections were changed to allow the flow in the right direction.