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

Update HeatPumpType.md #142

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

Update HeatPumpType.md #142

wants to merge 2 commits into from

Conversation

kampsj
Copy link

@kampsj kampsj commented Oct 19, 2024

Added:
WH-ADC0309J3E5 WH-UD07JE5 KIT-ADC07JE5
62 D2 0B 45 54 42 D2 0B 72 66

Added:
WH-ADC0309J3E5 WH-UD07JE5 KIT-ADC07JE5
62 D2 0B 45 54 42 D2 0B 72 66
WH-ADC0309J3E5 WH-UD07JE5 KIT-ADC07JE5
62 D2 0B 45 54 42 D2 0B 72 66
Copy link
Author

@kampsj kampsj left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

fixed numbering

@IgorYbema
Copy link
Owner

Can you confirm your model number?
The last part of the bytes suggest a WH-UD09JE5 and not the 07.
Look at model 8 from the https://github.com/IgorYbema/HeishaMon/blob/main/HeatPumpType.md which is from byte 5 to byte 10 the same.

@MiG-41
Copy link

MiG-41 commented Dec 22, 2024

In several commerciall annocements you can find ,that KIT-ADC07JE5 is WH-ADC0309J3E5 and WH-UD07JE5 ... So probably that are correct data.

Since internal is 0309 ( so it is universal from 3kW to 9kW ) ,only difference to vary them is outdoor unit ( even in practice is the same)....

@IgorYbema
Copy link
Owner

Yes I know this but I am trying, once again, to decode the byte values and "42 D2 0B 72 66" should be the outdoor unit. Which is this case already points to WH-UD09JE5 in model type 8 and with this suggested PR this would be pointing to WH-UD07JE5 also, which is unexpected compared to the other values in the table.

@IgorYbema
Copy link
Owner

@MiG-41 this model nr 8 was inserted by you years ago, do you remember where it came from? see d7e7c4d

Maybe that entry is wrong?

@kampsj
Copy link
Author

kampsj commented Dec 22, 2024

This is the heatpump data on which I based my edit:
[000]: 71 C8 01 10 56 55 62 49 00 A9 00 02 00 00 00 00 00 00 00 00 29 15 13 55 96 95 55 55 55 29 00 00
[032]: 00 00 00 00 00 00 94 9C 80 80 B0 68 01 51 99 00 00 00 00 00 00 00 00 00 00 00 80 85 15 8A 85 85
[064]: D0 7B 78 1F 7E 1F 1F 79 79 8D 8D 9F 9A 7B 8A B7 A3 7B 8F 8A 84 76 8F 8A 94 9E 8F 8A 94 9E 85 8F
[096]: 8A 03 5B 7E BC 15 7E 7C 1F 7C 7E 00 00 00 55 55 55 21 78 15 59 05 1C 13 66 00 00 00 00 00 00 00
[128]: 00 62 D2 0B 45 54 42 D2 0B 72 66 94 00 AC 84 95 96 32 32 9C AE 32 32 32 94 9C 95 86 94 94 83 80
[160]: 83 80 84 2B 01 01 01 00 00 22 00 01 01 01 01 79 79 01 01 42 10 00 6E 0A 00 6A 01 00 28 00 00 0A
[192]: 01 01 01 01 01 01 01 02 00 00 C0

@MiG-41
Copy link

MiG-41 commented Dec 22, 2024

Don't remember... It could be even someone from Slack , who give this datas...

This one was first J type , probably was a good check of it....

@MiG-41
Copy link

MiG-41 commented Dec 28, 2024

@IgorYbema ,if you have some ideas , we could try to send them via TAW1 to Service Cloud, and see ,how Panasonic will interprate it ( what model it will give).
Long time ago i had prepared such "tests" with NodeRED , and made some discoveres for new L platform...

@IgorYbema
Copy link
Owner

Ah yes that is a good method to test. First of all maybe just try to test all known heatpump types we already have? To make sure we didn't make any mistake.

@MiG-41
Copy link

MiG-41 commented Dec 28, 2024

Ok.... But looks like update on SerwiceCloud this particular data takes long time , new model is refreshing app after 15min... So now i'm on pos3 of the list ( from 49 :) )

@MiG-41
Copy link

MiG-41 commented Dec 28, 2024

Looks like this update of HP takes long time , even 3h....
So veryfy all will took a lot of time... Anyway this one ,on with you have doupt can be confirmed:
image

image

@MiG-41
Copy link

MiG-41 commented Dec 29, 2024

So generally , b[131] and b[136] don't influence/change thy type of inner and/or outher unit , i can place anything there...

Changing the b[132] to 0x45 gives Inner unit as WH-ADC0309J3E5 , and outher remains same WH-UD09JE5-1 ....
image

image

@geduxas
Copy link

geduxas commented Dec 30, 2024

Panasonic in LCD, also taw a2w app will split it to 4 segments,
0xD261 (130-129)
0x54450D (133-131)
0xD241 (135-134)
0x66720C (138-136)

One segment is for IDU, and second for ODU, i almost shure that last digit in longer value represents some sort of version/revision(A/B/C/D) of model.. also because of same structure are in monoblocks, i think it somehow tied to main board version/configuration..

@IgorYbema
Copy link
Owner

IgorYbema commented Dec 30, 2024

Yes when I remove byte 131 and 136 I already can remove some heatpump types differences (which are marked new version). So that is a good thing.
Byte 132 seems to store the capacity of the heatpump on the indoor side.
Byte 137 does the same for the outdoor side.
But there is not regularity in those values. For each heatpump models these are different. I believe it is just a incremental list of numbers based on the release of new heatpumps.
Take for example this outdoor unit list:

Bytes ODU
12 D0 91 05 WH-UQ09HE8
12 D0 92 05 WH-UQ12HE8
12 D0 94 05 WH-UX09HE8
12 D0 95 05 WH-UX12HE8
12 D0 96 05 WH-UX16HE8
12 D0 97 05 WH-UD09HE8
12 D0 98 05 WH-UD12HE8

But this ones are also weird. For example the 12 D0 06 11 which exists twice with different model number. And the model WH-UD12HE5 has two different bytes (06 vs 10 at b[137]) @MiG-41 can you try this ODU bytes in your test to see what happens?

Bytes ODU
12 D0 05 11 WH-UX09HE5
12 D0 06 11 WH-UD12HE5
12 D0 06 11 WH-UX12HE5
12 D0 10 11 WH-UD12HE5

@MiG-41
Copy link

MiG-41 commented Dec 30, 2024

Ok , will try..
@IgorYbema byte 132 not only the capacity , but "purpose" as well ( changing this from 043 to 0x45 it changed from SDC to ADC , so from "regular" to All-In-One)...

Now such thing:
image

Gives Outer unit unknonwn:
image

So tried to put to my HP T-CAP 12kW 3phase put outher unit T-CAP 9kW 1phase...

EDIT: No , after some time finally it apears:
image

@geduxas
Copy link

geduxas commented Dec 30, 2024

I think first two bytes will mean some configuration, and rest 3 will remain to software or pcb with some sort of revision..

@MiG-41
Copy link

MiG-41 commented Dec 30, 2024

image
image

@MiG-41
Copy link

MiG-41 commented Dec 30, 2024

image
image

@MiG-41
Copy link

MiG-41 commented Dec 30, 2024

image
image
Let's wait a while , if this will be recognized later...
Changed only byte 136 to be more KIT-ADC12HE5...

@MiG-41
Copy link

MiG-41 commented Dec 30, 2024

image
image

Still unknown , so will make another mix...

@MiG-41
Copy link

MiG-41 commented Dec 30, 2024

image
image

@MiG-41
Copy link

MiG-41 commented Dec 30, 2024

So pos 25 is wrong, It can't be UD since it is T-CAP ,it have to be UX...
So let's confirm this

EDIT: Yes , pos.25 on the list has wrong ODU , should be described as WH-UX12HE5
image
image
Now everything should be match.

@MiG-41
Copy link

MiG-41 commented Dec 30, 2024

Also funny , when it can be two Monoblock at the same time :)
Took halfbytes from one and next half from other ,
image
image

@geduxas
Copy link

geduxas commented Dec 30, 2024

So indeed they have some kind code>model mapping table.. and for monoblocks i think they have same models for odu and idu.. so this explains why it's showing different model's.. if we could somehow access that table :D

@IgorYbema
Copy link
Owner

IgorYbema commented Dec 30, 2024

I'll update nr 25 on the list. Thanks for finding that.
I asked the one who changed this one (was nr 23 in that commit, but same line): 9f8f247

@MiG-41
Copy link

MiG-41 commented Dec 30, 2024

Looks like it can be any kind of mix , like
image
image

@IgorYbema
Copy link
Owner

@MiG-41 @geduxas I'm am testing out v3.9 and want to release it soon. I was hopening to make some smart model test logic in the code but this seems not logical yet. And as it doesn't provide any information I am now thinking of removing the model TOPIC completly (until we finally figure out wat the logic is, if there is any). What are your thoughts about removing the TOPIC?

@MiG-41
Copy link

MiG-41 commented Jan 12, 2025

I don't belive is there any logic , they are just random ( or better next free) with Panasonic takes.
Maybe just now remove byte 131 and 136 (as it is only no with not influence on models) , and display (instead of unknown model ) just all 129-138 bytes , so users could easly give them ( instead questions how to get these no).
Next thing is ,if githhub can have some formular ,for sending such informations to speed up this ( now some do complete PR , other creating new topic in issues , and here still onse give complete info , others asking what means unknown)...

Without input from users in my opinion we will not find logic here ( if any).
Or do you have other opinion? I can tests/check some ready bytes ,but i don't think it will be possible to check it one by one tarting from zeros in bytes :)

@IgorYbema
Copy link
Owner

We could do that but then ending up with a big lists of model types costing a lot of storage memory on the controllers. And it has no function at all. So why bother doing this?
We could however just show the bytes only and let the home integrations (like home assistant) do the translation further? It doesn't make any functional change for heishamon to translate it

@MiG-41
Copy link

MiG-41 commented Jan 12, 2025

Such option was also some times ago , Heihamon show only pure bytes , in github translation table , and in Heishamon web if possible also online translation from that table.

For sure this TOPIC have no function at all.

@geduxas
Copy link

geduxas commented Jan 12, 2025

Best way is to use same logic as panasonic do, and leave model recognition/mapping to endpoints... In some time we could get mapping table and all endpoints could map models.. for now as is it's not logical.. also it's complicated to maintain.

And we don't have big database to investigate model/code mapping for now..

As i find out all codes could lead not to model, but in some kind of pcb firmware version and some modification code..

@IgorYbema
Copy link
Owner

Ok so if you agree I will remove the translation and only show the 10 bytes.
And I'll try to add some javascript code to lookup the address bytes in the github table so it runs on the web client

@geduxas
Copy link

geduxas commented Jan 12, 2025

If possible split bytes to 4 Topic, 10 bytes will not give desired function

@IgorYbema
Copy link
Owner

I understand but that can be easily done by the end device. And if you keep all 10 bytes in one topic it doesn't need to receive all topics before trying to match to the github list

@geduxas
Copy link

geduxas commented Jan 12, 2025

At least seperate idu and odu..

@MiG-41
Copy link

MiG-41 commented Jan 12, 2025

First this ( 10 bytes searching in table) , display as much as possible from table .
What about topic in mqtt ? ( maybe some json from this table download , or at least 10 bytes).

Later have to be considered to ommit byte 131 and 136 , treated as a version ... ( so in github table have to be considered)

@IgorYbema
Copy link
Owner

Currently I changed it to this. Acceptable?
image

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 this pull request may close these issues.

4 participants