Solaredge Modbus Configuration for Single Inverter and Battery

markpurcell · October 29, 2023, 1:34am

Looks like I have lost the ability to set site limit with the last update.

2.4.7-pre is bad

2.4.6 is good:

Issue raise, looks like a Type Error:

github.com/WillCodeForCats/solaredge-modbus-multi

Unable to set Site Limit with 2.4.7-pre - Type Error

opened 12:34AM - 30 Oct 23 UTC

purcell-lab

### Describe the bug https://community.home-assistant.io/t/solaredge-modbus-con…figuration-for-single-inverter-and-battery/464084/509?u=markpurcell With version 2.4.7-pre, I am unable to set the Site Limit entity. ![image](https://github.com/WillCodeForCats/solaredge-modbus-multi/assets/79175134/22d199d6-1032-4ca7-a58f-85e25235fc7d) ### Expected behavior With version 2.4.6 and earlier I can set the Site Limit entity. ### Screenshots ![image](https://github.com/WillCodeForCats/solaredge-modbus-multi/assets/79175134/8d74f1c8-962d-44a8-8799-f8b3527f45ab) ![image](https://github.com/WillCodeForCats/solaredge-modbus-multi/assets/79175134/ae365ed0-71ac-44d2-994f-84be14d55656) ### Diagnostic File [config_entry-solaredge_modbus_multi-0e4f1fd185e47f73b9a07d4577da5b46.json (1).txt](https://github.com/WillCodeForCats/solaredge-modbus-multi/files/13199676/config_entry-solaredge_modbus_multi-0e4f1fd185e47f73b9a07d4577da5b46.json.1.txt) ### Debug logs ```shell 2023-10-30 10:31:55.112 ERROR (MainThread) [homeassistant] Error doing job: Task exception was never retrieved Traceback (most recent call last): File "/usr/src/homeassistant/homeassistant/helpers/update_coordinator.py", line 233, in _handle_refresh_interval await self._async_refresh(log_failures=True, scheduled=True) File "/usr/src/homeassistant/homeassistant/helpers/update_coordinator.py", line 389, in _async_refresh self.async_update_listeners() File "/usr/src/homeassistant/homeassistant/helpers/update_coordinator.py", line 172, in async_update_listeners update_callback() File "/config/custom_components/solaredge_modbus_multi/number.py", line 101, in _handle_coordinator_update self.async_write_ha_state() File "/usr/src/homeassistant/homeassistant/helpers/entity.py", line 743, in async_write_ha_state self._async_write_ha_state() File "/usr/src/homeassistant/homeassistant/helpers/entity.py", line 843, in _async_write_ha_state state, attr = self._async_generate_attributes() ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ File "/usr/src/homeassistant/homeassistant/helpers/entity.py", line 783, in _async_generate_attributes available = self.available # only call self.available once per update cycle ^^^^^^^^^^^^^^ File "/config/custom_components/solaredge_modbus_multi/number.py", line 534, in available or self._platform.decoded_model["I_Power_Limit"] > 100 ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ TypeError: '>' not supported between instances of 'str' and 'int' ``` ### Home Assistant Version 2023.11.0b2 ### solaredge-modbus-multi Version 2.4.7-pre ### Installation Type Supervised ### Read the Instructions - [X] I have read https://github.com/WillCodeForCats/solaredge-modbus-multi/wiki - [X] My configuration follows https://github.com/WillCodeForCats/solaredge-modbus-multi/wiki/Configuration - [X] This issue is not about a template error (template errors should not be reported as an issue, read https://github.com/WillCodeForCats/solaredge-modbus-multi/wiki/Template-Design-Notes) ### Additional Context I use site limit control daily to manage negative prices and it is a great feature: When I have a negative feed in price, I set Export Control mode and site limit to 0W, which prevents my system from exporting power to the grid, but it varies solar production to cover my household load. When I have a negative general price, I set Production Control mode and site limit to 0W, which switches off all solar production and my house load runs 100% from the grid. ![image](https://github.com/WillCodeForCats/solaredge-modbus-multi/assets/79175134/bbd7f75f-cad5-48db-b813-7ee6d12dc045)