pawelhulek / pgnig-sensor

This sensor is gathering gas usage data from PGNIG ebok page.
MIT License
32 stars 8 forks source link

pgnig_gas_sensor: Error on device update! #22

Closed tomhash2 closed 2 years ago

tomhash2 commented 2 years ago

W moim przypadku nie pokazuje się info o fakturze ponieważ:

2022-06-17 21:15:33 ERROR (MainThread) [homeassistant.components.sensor] pgnig_gas_sensor: Error on device update! Traceback (most recent call last): File "/usr/src/homeassistant/homeassistant/helpers/entity_platform.py", line 431, in _async_add_entity await entity.async_device_update(warning=False) File "/usr/src/homeassistant/homeassistant/helpers/entity.py", line 736, in async_device_update await task File "/config/custom_components/pgnig_gas_sensor/sensor.py", line 168, in async_update self._state = await self.hass.async_add_executor_job(self.invoices_summary) File "/usr/local/lib/python3.9/concurrent/futures/thread.py", line 58, in run result = self.fn(*self.args, **self.kwargs) File "/config/custom_components/pgnig_gas_sensor/sensor.py", line 179, in invoices_summary next_payment_item = min(filter(upcoming_payment_for_meter, self.api.invoices().invoices_list), key=lambda z: z, TypeError: '<' not supported between instances of 'InvoicesList' and 'InvoicesList'

Trudno mi powiedzieć dlaczego tak się dzieje, ale mam dwie faktury do zapłaty. Czy to może być powód? Tomek.

pawelhulek commented 2 years ago

@tomhash2 No to moze byc wina 2 faktur, bo akurat sie wywaliło na ich sortowaniu... Dzieki za zgloszenie w wolnej chwili rzuce okiem. Na szczescie juz ogarnałem testy jednostkowe w HA, wiec myśle ze uda sie zreprodukować.

luzik commented 2 years ago

u mnie to samo..

pawelhulek commented 2 years ago

@luzik @tomhash2 opublikowalem bete sensora pod tagiem: https://github.com/pawelhulek/pgnig-sensor/releases/tag/2.2.0-beta. Sprawdzicie prosze czy teraz ostatnia faktura poprawnie sie wyświetla?

luzik commented 2 years ago

HACS nie pokazuje bety

pawelhulek commented 2 years ago

@luzik Zobacz prosze w zakldce HACS -> PGNIG sensor -> ... w prawym gornym rogu - > redownload -> show beta version. Wtedy powinna sie pojawic wersja 2.2.0-beta

luzik commented 2 years ago

Mam przypisane do konta dwa liczniki. Stany liczydeł są poprawnie z obu. Faktura tylko jedna, druga pokazuje 0 i puste atrybuty

pawelhulek commented 2 years ago

@luzik a czy to sie zgadza ze stanem z eboku? Tam tez masz tylko 1 fakture?

luzik commented 2 years ago

Nie do końca rozumiem co powinienem widzieć. Mam dwie umowy. 1. a) boa Jak w boa filtruję fv z umowy A - widzę listę opłaconych faktur - bilans do zapłaty jest 0 bo wszystko opłacone. b) integracja wartosc 0, atrybyty puste

2. a) boa bilans XXX PLN b) integracja wartosc XXX, atrybuty uzupełnione

pawelhulek commented 2 years ago

No to jest OK w takim razie. Czesc faktura integracja pokazuje tylko faktury do zaplaty. Te oplacone nie bede wyswietlane.