-
Notifications
You must be signed in to change notification settings - Fork 7
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
Add support for Wpl13 #70
base: master
Are you sure you want to change the base?
Conversation
The GitHub Actions build has failed. Check the workflow run details for more information. |
There was a header missing. Should build now |
The GitHub Actions build has failed. Check the workflow run details for more information. |
https://github.com/kr0ner/OneESP32ToRuleThemAll/blob/master/yaml/common.yaml#L60 |
yes, just found it ;-) now I get a lot of messages:
|
You got a weird mixture of my code on the other there ... e.g. HILFSKESSELSOLL is nowhere defined in my code, but still edit: I saw that this one is already correctly mapped ... so maybe continue with stuff like VORLAUFIST, VORLAUFSOLL and so on. Keep me in the loop so that we can progress faster ;) |
fyi: I added HILFSKESSELSOLL myself, since this was already catched with the other repo's code... I just need to verify if it's really that value... |
ok, no problemo ... everything is fine then :) was just wondering where this thingy comes from and locally it only found one reference in the trash of the old repo :D |
No, you are not. There are multiple sources for each property. The one with zero is the one that request the value, important so that we can later fake that request to get the updated value on demand. And the other two make me think that HK1 and HK2 are actually 0x480 and 0x339 😉 For the screenshots: trust me, we are almost there 👍 |
I'll now try to create some pictures of the screens and also attach a bigger log or that time range... |
@adrai based on what I could guess from the logs I updated the yaml and simplified it a bit for now. Also added some hints there to guide you ... first thing we need to do now is to identify the can members correctly ... once we have that we should find out how to request stuff and after that it is your turn to fill the list of properties ;) |
Btw. do you have more than one ESP connected to the heatpump? Cause the amount of CAN messages is quite high ... or are you scrolling through the menu while logging? |
The GitHub Actions build has failed. Check the workflow run details for more information. |
No, I have just 1 ESP. Yes, while logging I made those screen pictures... but I did not modify any setting. Will check your update tomorrow if I have enough time. |
wanted to test it one time before going to sleep ;-)
and also
|
@adrai can you please enable verbose logging? I need to see those lines as well
So that just means we are registering the callback for the wrong CAN-Id. That is just a config issue and it will get better soon. The initial setup is always a bumpy road. I changed the ids again and added an additional log.
So whenever the view in the display changes and values are being displayed, there were CAN messages sent out ;) The heatpump sends most sensor values only on request, which is a CAN message either sent by us or by the display. |
The GitHub Actions build has failed. Check the workflow run details for more information. |
logs_esp32-heatingpump_logs.txt |
1, no longer ignoring write commands |
The GitHub Actions build has failed. Check the workflow run details for more information. |
Because the response to our request is 32768 🤷
How can you obtain the RAUMISTTEMP_HK1 manually? |
I have no clue. There is also 0x514 |
The GitHub Actions build has failed. Check the workflow run details for more information. |
I'd say we focus now on getting the system stable and to a point where we can merge the first version to master. 1, Can you please click through "Programmschalter" and find the ID that keeps changing when you select "Automatik", "Warmwasser", "Handbetrieb" and so on. Please focus on the important values first. e.g. hot water temp ist/soll , rücklauftemp, raumeinfluss, heizleistung, heizkreispumpe leistung, hysterese warmwasser, and leistungsaufnahme + wärmemenge for COP calculation. After we got those, i'll get the stuff into shape and merge it. Next step is to enable Programmschalter and the climate controls for heating and hot water. After that COP calculation |
The GitHub Actions build has failed. Check the workflow run details for more information. |
The GitHub Actions build has failed. Check the workflow run details for more information. |
There is just the FET2 attached to the bus (outside of the heatpump)... I use the heatpump only for heating, not for warm water.
I have these options: I changed the status manually and traced the logs, but I have no clue.... |
so looks like 0x480 is the FES then
thanks, gonna have a look later |
looks like the mapping of the TTF07c. https://github.com/mkaiser/OneESP32ToRuleThemAll/blob/edf095dfec1b9aaff635bd644db8eb59c3f2f504/yaml/ttf07.yaml#L398 Betriebszustand here: |
@mkaiser I think that the TTF uses different encoding ... it uses the bits while WPL13 uses numbers 0 to 5 |
The GitHub Actions build has failed. Check the workflow run details for more information. |
You're right... and I've found the property that holds this information: 0x4ecd And I also found some more properties:
btw: just noticed, STROM_INVERTER and SPANNUNG_INVERTER needs accuracy_decimals 1
PS. where did you find this information? Can you send me that document? |
|
May be it is helpful.... Attached you will find 1hour log of my WPL13...I used the thz504.yaml Config |
@adrai are you sure about that? What value is there in the display? IMO the type should be set to default (uint16_t) and with that accuracy_decimals is 0 resulting in e.g. 230V @ 5A |
The display shows 1 decimal 🤷♂️ |
oki doki, seems to be different on the THZ504 then ;) |
🤔 Wrong repo? 😅 |
omg... too many github projects 🤦♂️ and too early in the morning... sorry... deleted the comments |
No description provided.