search

littleyoda / ha-pysmaplus

home assistant custom integration for pysma-plus
23 stars 1 forks source link

No connection to Sunny Tripower 4.0 after update #15

Closed DeltaTango69 closed 1 month ago

DeltaTango69 commented 2 months ago

Used Version of Home Assistant:2024.8.2 Used Version of ha-pysmaplus:04.1

Describe the bug, Remarks, ... A clear and concise description of what the bug is.

No connection

Used Interface/Access Method see https://raw.githubusercontent.com/littleyoda/ha-pysmaplus/main/docs/images/interfaces.png Speedwire: [x] Webconnect: [x ] EnnexOS-based: [x ] Energymeter/SHM2: [ ]

all tried

List of the integrated SMA devices STP 15000TL-30

Screenshots and Diagnose Information image

2024-08-25 15:38:25 homeassistant  | 2024-08-25 15:38:25.626 ERROR (MainThread) [homeassistant.config_entries] Error setting up entry STP4.0-3AV-40 (Sunny Tripower 4.0) (3012423268) for pysmaplus
2024-08-25 15:38:25 homeassistant  | Traceback (most recent call last):
2024-08-25 15:38:25 homeassistant  |   File "/usr/src/homeassistant/homeassistant/config_entries.py", line 604, in async_setup
2024-08-25 15:38:25 homeassistant  |     result = await component.async_setup_entry(hass, self)
2024-08-25 15:38:25 homeassistant  |              ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
2024-08-25 15:38:25 homeassistant  |   File "/config/custom_components/pysmaplus/__init__.py", line 91, in async_setup_entry
2024-08-25 15:38:25 homeassistant  |     sma = await getPysmaInstance(hass, entry.data)
2024-08-25 15:38:25 homeassistant  |           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
2024-08-25 15:38:25 homeassistant  |   File "/config/custom_components/pysmaplus/__init__.py", line 84, in getPysmaInstance
2024-08-25 15:38:25 homeassistant  |     await sma.new_session()
2024-08-25 15:38:25 homeassistant  |   File "/usr/local/lib/python3.12/site-packages/pysmaplus/device_speedwire.py", line 426, in new_session
2024-08-25 15:38:25 homeassistant  |     raise SmaConnectionException(
2024-08-25 15:38:25 homeassistant  | pysmaplus.exceptions.SmaConnectionException: No connection to device: 192.168.178.10:9522  (3/3)
DeltaTango69 commented 2 months ago

Wenn ich "Neu laden" drücke sehe ich kurz das die Verbindung zu 192.168.178.10:9552 nicht funktioniert. Woher kommen die 9552? Wenn ich die Adresse so im Browser eingebe bekomme ich auch keine Verbindung. Nur ohne Port!

Wo wird denn eigentlich die Konfiguration gespeichert?

Vor dem Update habe ich mir eine Kopie von custom_components/pysmaplus.. gemacht. Die habe ich jetzt einfach zurückgespielt - jetzt geht es wieder. Im manifest.json steht das:

{
  "domain": "pysmaplus",
  "name": "SMA Devices Plus (not for production use)",
  "codeowners": ["@littleyoda", "@kellerza", "@rklomp"],
  "config_flow": true,
  "documentation": "https://github.com/littleyoda/ha-pysmaplus",
  "integration_type": "device",
  "iot_class": "local_polling",
  "issue_tracker": "https://github.com/littleyoda/ha-pysmaplus/issues",
  "loggers": ["pysma"],
  "requirements": ["pysma-plus>=0.3.1"],
  "version": "0.3.1"
}
littleyoda commented 2 months ago

Sehe ich es richtig, dass die Verbindung über webconnect funktioniert, "nur" Speedwire nicht funktioniert?

Webconnect läuft über http/s (=> Browser) Speedwire läuft aber über UDP Port 9552. Deshalb hast du darauf gar keinen Einfluss.

Wenn Webconnect funktioniert, dann bleibt bei Webconnect.

Speedwire sollte eigentlich immer funktionieren. Von daher ist es schon komisch. Wenn du eine Linux-Maschine im Netzwerk verfügbar hast, könnten wir uns etwas probieren. Melde dich dann per Mail bei mir. https://littleyoda.github.io/

EDIT: Sehe gerade, dass du im Betreff geschrieben hast, dass es nach dem Update auftritt. Funktioniert es vor?

DeltaTango69 commented 2 months ago

Nein es funktionierte nach dem Update gar keine Verbindung mehr. Vorher schon. Bei mir klappt generell nur die Speedwire Verbindung. Ich habe alles unter Windows WSL2 mit Docker Compose am laufen.

littleyoda commented 2 months ago

Ok. Das muss ich jetzt etwas sortieren:

DeltaTango69 commented 2 months ago

Ok, ich versuche Mal Licht ins Dunkel zu bringen :-)

1) Bei mir funktioniert generell nur die Speedwire Verbindung 2) Vor dem Update funktionierten beide Wechselrichter damit - nach dem Update nur noch der STP-15000TL-30 (siehe Bild oben) 3) Ich habe Homeassistant aktualisiert und dann die HACS Integration von SMA von 0.3.1 auf 0.4.1

littleyoda commented 2 months ago

Du bist der zweite mit so einem Fehlerbild. Ich weiß leider aktuell nicht, wo das Problem ist.

Du könntest mit Hilfe von HACS erst mal wieder die 0.3.1 installieren.

EDIT: Wenn du dann noch Nerven hast, könntest du dich mal vorsichtig eine Version nach der anderen hoch tasten. Mir würde es ggf. sehr zum Eingrenzen des Fehlers helfen, wenn du mir genau sagen könntest, ab welcher Version es nicht mehr funktioniert.

DeltaTango69 commented 2 months ago

Ich habe es mal durchprobiert. Die 0.3.4 geht noch , ab der 0.3.5 nicht mehr. Gruß Dirk

tonyhammainen commented 1 month ago

Hi @littleyoda , continuing the discussion from home assistant forum, as I hit a "new user cant send more than 3 messages" limit

  1. I had successfully added the devices with 0.3.4. Upgrading to 0.4.2 didn't break the old ones, so while the device adding may have problems, existing devices stay functional.
  2. Running service discovery failed; didn't produce output
  3. Trying to add a device fails

I see this error in logs:

2024-09-04 13:13:52.921 ERROR (MainThread) [homeassistant.helpers.script.websocket_api_script] websocket_api script: Error executing script. Unexpected error for call_service at pos 1: cannot access local variable 'di' where it is not associated with a value
Traceback (most recent call last):
File "/usr/local/lib/python3.12/site-packages/pysmaplus/device_speedwire.py", line 545, in detect
raise SmaConnectionException("protocol not initialized")
pysmaplus.exceptions.SmaConnectionException: protocol not initialized
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "/usr/src/homeassistant/homeassistant/helpers/script.py", line 527, in _async_step
await getattr(self, handler)()
File "/usr/src/homeassistant/homeassistant/helpers/script.py", line 764, in _async_call_service_step
response_data = await self._async_run_long_action(
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/usr/src/homeassistant/homeassistant/helpers/script.py", line 727, in _async_run_long_action
return await long_task
^^^^^^^^^^^^^^^
File "/usr/src/homeassistant/homeassistant/core.py", line 2731, in async_call
response_data = await coro
^^^^^^^^^^
File "/usr/src/homeassistant/homeassistant/core.py", line 2774, in _execute_service
return await target(service_call)
^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/config/custom_components/pysmaplus/services.py", line 157, in discovery
ident = await identify(r[0], False)
^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/config/custom_components/pysmaplus/services.py", line 136, in identify
ret = await pysma.autoDetect(session, url)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/usr/local/lib/python3.12/site-packages/pysmaplus/__init__.py", line 90, in autoDetect
ret = await asyncio.gather(
^^^^^^^^^^^^^^^^^^^^^
File "/usr/local/lib/python3.12/site-packages/pysmaplus/__init__.py", line 77, in _run_detect
ret = await sma.detect(ip)
^^^^^^^^^^^^^^^^^^^^
File "/usr/local/lib/python3.12/site-packages/pysmaplus/device_speedwire.py", line 566, in detect
di.status = "failed"
^^
UnboundLocalError: cannot access local variable 'di' where it is not associated with a value

This unbound error should be fixed by moving the DiscoveryInformation initialization outside of the try except block. (below)

The root issue still is the fact that ._protocol is not initialized. However, looking at the code, I think the if statement checking if ._protocolexists only makes sense to be after .new_session(), since ._protocolis initialized as part of .new_session().

So both together:

    async def detect(self, ip: str) -> list[DiscoveryInformation]:

        di = DiscoveryInformation()
        di.tested_endpoints = str(ip) + ":9522"
        try:            
            await self.new_session()
            if self._protocol is None:
                raise SmaConnectionException("protocol not initialized")
            fut = asyncio.get_running_loop().create_future()
            await self._protocol.start_query(["TypeLabel"], fut, self._group)
            try:
                await asyncio.wait_for(fut, timeout=5)
            except TimeoutError:
                _LOGGER.warning("Timeout in detect")
            if (
                "error" in self._protocol.data_values
                and self._protocol.data_values["error"] == 0
            ):
                raise SmaReadException("Reply for request not received")
            raise SmaConnectionException("No connection to device")
        except SmaAuthenticationException as e:
            di.status = "maybe"
            di.exception = e
            di.remark = "only unencrypted Speedwire is supported"
        except Exception as e:
            di.status = "failed"
            di.exception = e
        return [di]
littleyoda commented 1 month ago

Can you please retest the latest version (0.4.3)? I hope that I have corrected the problems.

littleyoda commented 1 month ago

There is now a consolidated bug for the problems with Speedwire. #18