added power limit zero export in home assistant

[zorly57]

Hello, is it possible to add the ability to choose the power limit in Home Assistant rather than having to change the code with each update? (ecu-c)

ps: I'm not a coder, I modified this with chatgpt

[HAEdwin]

Sorry that it took me some time to dive into this. You can paste code between <> that's preferred instead of images.
Please refresh my memory on how the ECU-C works, I have not got one so you have to help me.
What is the reason for setting the default power_limit to 0.2?
There should already be an option to set the MaxPwr to the preferred value.

[zorly57]

Hello,
No problem, thank you for your work!

I have a Deye hybrid inverter in AC coupling with 8 APSystem micro inverters to charge the battery as well as a solar router. The problem with APSystem when the zero export is active is that it is not able to exploit 100% of the power of the panels connected to it, so I am obliged to deactivate the zero export every evening to reactivate it the next day when my battery is full and my solar router starts up but for it to work correctly I need to reinject a minimum into the network otherwise it drops to zero after a while.

Using the integration you developed, I can perfectly enable/disable zero export, but it automatically resets to 0 kW in my ECU-C. By fiddling with chatgpt, I managed to correct this in the code, but I haven't been able to integrate the ability to select this value into Home Assistant.
To date, it's impossible to update without losing this change, so I was wondering if you have the option to add this value modification to Home Assistant.

PS: I don't speak English and this is translated via Google Translate

Thanks.

[HAEdwin]

You did it right.
I understand again. The zero-export function works differently than the Maxpwr option. What I can do is add an additional number entity. That way, you can set the limit on the zero-export yourself between 0 and 3kW in 100W increments, so you can still set it to 0.2 kW. Then it's still usable for everyone, even those who want full zero-output. Would that be allright?

[zorly57]

That would be perfect. Thank you very much.

[HAEdwin]

@zorly57 I just released 2.13.0 🎉 So hopefully you can from now on upgrade easier. Read the release notes, this release was not tested with hardware, only software so I hope it works as expected. Please let me know.

[zorly57]

Hello,

Thank you for your work. I just ran some tests and encountered a problem.

When zero export is disabled on my ECU-C, if I enable zero export via Home Assistant, it is correctly taken into account with the value selected in the power limit field. However, when I disable zero export, still via Home Assistant, I get the following error: Failed to execute action switch/turn.set_zero_export(). Missing 1 required positional argument: 'power_limit'

The action is not executed in my ECU-C.

[zorly57]

By modifying the code of the "switch" file like this it works :

async def async_turn_off(self, **kwargs):
        """Turn off zero export."""
        try:
            await self._ecu.set_zero_export(0, 0)
            self._state = False
            self.async_write_ha_state()
        except (ConnectionError, TimeoutError) as e:
            _LOGGER.warning(
                "Failed to turn off zero export for ECU %s: %s", self._ecu.ecu.ecu_id, e
            )

Again a big thank you for your work!

[HAEdwin]

I just came across this loose end. I believe it's been addressed and resolved in the release. Therefore, I'm closing this thread, partly for my own clarity. If you have any questions or comments, please let me know.