Add data: use allData[*].Status to check if whole day is available #224
Conversation
Sprawdzanie czy w ostatnich 12 wpisach za dany dzień jest jakiś z niezerową wartością.
|
Obawiam się, że to ciągle nie wystarczy :/ potrzebne tu będzie coś bardziej złożonego |
|
Chyba zamiast patrzeć na ["values"], trzeba sprawdzić ["allData'] i tam każdy element ma |
|
@PiotrMachowski ok, przy normalnym widoku w eLiczniku (a nie zbilansowanym) mam dane do godziny 5. |
|
O, ciekawe znalezisko. Będę musiał sprawdzić, czy u mnie jest podobnie, bo jeśli faktycznie to przeoczyłem, to może to być rozwiązanie problemów |
Check if all elements in allData have status not equal to None/null
|
@PiotrMachowski ok, przy normalnym widoku w eLiczniku (a nie zbilansowanym) mam dane do godziny 5. Zmieniłem ten PR, ale uprzedzam że Python to nie język który znam, więc bazuję trochę na podpowiedziach AI. |
tomasz-soltysik
changed the title
|
Obliczanie zbilansowanych danych nie działa z tą zmianą. 
|
|
@PiotrMachowski nie wiem, czy powinienem tak robić, ale zwracam wartości z metody nawet jak są niepełne. Jedynie nie wrzucam ich do cache. I wykorzystuję |
|
@PiotrMachowski mam u siebie taką wersję od kilku dni i działa dobrze. Mam dane w karcie Energia, same się odświeżają. Nie ma błędów w logach. Jako że najnowsza wersja, po moich poprzednich zmianach, jest gorsza, to fajnie by było wrzucić coś co naprawi tą integrację. |
|
@tomasz-soltysik przez ostatnie 3 tygodnie byłem na wakacjach, teraz powoli wracam do tematu |
|
Minęło kilka kolejnych dni i cały czas wydaje się dobrze działać. Nie muszę nic ręcznie odświeżać, pojawiają mi się dane do 5 rano a potem na cały dzień. Zero błędów w logach. |
|
Dzięki, zmergowane 👍 |
|
Ok. A może dałoby się już zrobić release z tym fixem? Choćby wersję beta, żeby więcej osób mogło zainstalować i sprawdzić czy to rozwiązanie działa u każdego. |
Sprawdzanie czy w ostatnich 12 wpisach za dany dzień jest jakiś z niezerową wartością.
@PiotrMachowski Nie jestem pewny czy te 12 ostatnich wartości w kolekcji to faktycznie druga połowa dnia, ale wydaje się to logiczne.
Mam nadzieję że naprawi to #223, ale jeszcze nie miałem czasu przetestować.