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

Short-cycling in heat pump example #1876

Closed
mwetter opened this issue May 9, 2024 · 1 comment · Fixed by #1877
Closed

Short-cycling in heat pump example #1876

mwetter opened this issue May 9, 2024 · 1 comment · Fixed by #1877
Assignees

Comments

@mwetter
Copy link
Contributor

mwetter commented May 9, 2024

@FWuellhorst
In the model Fluid.HeatPumps.ModularReversible.Examples.AirToWater2D_OneRoomRadiator the heat pump short-cycles during part of the simulation. It switches on and off in a 1 minute interval during cooling mode.
It looks like the controller needs to be changed to avoid such operation.
This is with the model from #1628 but I suggest to handle this in a new PR so that we can merge #1628 once it passes the CI tests.

@FWuellhorst
Copy link
Contributor

@mwetter :
The underlying issue is that QCoo_flow_nominal is set to a value which oversizes the heat pump during cooling operation:

Warning: The following was detected at time: 0
  In AirToWater2D_OneRoomRadiator.heaPum: Scaling factors for heating and cooling
    operation differ by -25.8333 %. The simulated nominal heating and cooling
    capacities may not be realistic for a single device
  Failed condition: abs(heaPum.scaFacHea-heaPum.scaFacCoo)/heaPum.scaFacHea < heaPum.limWarSca

Changing the control would be difficult, as

  • the heat pump already runs at minimal part load, decreasing it further is not realistic
  • Decreasing the minimal radiator temperature would lead to condensation effects in reality.
  • Setting a minimal run time would lead to temperature violating the minimal radiator temperature.

So, this system would be a bad fit in reality, as well, as no energy storage decouple the low-inertia radiator from the oversized heat pump.

With the correct sizing, the behavior looks like this:
image
I wanted to trigger the warning to have examples for the different warnings which may occur.
I could change it to undersizing, so that the warning still occurs. Would that be fine for you?

In any case, I will update the doc of the example to explain to conflict of sizing for these systems.

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