The integration stopped to work after upgrade from "Home Assistant Core 2022.6.1" to "Home Assistant Core 2022.6.3".

gustavstrandberg commented 2 years ago

Version of the custom_component

Describe the bug

Debug log



Add your logs here.

Logger: custom_components.thermiagenesis
Source: custom_components/thermiagenesis/__init__.py:98
Integration: Thermia Genesis ([documentation](https://github.com/CJNE/thermiagenesis), [issues](https://github.com/CJNE/thermiagenesis/issues))
First occurred: 10:48:03 (4 occurrences)
Last logged: 10:49:07

Unexpected error fetching thermiagenesis data: 'bool' object is not callable
Traceback (most recent call last):
  File "/usr/src/homeassistant/homeassistant/helpers/update_coordinator.py", line 191, in _async_refresh
    self.data = await self._async_update_data()
  File "/config/custom_components/thermiagenesis/__init__.py", line 98, in _async_update_data
    data = await self.thermia.async_update(only_registers=registers)
  File "/usr/local/lib/python3.9/site-packages/pythermiagenesis/__init__.py", line 65, in async_update
    if not self._client.is_open():
TypeError: 'bool' object is not callable

Lord-Bam commented 2 years ago

I have the same issue upgrading from 2022.6.0 to 2022.6.3. Upgrading to 2022.6.4 did not resolve my issue.

I would add some logs, I but haven't figured out yet where to get them from.

gustavstrandberg commented 2 years ago

I have the same issue upgrading from 2022.6.0 to 2022.6.3. Upgrading to 2022.6.4 did not resolve my issue.

I would add some logs, I but haven't figured out yet where to get them from.

Hi!

You can fetch the logs from "Settings - System - Logs" and klick on the "Unexpected error fetching thermiagenesis data: 'bool' object is not callable"

/Gustav

Lord-Bam commented 2 years ago

thx for the tip. I disabled logging to reduce wear on my SD card and forgot about it :-)

The error message is identical.

Lord-Bam commented 2 years ago

When I reinstall the integration I get a "connection error" message.

my connection from custom.yaml still works:

modbus:
  - type: tcp
    name: Thermia
    host: 192.168.0.7
    port: 502
    sensors:
      - name: wp_outside_temp
        unit_of_measurement: °C
        address: 13
        input_type: input
        scan_interval: 300
        data_type: int16
        scale: 0.1

matake31 commented 2 years ago

Has anything changed with the arrival of 2022.6.5?

gustavstrandberg commented 2 years ago

I have upgraded to 2022.6.5 but still have the same issue. I have the same problem as Lord-Bam. A custom Modbus sensor in yaml works fine.

matake31 commented 2 years ago

I'v ben holding of on upgrading until there is a fix. But sooner or later I'll have to upgrade despite this problem.

gustavstrandberg commented 2 years ago

Ping @CJNE :-)

CJNE commented 2 years ago

Sorry for the delay, I'm a bit swamped with work at the moment. The issue was that pyModbusTCP had been updated with some breaking changes and the requirement wasn't pinned to a specific version. Version 0.0.9 is being released now which should fix this.

gustavstrandberg commented 2 years ago

No worries Johan! I did not mean to stress you! Great work as always!

gustavstrandberg commented 2 years ago

Confirmed working. Thanks @CJNE!

gustavstrandberg commented 2 years ago

@CJNE Försökte nå dig via mailen som står här på github men vet inte om det gick fram. Gör ett försök här om det är ok.

Hej Johan!

Jag har en fråga om pluginet thermogenesis som du byggt om det är ok.

Först och främst stort tack för den tid du lagt ner på ett lysande plugin. Det är väldigt användbart.

Jag har en dialog med en kompis på Thermias utvecklingsavdelning för genesis-pumparna angående smart styrning efter elpris som jag har implementerat på min pump med Home Assistant och ditt plugin.

Den climate-termostat (climate.tap_water) som skapas för Varmvatten inne HomeAssistant, vilka register skriver den till?

Mvh Gustav

CJNE commented 2 years ago

Hej Gustav!

Det låter som ett intressant projekt, jag var ute efter att göra något likande men det har inte blivit av, vore intressant att höra om du får det att fungera bra! Det skrivs till holding register 22 och 23, de som är med i din bild.

Med vänlig hälsning, Johan

30 juni 2022 kl. 10:44 skrev Gustav Strandberg @.***>:

@CJNE https://github.com/CJNE Försökte nå dig via mailen som står här på github men vet inte om det gick fram. Gör ett försök här om det är ok.

Hej Johan!

Jag har en fråga om pluginet thermogenesis https://github.com/CJNE/thermiagenesis som du byggt om det är ok.

Först och främst stort tack för den tid du lagt ner på ett lysande plugin. Det är väldigt användbart.

Jag har en dialog med en kompis på Thermias utvecklingsavdelning för genesis-pumparna angående smart styrning efter elpris som jag har implementerat på min pump med Home Assistant och ditt plugin.

Den climate-termostat (climate.tap_water) som skapas för Varmvatten inne HomeAssistant, vilka register skriver den till?

https://user-images.githubusercontent.com/38432101/176633828-f6ac062f-26dc-4023-a784-c1f6f4fb4e04.png Mvh Gustav

— Reply to this email directly, view it on GitHub https://github.com/CJNE/thermiagenesis/issues/105#issuecomment-1170939765, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAIASW3TH6CBA7UC3GNEI4DVRVM6RANCNFSM5YCEHWNA. You are receiving this because you were mentioned.

gustavstrandberg commented 2 years ago

@CJNE Jag delar gärna med mig om hur jag gjort och kan dema också om du vill. Det funkar riktigt bra och enligt offpeak.se så sparar jag i alla fall några kronor (kör Tibber). Hur når jag dig enklast så slipper vi skräpa ner den här tråden :-).

Mvh Gustav

CJNE commented 2 years ago

Det går bra att maila till @. @.>

Med vänlig hälsning, Johan

30 juni 2022 kl. 11:00 skrev Gustav Strandberg @.***>:

@CJNE https://github.com/CJNE Jag delar gärna med mig om hur jag gjort och kan dema också om du vill. Det funkar riktigt bra och enligt offpeak.se så sparar jag i alla fall några kronor (kör Tibber). Hur når jag dig enklast så slipper vi skräpa ner den här tråden :-).

Mvh Gustav

— Reply to this email directly, view it on GitHub https://github.com/CJNE/thermiagenesis/issues/105#issuecomment-1170956544, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAIASWYRZ6IF2FYC2LVCYRLVRVO3TANCNFSM5YCEHWNA. You are receiving this because you were mentioned.

gustavstrandberg commented 2 years ago

@CJNE Tyvärr ser jag ingen adress bara @. @.> :-)

matake31 commented 2 years ago

@CJNE Jag delar gärna med mig om hur jag gjort och kan dema också om du vill. Det funkar riktigt bra och enligt offpeak.se så sparar jag i alla fall några kronor (kör Tibber). Hur når jag dig enklast så slipper vi skräpa ner den här tråden :-).

Mvh Gustav

Det där vore intressant för mig med så fortsätt gärna i denna tråden :-D

gustavstrandberg commented 2 years ago

@CJNE Om det är ok fortsätter jag i denna tråden. Eller ska jag skapa en ny? Har en presentation (PDF) jag drog för Thermia som jag kan dela samt lite HA-kod för automatiseringar.

@matake31 Självklart får du också ta del av detta. Och kommer ni på bättre lösningar får ni gärna dela med er av dem tillbaka.

CJNE commented 2 years ago

@gustavstrandberg gör en ny issue så kan vi hålla den öppen och diskutera där

CJNE / thermiagenesis