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

Rebuilding the reversible HP model based on a new general partial model for reversible HPs and chillers #715

Closed
JulianMatthes opened this issue May 8, 2019 · 5 comments
Assignees
Milestone

Comments

@JulianMatthes
Copy link
Contributor

JulianMatthes commented May 8, 2019

What is the problem?

A new reversible chiller model based on the structure of the reversible heat pump model is required. To optimize its usability a new partial model for thermal machines (e.g. heat pumps, chillers) shall be developed. Furthermore the previous reversible heat pump model and the new reversible chiller model are to be extended from this partial model. Also, the improvements in applicability include variable naming.
For that reason the variable naming when changing the operation mode shall fit to its belonging function (evaporator or condenser).

How do we want to solve it?

A new partial model shall be developed. The heat pump and the chiller model extend from this partial model placed in the base class of the Fluid library. A query is defined as to whether reversible operation is possible. In addition, the elements on the condenser and evaporator side are named and connected starting from the main operating mode (heating or cooling).
Finally belonging FastHVAC models are to be adapted analogously.

JulianMatthes added a commit that referenced this issue May 8, 2019
The package structure was created.
@JulianMatthes JulianMatthes changed the title Conversion of the reversible HP model into a new chiller model Extension of the reversible HP model by a new just chilling operation mode May 13, 2019
@JulianMatthes JulianMatthes changed the title Extension of the reversible HP model by a new just chilling operation mode Extension of the reversible HP model by a new 'just chilling' operation mode May 13, 2019
JulianMatthes added a commit that referenced this issue May 13, 2019
…al 'just cooling' operation mode. Simulation results match with the current heat pump model. #715
@JulianMatthes JulianMatthes changed the title Extension of the reversible HP model by a new 'just chilling' operation mode Rebuilding the reversible HP model based on a new general partial model for reversible HPs and chillers May 15, 2019
JulianMatthes added a commit that referenced this issue May 16, 2019
…hermalMachine.mo. Furthermore building of the new reversible heat pump model based on the partial models. Also, the belonging new example for the reversible heat pump was built and tested. #715
JulianMatthes added a commit that referenced this issue May 20, 2019
JulianMatthes added a commit that referenced this issue May 20, 2019
JulianMatthes added a commit that referenced this issue May 22, 2019
Furthermore changed heat pump Systems/HeatPumpSystems/HeatPumpSystem.mo because of this issue.
Next step is to also adapt the FastHVAC models.
JulianMatthes added a commit that referenced this issue May 22, 2019
…Components/HeatGenerators/HeatPump.mo for issue #715
JulianMatthes added a commit that referenced this issue May 27, 2019
… old models from the first development steps in this issue. #715
JulianMatthes added a commit that referenced this issue May 29, 2019
… to hand down the heat pump characteristics chosen on the upper model level to the used heat pump model. #715
@DaJansenGit DaJansenGit added this to the Hackday milestone Jul 9, 2019
JulianMatthes added a commit that referenced this issue Jul 10, 2019
…HeatPump/PerformanceData.mo to improve the handling and use of the thermal machine model.

Furthermore fixed consideration of the icing factor in Chillers/BaseClasses/PerformanceData.mo. #715
JulianMatthes added a commit that referenced this issue Jul 10, 2019
…erefore corrected examples in FastHVAC and Fluid library. #715
JulianMatthes added a commit that referenced this issue Jul 15, 2019
@DaJansenGit DaJansenGit modified the milestones: Hackday, v0.8.0 Sep 25, 2019
@DaJansenGit
Copy link
Member

DaJansenGit commented Sep 25, 2019

@JulianMatthes please have a look at #769 and check if everything still works or if their are necessary changes. If no changes needed, can we merge this into development?

@JulianMatthes
Copy link
Contributor Author

@DaJansenGit #769 is based on the current dev and this branch was already 2 months behind it. So changes had to be made merging it into this branch. They can be found in the belonging PR #724

@KremerMartin
Copy link
Contributor

@DaJansenGit Can you have a look at the failed check in PR #724 ? As I see it, the branch will be ready to merge after merging the last changes in the development into this branch, if the tests pass. Maybe you have got a clue, what went wrong. I am no expert on CI...

@DaJansenGit
Copy link
Member

DaJansenGit commented Oct 8, 2019

@KremerMartin this seems to be due to missing regression test results. We have the same problem with #774. In the next days I will try to update the regression test files.
Edit: Ok the problem seems not to be missing regression test files but too big deviation from reference results for some tests and/or missing tolerances in regression scripts. I will have a closer look the next days.

@DaJansenGit
Copy link
Member

closed by merge of #724

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

No branches or pull requests

3 participants