Pytania techniczne dotyczące kodu [bramka ZigBee]

[vajera]

@klew To znowu ja - mam taki przypadek:

1. dodaję do bramki nowe urządzenie Zigbee, które zaraz po sparowaniu radośnie wysyła do bramki raport o stanie baterii, więc bramka przekazuje go do Supla przez setBatteryLevel() i w aplikacji pojawia się ikona baterii,
2. użytkownik orientuje się, że co jest nie tak bo urządzenie jest przecież zasilanie z portu USB i nie ma żadnego akumulatora/baterii w środku,
3. z tego powodu dodałem flagę DISABLE_BATTERY_MSG,
4. tyle, że zapomniałem o fakcie, że serwer Supla przechowuje stan baterii, więc reset nic nie pomoże,
5. krótka analiza kodu setBatteryLevel() wyklucza możliwość użycia w tej funkcji wartości 0xFF - ona jej nie przyjmuje,
6. setBatteryPowered() też nie pozwala ustawić wartości batterypowered na 0 (unsed) tylko 1/2,
7. co prowadzi do wniosku, że takiej baterii nie da się "odzobaczyć", chyba, że użytkownik skasuje kanał z Cloud

[Robert Błaszczak]

Tak dokładnie jest. Ale po usunięciu z Cloud kanał chyba powinien dodać się sam ponownie już poprawnie.

[vajera]

Tak dokładnie jest. Ale po usunięciu z Cloud kanał chyba powinien dodać się sam ponownie już poprawnie.

Zgadza się, ale jak ktoś sobie już ten kanał skonfigurował to będzie narzekać

[Robert Błaszczak]

No niestety. Wygląda na to, że protokół SUPLA nie przewiduje takiego zachowania urządzenia.

[vajera]

No niestety. Wygląda na to, że protokół SUPLA nie przewiduje takiego zachowania urządzenia.

To nie jest kwestia protokołu tylko tej funkcji:

Code: Select all

bool Channel::isBatteryPoweredFieldEnabled() const {
  return batteryPowered != 0;
}

która zwaraca false, gdy batteryPowered == 0, ale ta funkcja

Code: Select all

void Channel::setBatteryPowered(bool value) {
  uint8_t newValue = value ? 1 : 2;
  if (newValue != batteryPowered) {
    SUPLA_LOG_DEBUG(
        "Channel[%d] battery powered changed to %d", channelNumber, newValue);
    batteryPowered = newValue;
    setSendStateInfo();
  }
}

nie pozwala wyzerować wartości batteryPowered .

Oczywiście mogę to zmienić w swojej kopii SuplaDevice, ale chciałbym zachować kompatybilność z biblioteką zwłaszcza, że większość kompilacji idzie przez GH.

[Zibi_007]

No niestety. Wygląda na to, że protokół SUPLA nie przewiduje takiego zachowania urządzenia.

To nie jest kwestia protokołu tylko tej funkcji:

Code: Select all

bool Channel::isBatteryPoweredFieldEnabled() const {
  return batteryPowered != 0;
}

która zwaraca false, gdy batteryPowered == 0, ale ta funkcja

Code: Select all

void Channel::setBatteryPowered(bool value) {
  uint8_t newValue = value ? 1 : 2;
  if (newValue != batteryPowered) {
    SUPLA_LOG_DEBUG(
        "Channel[%d] battery powered changed to %d", channelNumber, newValue);
    batteryPowered = newValue;
    setSendStateInfo();
  }
}

nie pozwala wyzerować wartości batteryPowered .

Oczywiście mogę to zmienić w swojej kopii SuplaDevice, ale chciałbym zachować kompatybilność z biblioteką zwłaszcza, że większość kompilacji idzie przez GH.

Może zrób "issue" a @Klew w kolejnej wersji biblioteki to uzupełni. Bo mnie to po prostu wygląda na drobne niedopatrzenie w produkcji. Funkcja baterii została wprowadzona nie tak dawno temu, to takie rzeczy mogą jeszcze wychodzić...

[klew]

Tak to było założone że nie da się tego skasować. Obecnie serwer to przechowuje i nie da się z poziomu urządzenia skasować baterii.

[Zenon]

Tak to było założone że nie da się tego skasować. Obecnie serwer to przechowuje i nie da się z poziomu urządzenia skasować baterii.

Witam!
Nikt po prostu nie przewidział genialnych pomysłów Kolegi @vajera który wprowadził Suplę na nowy poziom rozwoju. Coraz mniej urządzeń z Chin przychodzi na ESP którym szło wgrać soft Supli. Za to urządzeń Zigbee jest ogromna ilość w przystępnych cenach. Jakby nie bramką Z2S to na tym Forum byłoby chyba połowę postów mniej. @vajera tak trzymaj. Robisz świetną robotę.
Pozdrawiam
Zenon

[endrju_88]

Tak to było założone że nie da się tego skasować. Obecnie serwer to przechowuje i nie da się z poziomu urządzenia skasować baterii.

Witam!
Nikt po prostu nie przewidział genitalnych pomysłów Kolegi @vajera który wprowadził Suplę na nowy poziom rozwoju. Coraz mniej urządzeń z Chin przychodzi na ESP którym szło wgrać soft Supli. Za to urządzeń Zigbee jest ogromna ilość w przystępnych cenach. Jakby nie bramką Z2S to na tym Forum byłoby chyba połowę postów mniej. @vajera tak trzymaj. Robisz świetną robotę.
Pozdrawiam
Zenon

Jak nie można usunąć ikony baterii to nie można , w sumie to wina po stronie nie naszej a zza oceanu .Wypuszczają urządzenia o tym samym software do różnych urządzeń. Żenada. Żeby nie Z2S to nawet byśmy nie wiedzieli o takich wtopach. Popieram zdanie kolegi @Zenon - dobra robota

[Zibi_007]

Tak to było założone że nie da się tego skasować. Obecnie serwer to przechowuje i nie da się z poziomu urządzenia skasować baterii.

Witam!
Nikt po prostu nie przewidział genitalnych pomysłów Kolegi @vajera który wprowadził Suplę na nowy poziom rozwoju. Coraz mniej urządzeń z Chin przychodzi na ESP którym szło wgrać soft Supli. Za to urządzeń Zigbee jest ogromna ilość w przystępnych cenach. Jakby nie bramką Z2S to na tym Forum byłoby chyba połowę postów mniej. @vajera tak trzymaj. Robisz świetną robotę.
Pozdrawiam
Zenon

Jak nie można usunąć ikony baterii to nie można , w sumie to wina po stronie nie naszej a zza oceanu .Wypuszczają urządzenia o tym samym software do różnych urządzeń. Żenada. Żeby nie Z2S to nawet byśmy nie wiedzieli o takich wtopach. Popieram zdanie kolegi @Zenon - dobra robota

Baterie usunąć można bez większych problemów. @vajera nie chce po prostu stracić pełnej kompatybilności z biblioteką SD i stąd jego pytanie do @Klew...