home-assistant / core

:house_with_garden: Open source home automation that puts local control and privacy first.
https://www.home-assistant.io
Apache License 2.0
73.74k stars 30.85k forks source link

Nibe integration S1255 problems #83749

Closed HansKatsman closed 1 year ago

HansKatsman commented 1 year ago

The problem

When adding the S1255 heat pump it will report back as S1155 with sensors

adding the current power makes the integration stop and unavailable Logger: homeassistant.components.nibe_heatpump Source: helpers/update_coordinator.py:201 Integration: Nibe Heat Pump (documentation, issues) First occurred: 10 december 2022 21:20:27 (5 occurrences) Last logged: 09:26:40

Error fetching Nibe Heat Pump data: Failed to update: Error 'Function code is not an allowable action for the server.' reading current-power-eme-20-32177 starting: 2176 count: 2 from: 0

Adding the current power for the Nibe in configuration.yaml over modbus tcp works without a problem

What version of Home Assistant Core has the issue?

12.0

What was the last working version of Home Assistant Core?

No response

What type of installation are you running?

Home Assistant Container

Integration causing the issue

Nibe S1255

Link to integration documentation on our website

No response

Diagnostics information

No response

Example YAML snippet

No response

Anything in the logs that might be useful for us?

Logger: homeassistant.components.nibe_heatpump
Source: helpers/update_coordinator.py:201 
Integration: Nibe Heat Pump (documentation, issues) 
First occurred: 10 december 2022 21:20:27 (5 occurrences) 
Last logged: 09:26:40

Error fetching Nibe Heat Pump data: Failed to update: Error 'Function code is not an allowable action for the server.' reading current-power-eme-20-32177 starting: 2176 count: 2 from: 0

Additional information

No response

home-assistant[bot] commented 1 year ago

Hey there @elupus, mind taking a look at this issue as it has been labeled with an integration (nibe_heatpump) you are listed as a code owner for? Thanks!

Code owner commands Code owners of `nibe_heatpump` can trigger bot actions by commenting: - `@home-assistant close` Closes the issue. - `@home-assistant rename Awesome new title` Change the title of the issue. - `@home-assistant reopen` Reopen the issue. - `@home-assistant unassign nibe_heatpump` Removes the current integration label and assignees on the issue, add the integration domain after the command.

(message by CodeOwnersMention)


nibe_heatpump documentation nibe_heatpump source (message by IssueLinks)

elupus commented 1 year ago

When you request over modbus, do you request two registers or one? Its a 32bit value according to our database.

HansKatsman commented 1 year ago

Hi,

This is a part of my modbus configuration to my Nibe S1255

The yellow part is the overall power usage I receive and it has a INT16 type

Let me know if you need additional information

Regards Hans Katsman

Van: Joakim Plate @.> Verzonden: zondag 11 december 2022 19:41 Aan: home-assistant/core @.> CC: Hans Katsman @.>; Author @.> Onderwerp: Re: [home-assistant/core] Nibe integration S1255 problems (Issue #83749)

When you request over modbus, do you request two registers or one? Its a 32bit value according to our database.

— Reply to this email directly, view it on GitHub https://github.com/home-assistant/core/issues/83749#issuecomment-1345627106 , or unsubscribe https://github.com/notifications/unsubscribe-auth/AIAK6VYHH4GQCNVDSUCZEPTWMYN2DANCNFSM6AAAAAAS23HCZM . You are receiving this because you authored the thread. https://github.com/notifications/beacon/AIAK6VZRBZOF3WJC46HUG33WMYN2DA5CNFSM6AAAAAAS23HCZOWGG33NNVSW45C7OR4XAZNMJFZXG5LFINXW23LFNZ2KUY3PNVWWK3TUL5UWJTSQGSR6E.gif Message ID: @. @.> >

elupus commented 1 year ago

You are reading it as int16. Docs say int32. Can you see if you get same failure of you request int32?

HansKatsman commented 1 year ago

Hi

I changed to int 32 and it did not give an error in logging

However the reading was suddenly very high

Before it gave 742 W and after the change I had 655.360 W

I changed it back to Int16 for modbus for proper history

Van: Joakim Plate @.> Verzonden: zondag 11 december 2022 20:11 Aan: home-assistant/core @.> CC: Hans Katsman @.>; Author @.> Onderwerp: Re: [home-assistant/core] Nibe integration S1255 problems (Issue #83749)

You are reading it as int16. Docs say int32. Can you see if you get same failure of you request int32?

— Reply to this email directly, view it on GitHub https://github.com/home-assistant/core/issues/83749#issuecomment-1345632965 , or unsubscribe https://github.com/notifications/unsubscribe-auth/AIAK6VZC72E6R3XWUBJ3T43WMYRMBANCNFSM6AAAAAAS23HCZM . You are receiving this because you authored the thread. https://github.com/notifications/beacon/AIAK6V6ZLQI6R3ESRSPO7NTWMYRMBA5CNFSM6AAAAAAS23HCZOWGG33NNVSW45C7OR4XAZNMJFZXG5LFINXW23LFNZ2KUY3PNVWWK3TUL5UWJTSQGS5MK.gif Message ID: @. @.> >

elupus commented 1 year ago

You seen to have enabled the wrong entity? 32167 would match your modbus config.

dkmh commented 1 year ago

These are 2 different things: "32177" is the currently availabe PV Solar power if EME or Sunspec is used "32167" is the current power consumption of the heatpump <-- most propably this is what you want

But I can confirm that "current-power-eme-20-32177" seems to make problems since the last software update on my S1255, they seem to have changed something there. Will check the new register export when I'm at home.

HansKatsman commented 1 year ago

Yes the 32167 is the right one which I overlooked when selecting the data from your Nibe integration.

Now it’s working fine with the right data

Thanks

Hans Katsman

Van: Joakim Plate @.> Verzonden: maandag 12 december 2022 22:56 Aan: home-assistant/core @.> CC: Hans Katsman @.>; Author @.> Onderwerp: Re: [home-assistant/core] Nibe integration S1255 problems (Issue #83749)

You seen to have enabled the wrong entity? 32167 would match your modbus config.

— Reply to this email directly, view it on GitHub https://github.com/home-assistant/core/issues/83749#issuecomment-1347395479 , or unsubscribe https://github.com/notifications/unsubscribe-auth/AIAK6V26HIQYPHWVUENTBD3WM6NPBANCNFSM6AAAAAAS23HCZM . You are receiving this because you authored the thread. https://github.com/notifications/beacon/AIAK6V7BS6D3EV7U3L5XIIDWM6NPBA5CNFSM6AAAAAAS23HCZOWGG33NNVSW45C7OR4XAZNMJFZXG5LFINXW23LFNZ2KUY3PNVWWK3TUL5UWJTSQJ6PZO.gif Message ID: @. @.> >

HansKatsman commented 1 year ago

selecting 32167 solved the crash problem Thanks for your support