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
214 stars 57 forks source link

Solar Max Power #436

Closed devilliersjohnny closed 6 months ago

devilliersjohnny commented 6 months ago

Is there an existing issue for this?

Current Behavior

Hi Again...

You closed the previous issue that I opened with regards to the input of sensors within the options of the card.

Sadly this issue is still not resolved using the latest version of your card... the work around ( using the code editor ) works for the battery energy and and battery shutdown_soc, however the solar max_power does not seem to accept this format?

Could you please look into this

P.S thank you for the great work so far

Steps To Reproduce

No response

Expected behaviour

A working front end UI input not the back end code version

Card Version

4.37.1

Home Assistant Version

2024.5.4

Configuration

No response

Relevant log output

No response

slipx06 commented 6 months ago

Hi. The latest update allows you to configure a sensor for battery energy and and battery shutdown_soc. If you look at the screenshot below there are two new input fields on the right that allow you to select entities. Alternatively you can use the numeric selectors on the left. The two are mutually exclusive. Can you please test and let me know why this is not working for you?

image

Solar max_power does not currently support sensor input (unlike battery max_power) but I can add that.

slipx06 commented 6 months ago

To make this a bit clearer in the visual editor I'm adding an expandable optional sensors section to each of the card sections which can be used to provide sensors as an alternative to entering values in the options above. If you've provided a sensor the input field above will be empty as the selector is expecting a numeric value etc. If you provide a value above it pulls this through to the sensor field below.

image

slipx06 commented 6 months ago

@devilliersjohnny this has been implemented in https://github.com/slipx06/sunsynk-power-flow-card/releases/tag/v4.37.2

devilliersjohnny commented 6 months ago

@slipx06 THANK YOU! For indulging me... so far!!! For now I still have one more question? About the solar_max?

slipx06 commented 6 months ago

What do you mean be solar_max?

devilliersjohnny commented 6 months ago

Apologies should have been clearer, the ability to implement a sensor within the solar_max fields?

slipx06 commented 6 months ago

Do you mean

  pv1_max_power: 
  pv2_max_power: 
  pv3_max_power:
  pv4_max_power:
devilliersjohnny commented 6 months ago

Apologies for the rough edit but either the max_power total or the 4 pv_max powers that add up please!

max_power

slipx06 commented 6 months ago

You can already provide a sensor for max_power. I can add the others.

image image

devilliersjohnny commented 6 months ago

HI @slipx06 thank you can confirm that the max power appears to be working, I am currently using a template to calculate every thing for my end users...

  - name: "EDB I1 MPPT1 Max Power"
    unique_id: edb_i1_mppt1_max_power
    unit_of_measurement: "W"
    icon: mdi:solar-power
    device_class: power
    state: >
      {% if states('input_number.edb_inverter_count_input_number') | float >= 1
         and states('input_select.edb_i1_config_has_solar') == "True"
         and states('input_number.edb_i1_mppt_count_input_number') | float >= 1
         and states('input_text.edb_i1_mppt1_panel_size_input_text') | is_number
         and states('input_text.edb_i1_mppt1_panel_amount_input_text') | is_number %}
          {% set panel_size = states('input_text.edb_i1_mppt1_panel_size_input_text')  | float | round(0) %}
          {% set panel_count = states('input_text.edb_i1_mppt1_panel_amount_input_text')  | float | round(0) %}
          {% set mppt_max_power = panel_size * panel_count %}
        {% else %}
          {% set mppt_max_power = 'None' %}
      {% endif %}
      {{ mppt_max_power }}

would be nice if I could have the animation that you include ( the partial coloration ) working with my setup as well.

Again THANK YOU for all the work done so far!