home-assistant / core

:house_with_garden: Open source home automation that puts local control and privacy first.
https://www.home-assistant.io
Apache License 2.0
71.11k stars 29.79k forks source link

Solax inverter discovery error #36598

Closed riffy99 closed 3 years ago

riffy99 commented 4 years ago

The problem

I've configured a reverse proxy on a Pi and I'm able to retrieve the realtime stats via curl -X POST http://192.268.5.178/?optType=ReadRealTimeData from my Home Assistant server. However, when I add the sensor config in it fails with the below log entry.

Environment

Problem-relevant configuration.yaml

sensor:
  - platform: solax
    ip_address: 192.168.5.178

Traceback/Error logs

2020-06-08 14:22:32 ERROR (MainThread) [homeassistant.components.sensor] Error while setting up solax platform for sensor
Traceback (most recent call last):
  File "/usr/src/homeassistant/homeassistant/helpers/entity_platform.py", line 178, in _async_setup_platform
    await asyncio.wait_for(asyncio.shield(task), SLOW_SETUP_MAX_WAIT)
  File "/usr/local/lib/python3.7/asyncio/tasks.py", line 442, in wait_for
    return fut.result()
  File "/usr/src/homeassistant/homeassistant/components/solax/sensor.py", line 33, in async_setup_platform
    api = await real_time_api(config[CONF_IP_ADDRESS], config[CONF_PORT])
  File "/usr/local/lib/python3.7/site-packages/solax/__init__.py", line 37, in real_time_api
    i = await inverter.discover(ip_address, port)
  File "/usr/local/lib/python3.7/site-packages/solax/inverter.py", line 74, in discover
    raise DiscoveryError()
solax.inverter.DiscoveryError

Additional information

probot-home-assistant[bot] commented 4 years ago

Hey there @squishykid, mind taking a look at this issue as its been labeled with a integration (solax) you are listed as a codeowner for? Thanks! (message by CodeOwnersMention)

squishykid commented 4 years ago

Hi, could you please post a copy of the output you get from curl -X POST http://192.268.5.178/?optType=ReadRealTimeData

riffy99 commented 4 years ago

Sure thing: curl -X POST http://192.168.5.178/?optType=ReadRealTimeData {"type":"X1-Boost-Air-Mini","SN":"SW5PZWZCWS","ver":"2.32.6","Data":[3.2,0.0,217.5,0.0,2.7,250.2,655,39,7.2,3366.4,0,706,0,0.00,0.00,0,0,0,0.0,0.0,0.00,0.00,0,0,0,0.0,0.0,0.00,0.00,0,0,0,0.0,0.0,0,0,0,0,0,0,0,0.00,0.00,0,0,0,0,0,0,0,49.95,0,0,0,0,0,0,0,0,0,0.00,0,0,0,0,0.00,0,0,2],"Information":[2.500,4,"X1-Boost-Air-Mini","XAG252EA028145",1,3.25,1.09,1.10,0.00]}

squishykid commented 4 years ago

Ok, seems like we haven't added your inverter yet. Could you please follow the instructions here: https://github.com/squishykid/solax/wiki/DiscoveryError#inverter-not-supported-yet The section titled: "Inverter not Supported Yet"

stale[bot] commented 4 years ago

There hasn't been any activity on this issue recently. Due to the high number of incoming GitHub notifications, we have to clean some of the old issues, as many of them have already been resolved with the latest updates. Please make sure to update to the latest Home Assistant version and check if that solves the issue. Let us know if that works for you by adding a comment 👍 This issue now has been marked as stale and will be closed if no further activity occurs. Thank you for your contributions.