-
-
Notifications
You must be signed in to change notification settings - Fork 36
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
Implemented power limitation, #306
base: main
Are you sure you want to change the base?
Conversation
…elated to power limitation, not silent mode, added undefined_24 & 25 in switch off reasons
Great implementation! I Hope BenPru finds the time soon to approve this commit! |
I can only control the settings, power limitation does not work for me. When I open EVU, the heat pump is locked and will not start heating, whether power limitation is on or off. When I bridge EVU2 and enable smart grid, power limitation will be ignored |
Sorry I can't follow. So you wrote this without having the benefit from it? |
What to you mean with "open"? Bridged? When EVU is bridged and I activate power limitation, the heat pump will not be limited, it is not working. |
OK I double checked and you are right. The EVU must be bridged, since I had no EVU limitations prior to my activation of smart grid. On the hardware wiring part of view, my machine is stock. |
And this is working for you? |
Well in the controler it says: And with this setting power limitation is working |
But than you also have reduced target temperature right? |
Nope, regular target temperature. |
So with my setup I get the smart grid working condition "EVU locked". (out of the 4 possible conditions) |
And it is really heating? |
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.
Deleting lines 229-240 in const.py and re-adding lines for undefined_24 and undefined_25 is unnecessary and should be dropped.
And maybe after "UnitOfPower" in number_entities_predefined.py there should be a comma
The rest of the changes work fine.
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.
Approved with the second commit
@blue-bean Is the power limitation working for you? When it's enable while EVU is not bridged or Smart Grid is enabled, the status message "power limitation" will show up, but the heat pump never starts heating. |
It's working for me like @stefanovich1 mentioned. If power limit is 'on' when i activate Smart Grid, the WP is working respecting the power limit (see the marked symbol in the screenshots): If Smart Grid is 'off' and i activate the power limit, nothing happens, the WP is working as usual. |
I see the symbol, but when I enable Smart Grid, the heat pump immediatley stops heating and won't start again, although power limitation is enabled. |
Did you try to increase the power limit in several steps? |
And when status is not blocked - was the external flow out temperature really lower than (flow out temperature target - hysteresis)? How long did you wait after setting a new limit? |
Yes of course. Now, I managed to get power limitation activated while it is heating. After the start phase, (I don't know how long it takes), is shows "REDUCED", then I lowered the limit step by step from 1,5 to 0,6kW and it stops heating on 0,6kW, but the consumption was all the time above the limit. This feature feels a little bit buggy for me |
In another forum, a user wrote that the heat pump does not start when the power limit is activated. The power limit can only be activated when the pump is already running. This is probably to prevent the pump from exceeding the limit when starting up, as the pump always has to start at medium power. |
This happend with me when the Limit is too low. Especially during Start or water heating a too little value can cause Problems. |
But then, this feature is completely useless or buggy. The heat pump should
ignore the limit on start up.
stefanovich1 ***@***.***> schrieb am Fr. 20. Dez. 2024 um
09:07:
… This happend with me when the Limit is too low. Especially during Start or
water heating a too little value can cause Problems.
Here comes your commit into play. Now you can adjust the Limitaccording
Status, time or some Thing else
—
Reply to this email directly, view it on GitHub
<#306 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AOWXFIOOLCHL7IY2UGZPRLT2GPF5PAVCNFSM6AAAAABTZPEIMWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDKNJWGQ4DINZWGI>
.
You are receiving this because you authored the thread.Message ID:
***@***.***>
|
I can confirm your commit working.... Cool job! |
Why should it?
stefanovich1 ***@***.***> schrieb am Fr. 20. Dez. 2024 um
09:10:
… I can confirm your commit working.... Cool job!
Although the namens are little mixed up.... It does not Show up as
"Leistungslimit".
Any idea to fix this minor cosmetical issue?
—
Reply to this email directly, view it on GitHub
<#306 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AOWXFIN32WPFUCOS77RBQL32GPGFXAVCNFSM6AAAAABTZPEIMWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDKNJWGQ4DOOBZHE>
.
You are receiving this because you authored the thread.Message ID:
***@***.***>
|
It does for me - did you have the changes in de.json? |
Sorry, my bad. By manually copying the files to the HA I missed the translations folder. Everything works and translates fine.
I think the original implementation was to meet a goverment requirement. You must be able to limit big electrical machines to 4,2kW. With this limit 99% of the heat pumps are not impaired in any way. btw: The Limit IS ignored during deicing |
Hi guys. Thank you for working on this topic. As I understood, we can use the power limitation then out of HA. Right? Great! I would like to install it on my test system. But unfortunately I have no idea how GitHub works. So I don't know how to "copy" the modified girls to my HA config. Any easy tip? :) |
Reading the comments so far, it seems this functionality needs good documentation.
I'm not about to test this at this moment, but I'll check how my heat pump reacts next week. |
I would like to test it. But don#t know how to implement the changes in my HA system |
I can confirm this behavior.
[cudos to blue-bean aka Frogman] If you are german speaking, have a look to the following threads: |
I can confirm as well. After turning on Smart Grid, the heat pump stoped, and just start running again, when I set the power limit above 1,2 kW or so. Afterwards I'm able to set the power limit as low as I like to and the pump runs. Maybe warm water is not going to reach the set temperature. It's great work, now I'm able to adjust the heat pump according to PV Überschuss at a certain time. Apart from this, I realized, that the heat pump is running much more smooth. Like more or less on the same power level (below the power limit). Before my heat pump was pulsing up and down. For example between 0,5 kW and 1,4 kW within 30 minutes. I guess this is not very efficent. Another strange thing I realized, after installing the "updated" files to custom components in my HA test system and switching Smart Grid to on, some sensors disappeared in my working system. Even after a restart they did not appear again until I copied the "updated" files also to my working system. Now it works and all sensors are online.
Thank you for your information regarding merging the different branches. I works as described! Great! |
Hi guys, in the new version 2025.01.04 Beta is the power limitation also implemented? I guess not, right? |
@Mehli12 Nope, waiting to be merged. |
Unfortunately not, BenPru hasn't merged it up to now. @BenPru |
Is this feature working with 30.9.3? I have no clue how to use this
feature, even if I read the German forum as a German guy
blue-bean ***@***.***> schrieb am Do. 9. Jan. 2025 um 10:31:
… Unfortunately not, BenPru hasn't merged it up to now.
—
Reply to this email directly, view it on GitHub
<#306 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AOWXFINEFEGJVRCRIS74LZT2JY6YPAVCNFSM6AAAAABTZPEIMWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDKNZZGU3TONZTGY>
.
You are receiving this because you authored the thread.Message ID:
***@***.***>
|
I'm running V3.90.3. According to the Luxtronik-UI the power limit is set correctly. |
@Codectory Please Provider a changelog Text with breaking Changes and new Sensors. |
@BenPru Where should I provide it? Specific file? Here, as comment? ;-) |
Comment |
|
visibilities.Unknown_Parameter_357 is related to power limitation, not silent mode, added undefined_24 & 25 in switch off reasons