search

kellerza / sunsynk

Deye/Sunsynk Inverter Python library and Home Assistant OS Addon
https://kellerza.github.io/sunsynk/
MIT License
194 stars 84 forks source link

Charge Amps #222

Closed Leecavturbo closed 1 week ago

Leecavturbo commented 6 months ago

My inverter 5kw SS has Charge Amps and Grid Amps in battery settings. The sensor definition grid_charge_battery_current is the Grid Amps but I don’t see definition for Charge Amps. It is found ?

kellerza commented 5 months ago

Hi, please have a look at the modbus protocol document. Feel free to add custom sensors/mysensors to experiment with new registers

Leecavturbo commented 5 months ago

Thanks Johan. I found all bar 1 . Max sell power

On Mon, 29 Jan 2024 at 11:26, Johann Kellerman @.***> wrote:

Hi, please have a look at the modbus protocol document. Feel free to add custom sensors/mysensors to experiment with new registers

— Reply to this email directly, view it on GitHub https://github.com/kellerza/sunsynk/issues/222#issuecomment-1914496068, or unsubscribe https://github.com/notifications/unsubscribe-auth/A3VRXH72XO7GK2Z57JPEJVLYQ6BPPAVCNFSM6AAAAABBT7CKJCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSMJUGQ4TMMBWHA . You are receiving this because you authored the thread.Message ID: @.***>

kellerza commented 5 months ago

Did you do a custom sensor for max sell power? Do you mind sharing it?

Leecavturbo commented 5 months ago

Hi Johan, no it’s the one I can’t find ! I’ve tried max_sell , max_sell_power , sell_power and other possible combinations but no joy.

On Tue, 6 Feb 2024 at 18:25, Johann Kellerman @.***> wrote:

Did you do a custom sensor for max sell power? Do you mind sharing it?

— Reply to this email directly, view it on GitHub https://github.com/kellerza/sunsynk/issues/222#issuecomment-1930521207, or unsubscribe https://github.com/notifications/unsubscribe-auth/A3VRXH2UOTVCNWMLROYWKE3YSJYSTAVCNFSM6AAAAABBT7CKJCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSMZQGUZDCMRQG4 . You are receiving this because you authored the thread.Message ID: @.***>

rixxxx commented 3 months ago

This is not implemented for 1PH inverters. 1PH documentation I have doesn't list anything similar.

Leecavturbo commented 3 months ago

I have that working now !!! 1ph

On Tue, 9 Apr 2024 at 20:55, rivo nurges @.***> wrote:

This is not implemented for 1PH inverters. 1PH documentation I have doesn't list anything similar.

— Reply to this email directly, view it on GitHub https://github.com/kellerza/sunsynk/issues/222#issuecomment-2045950542, or unsubscribe https://github.com/notifications/unsubscribe-auth/A3VRXHZDXRECD7YVOWWAKVTY4RBSXAVCNFSM6AAAAABBT7CKJCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDANBVHE2TANJUGI . You are receiving this because you authored the thread.Message ID: @.***>

rixxxx commented 3 months ago

Please share what you did to get it to work?

Leecavturbo commented 3 months ago

Here is what I have . And result in Mqtt

On Tue, 9 Apr 2024 at 21:30, rivo nurges @.***> wrote:

Please share what you did to get it to work?

— Reply to this email directly, view it on GitHub https://github.com/kellerza/sunsynk/issues/222#issuecomment-2045998692, or unsubscribe https://github.com/notifications/unsubscribe-auth/A3VRXH4YKRECDZBBTXSAXCTY4RFWXAVCNFSM6AAAAABBT7CKJCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDANBVHE4TQNRZGI . You are receiving this because you authored the thread.Message ID: @.***>

Leecavturbo commented 3 months ago

I just can’t find and need Max Sell now and I’ll have every setting I need

On Tue, 9 Apr 2024 at 21:33, L Adams @.***> wrote:

Here is what I have . And result in Mqtt

On Tue, 9 Apr 2024 at 21:30, rivo nurges @.***> wrote:

Please share what you did to get it to work?

— Reply to this email directly, view it on GitHub https://github.com/kellerza/sunsynk/issues/222#issuecomment-2045998692, or unsubscribe https://github.com/notifications/unsubscribe-auth/A3VRXH4YKRECDZBBTXSAXCTY4RFWXAVCNFSM6AAAAABBT7CKJCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDANBVHE4TQNRZGI . You are receiving this because you authored the thread.Message ID: @.***>

rixxxx commented 3 months ago

Yeah, I'm talking about documentation lacking info about Max Sell.

Leecavturbo commented 3 months ago

Still stuck on that . I tried several obvious combinations of wording and still no joy

On Tue, 9 Apr 2024 at 21:44, rivo nurges @.***> wrote:

Yeah, I'm talking about documentation lacking info about Max Sell.

— Reply to this email directly, view it on GitHub https://github.com/kellerza/sunsynk/issues/222#issuecomment-2046018064, or unsubscribe https://github.com/notifications/unsubscribe-auth/A3VRXHZLU7NGK6HBUVDM7XDY4RHLHAVCNFSM6AAAAABBT7CKJCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDANBWGAYTQMBWGQ . You are receiving this because you authored the thread.Message ID: @.***>

rixxxx commented 3 months ago

As I said, there is no code to handle that and at least the documentation I have is lacking the regisgers to implement that.

jacekowski commented 1 month ago

`SENSORS += NumberRWSensor(53, "Max Solar Power", WATT, 1, min=0, max=10000) SENSORS += NumberRWSensor(245, "Max Sell Power", WATT, 1, min=0, max=10000)

`

kellerza commented 1 week ago

Thx @jacekowski - these are now in the 1ph definition files