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

Issues on heat pump condenser/evaporator heat flow calculation #1507

Closed
Tracked by #1520
wehuang16 opened this issue Jul 2, 2024 · 3 comments · Fixed by #1532
Closed
Tracked by #1520

Issues on heat pump condenser/evaporator heat flow calculation #1507

wehuang16 opened this issue Jul 2, 2024 · 3 comments · Fixed by #1532

Comments

@wehuang16
Copy link

Describe the bug
For the Aixlib heat pump model during "cooling" operation, there is a possibility that the calculation for the condenser/evaporator heat flow is not quite correct.

To Reproduce

  1. Use a modelica tool, such as Dymola or OpenModelica, to simulate this heat pump example in the "main" branch (https://github.com/RWTH-EBC/AixLib/blob/main/AixLib/Fluid/HeatPumps/Examples/HeatPump.mo)

  2. we will first see that the simulation runs for 3600 seconds. The first half (1800s) is heating mode, and the second half is cooling mode.Screenshot 2024-06-30 at 11 14 28 AM

  3. View the parameter QCon, QEva, and Pel, we notice that during the second half of simulation when the heat pump is in cooling mode, QCon is about -9.1kW, QEva is about 8.7kW, Pel is about 0.4kW.
    Screenshot 2024-07-01 at 9 42 52 PM

  4. QEva is positive, meaning that heat flows from the heat pump refrigerant at the evaporator to the flowing fluid (water or air), so the evaporator in the model actually acts as an actual "condenser". QCon is negative, meaning that heat flows from the flowing fluid (water or air) to the heat pump refrigerant at the condenser, so the condenser in the model actually acts as an actual "evaporator".

  5. The correct calculation should be that the absolute value of the actual "evaporator" heat flow (currently represented by QCon for cooling) plus the electric power equal to the absolute value of the actual "condenser" heat flow (currently represented by QEva for cooling). Thus, the absolute value of the actual "evaporator" heat flow (9.1kW) should be less than the absolute value of the actual "condenser" heat flow (8.7kW), which might not be correct.

Expected behavior
I expect that for cooling, when the QCon is acting as an actual "evaporator", and QEva is acting as an actual "condenser", absolute value of QCon should be less than the absolute value of QEva.

Desktop (please complete the following information):
I used Dymola 2023x on an ubuntu 20.04 environment

Additional context
This issue on heat pump condenser/evaporator heat flow calculation has actually been fixed on the IBPSA heat pump libraries about 2 months ago. Specifically, we can look at these examples at the IBPSA github "main" branch (https://github.com/ibpsa/modelica-ibpsa/tree/master/IBPSA/Fluid/HeatPumps/ModularReversible/Examples)

@FWuellhorst
Copy link
Contributor

Thanks for raising this well documented issue!
You are correct. I just merged the latest IBPSA version, were I fixed this issue, into AixLib. The "old" models will be moved to the Obsolete package, as the changes are not manageable using a conversion script. I would highly recommend using the new models. If you have questions on how to switch, please ask. I will try to give a short tutorial on how to switch from old to new HP models.

@wehuang16
Copy link
Author

Thanks for raising this well documented issue! You are correct. I just merged the latest IBPSA version, were I fixed this issue, into AixLib. The "old" models will be moved to the Obsolete package, as the changes are not manageable using a conversion script. I would highly recommend using the new models. If you have questions on how to switch, please ask. I will try to give a short tutorial on how to switch from old to new HP models.

Thank you!

@FWuellhorst
Copy link
Contributor

@wehuang16 we moved the model in the latest main. Please see https://github.com/RWTH-EBC/AixLib/wiki/Switch-from-model-HeatPump-to-ModularReversible if you want to move from the old to the new approach.

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