slipx06 / sunsynk-power-flow-card

⚡A customizable Home Assistant card to emulate the Sunsynk System flow that's displayed on the Inverter screen.
MIT License
161 stars 48 forks source link

Load Box color switched for Load2 and Load3 #467

Closed Scoperius closed 4 weeks ago

Scoperius commented 4 weeks ago

Is there an existing issue for this?

Current Behavior

Box color for Load2 is based on Load3 value and for Load3 is based on Load2 value

Steps To Reproduce

Last version available for today. Used is Lite card version

Expected behaviour

Load2 box color based on Load2 value Load3 box color based on Load3 value

Card Version

Last

Home Assistant Version

Last

Configuration

No response

Relevant log output

No response

slipx06 commented 4 weeks ago

@Scoperius I cant reproduce this. Everything looks good on my end. Please share your config or more info

essential_load1: sensor.sunsynk_pv1_power
  essential_load2: sensor.sunsynk_pv2_power
  essential_load3: input_number.test_number
  essential_load4: sensor.sunsynk_pv3_power

image

Scoperius commented 4 weeks ago

Hi, I have only 3 loads. see the configuration:

load:
  show_daily: true
  additional_loads: 3
  load1_name: IVT AirX-130
  load1_icon: mdi:heat-pump-outline
  load2_name: 'Bazén '
  load2_icon: mdi:pool
  auto_scale: true
  dynamic_colour: true
  dynamic_icon: true
  load3_name: Výřivka
  load3_icon: mdi:pool-thermometer

image

  essential_load2: sensor.pl06_plug_tu_power
  essential_load2_extra: sensor.filtrace_spotreba
  essential_load3: sensor.vyrivka_denni_spotreba
slipx06 commented 4 weeks ago

Found the problem. Thanks for reporting

Scoperius commented 3 weeks ago

Now the box for Load2 is not colored at all.

Scoperius commented 3 weeks ago

Screenshot_20240604_142410.jpg

slipx06 commented 3 weeks ago

Have you defined an off_threshold? I can reproduce the problem. I'll investigate.

Its a rounding and conversion issue. The logic was only checking the state which is 0.3 which is being rounded to 0.

Scoperius commented 3 weeks ago

Hi,

No off_threshold set.

Petr

------ Původní zpráva ------ Od "slipx06" @.> Komu "slipx06/sunsynk-power-flow-card" @.> Kopie "Scoperius" @.>; "Mention" @.> Datum 04.06.2024 14:32:24 Předmět Re: [slipx06/sunsynk-power-flow-card] Load Box color switched for Load2 and Load3 (Issue #467)

Have you defined an off_threshold?

— Reply to this email directly, view it on GitHub https://github.com/slipx06/sunsynk-power-flow-card/issues/467#issuecomment-2147420522, or unsubscribe https://github.com/notifications/unsubscribe-auth/AUTWXLR3DASICDZW3YF5G7TZFWXVRAVCNFSM6AAAAABIVB4WPSVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCNBXGQZDANJSGI. You are receiving this because you were mentioned.Message ID: @.***>

Scoperius commented 3 weeks ago

When > 1 box color is back

------ Původní zpráva ------ Od "Petr Lustyk" @.> Komu "slipx06/sunsynk-power-flow-card" @.>; "slipx06/sunsynk-power-flow-card" @.> Kopie "Scoperius" @.>; "Mention" @.***> Datum 04.06.2024 19:03:44 Předmět Re[2]: [slipx06/sunsynk-power-flow-card] Load Box color switched for Load2 and Load3 (Issue #467)

Hi,

No off_threshold set.

Petr

------ Původní zpráva ------ Od "slipx06" @.> Komu "slipx06/sunsynk-power-flow-card" @.> Kopie "Scoperius" @.>; "Mention" @.> Datum 04.06.2024 14:32:24 Předmět Re: [slipx06/sunsynk-power-flow-card] Load Box color switched for Load2 and Load3 (Issue #467)

Have you defined an off_threshold?

— Reply to this email directly, view it on GitHub https://github.com/slipx06/sunsynk-power-flow-card/issues/467#issuecomment-2147420522, or unsubscribe https://github.com/notifications/unsubscribe-auth/AUTWXLR3DASICDZW3YF5G7TZFWXVRAVCNFSM6AAAAABIVB4WPSVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCNBXGQZDANJSGI. You are receiving this because you were mentioned.Message ID: @.***>

slipx06 commented 3 weeks ago

It depends on your unit of measurement. The calculation will always try convert to W or Wh before checking if > than off_threshold. If you don't provide an expected unit of measurement i.e. W, kW, Wh, kWh then it will simply round that value and then do the calculation i.e. 01-0.4 will round down to 0, 0.5-0.9 will round up to 1.

purcell-lab commented 3 weeks ago

Thanks for this PR. I was just about to implement a off threshold in my power sensor template for each of my devices, but I see this capability is inbuilt.

Brilliant!

Scoperius commented 3 weeks ago

It will be nice to have possibility to set off_treshold for some decimal points, for instance 0.1 will suit my personal need.

------ Původní zpráva ------ Od "Mark Purcell" @.> Komu "slipx06/sunsynk-power-flow-card" @.> Kopie "Scoperius" @.>; "Mention" @.> Datum 05.06.2024 0:43:16 Předmět Re: [slipx06/sunsynk-power-flow-card] Load Box color switched for Load2 and Load3 (Issue #467)

Thanks for this PR. I was just about to implement a off threshold in my power sensor template for each of my devices, but I see this capability is inbuilt.

Brilliant!

— Reply to this email directly, view it on GitHub https://github.com/slipx06/sunsynk-power-flow-card/issues/467#issuecomment-2148529267, or unsubscribe https://github.com/notifications/unsubscribe-auth/AUTWXLTZWE4NZSR6ER5DXNDZFY7IJAVCNFSM6AAAAABIVB4WPSVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCNBYGUZDSMRWG4. You are receiving this because you were mentioned.Message ID: @.***>