search

wills106 / homeassistant-solax-modbus

SolaX Power Modbus custom_component for Home Assistant (Supports some Ginlong Solis, Growatt, Sofar Solar, TIGO TSI & Qcells Q.Volt Hyb)
313 stars 96 forks source link

[Bug] Should the remotecontrol_xxx entites be visble on a X1 RetroFit (X1-Fit G4) #569

Open bengutt opened 11 months ago

bengutt commented 11 months ago

Describe the bug Unable to find most of the remotecontrol_xxx entites. I've tried looking for hidden and disabled ones.

I have number.solax_remotecontrol_autorepeat_duration, sensor.solax_remotecontrol_autorepeat_remaining and number.solax_remotecontrol_duration though.

From https://github.com/wills106/homeassistant-solax-modbus/wiki/Modbus-Power-Control-(remotecontrol-power)#why-use-modbus-power-control it suggests its only available on G4 Hybrids. Am I wrong to be hoping these should be visble to me?

Mandatory details

  1. Integration Version: 2023.10.1
  2. HA Version: 2023.10.4
  3. Inverter Brand: solax
  4. Plugin Used: Solax Modbus
  5. First 6 digits of Serial: F4502T
  6. Connection Method: Modbus via Serial

Detailed Error Log

I'm also seeing this a significant number of times in the log 17 per second for a couple of minutes, then a gap of approx 5 mins and then another batch. 2023-10-21 14:25:33.312 WARNING (MainThread) [homeassistant.util.async_] Detected blocking call to sleep inside the event loop. This is causing stability issues. Please report issue to the custom integration author for solax_modbus doing blocking calls at custom_components/solax_modbus/init.py, line 344: return self._client.read_holding_registers(address, count, **kwargs)

Additional context My serial number start was added earlier this year, see https://github.com/wills106/homeassistant-solax-modbus/discussions/26#discussioncomment-5333337

bengutt commented 6 days ago

I've spotted there is a similar issue raised #579 which has the same changes as I was thinking would make things work. I'd have a go at adding a pull request in, but I've got concerns about damaging my inverter by enabling and using something I shouldn't. What are the risks of turning things on and writing values that the inverter doesn't support?

I'd be looking to enable remotecontrol_trigger, remotecontrol_active_power and remotecontrol_power_control.

The logging error also reported is no longer present.

wills106 commented 6 days ago

What are the risks of turning things on and writing values that the inverter doesn't support?

Short answer is I don't know. I would expect the Inverter to either Ignore them or give an error.

The current remotecontrol function takes into account 'pv_power_total' in the calculations, so in it's current format it probably wouldn't work correctly.