Nowa funkcjonalność - lokalne akcje i warunkowanie [bramka ZigBee]

[vajera]

Sprawa nie jest też jakaś krytyczna. Chodzi tylko o to by przy restarcie bramki nie wykonywała się akcja dotycząca zmiany stanu czujnika/czujników binarnych no i VB wychodzi na to że też. Ale jeśli to jakaś większa robota to może tak jak wyżej napisałem. Zmienić logikę dla tego jednego przypadku i czekać czy ta sytuacja będzie dotyczyła też innych czujników w przyszłości.

Widzę następujące rozwiązanie - wydaje się być najbardziej uniwersalne i bezpieczne:

- dodam zdarzenia ON_ZIGBEE_IAS_MESSAGE_STATUS_ON i ON_ZIGBEE_IAS_MESSAGE_STATUS_OFF , które będą się generować jedynie, gdy urządzenie ZigBee prześle zmianę stanu.

W sumie to mógłbym też dodać akcje ON_ZIGBEE_IAS_MESSAGE_STATUS_TAMPER i ON_ZIGBEE_IAS_MESSAGE_STATUS_LOW_BATTERY i zrezygnować z tych dodatkowych czujników binarnych (TAMPER, LOW BATTERY).

[zzrr]

Sprawa nie jest też jakaś krytyczna. Chodzi tylko o to by przy restarcie bramki nie wykonywała się akcja dotycząca zmiany stanu czujnika/czujników binarnych no i VB wychodzi na to że też. Ale jeśli to jakaś większa robota to może tak jak wyżej napisałem. Zmienić logikę dla tego jednego przypadku i czekać czy ta sytuacja będzie dotyczyła też innych czujników w przyszłości.

Widzę następujące rozwiązanie - wydaje się być najbardziej uniwersalne i bezpieczne:

- dodam zdarzenia ON_ZIGBEE_IAS_MESSAGE_STATUS_ON i ON_ZIGBEE_IAS_MESSAGE_STATUS_OFF , które będą się generować jedynie, gdy urządzenie ZigBee prześle zmianę stanu.

W sumie to mógłbym też dodać akcje ON_ZIGBEE_IAS_MESSAGE_STATUS_TAMPER i ON_ZIGBEE_IAS_MESSAGE_STATUS_LOW_BATTERY i zrezygnować z tych dodatkowych czujników binarnych (TAMPER, LOW BATTERY).

No nie wiem. Wiadomo że Ty wybierasz zawsze to co najlepsze. Ale ja nie wiem czy tutaj warto iść aż tak daleko. Łukasz sprawdź proszę jeśli to nie kłopot czy w moim czujniku jak go dopisywałeś do bramki bo on był DIY czy bramka przypisuje mu stan OFF jak przychodzi raport że jest w stanie 0. To by tez zdaje sie załatwiło sprawę a przecież nikt inny tego nie zgłaszał. Ja to zgłosiłem w tym temacie bo ogólnie wykonywanie akcji dla zmiany stanu czujników binarnych przy restarcie bramki wydaje mi się zbędne, jednak przywrócenie stanu uważam za właściwe. Ale nie brałem pod uwagę że są to kwestie ze sobą powiązane. Nawet nie uważam tego za jakiś błąd jedynie za efekt zbędny. Bo w sumie po co akcja że czujnik wraca do swojego stanu sprzed restartu. Co do tych zaproponowanych akcji do dodania to ja sie przyznaję że nie do końca rozumiem ich zamysł.
EDIT:
I nie chodzi o PIR tylko ChangePowerMonitor (DIY_ZZRR::ChangePowerMonitor)

[zzrr]

@vajera a zamiast dodawać ON_ZIGBEE_IAS_MESSAGE_STATUS_ON i ..OFF itd. nie lepiej zmienić obecną akcję dla binary ON_TURN_ON i ON_TURN_OFF aby wykonywały się dokładnie tak jak masz w zamyśle jak wyżej tylko w przypadku kiedy przychodzi raport z urządzenia zigbee zmieniający jego stan skoro mówisz że tak się da?

[vajera]

@vajera a zamiast dodawać ON_ZIGBEE_IAS_MESSAGE_STATUS_ON i ..OFF itd. nie lepiej zmienić obecną akcję dla binary ON_TURN_ON i ON_TURN_OFF aby wykonywały się dokładnie tak jak masz w zamyśle jak wyżej tylko w przypadku kiedy przychodzi raport z urządzenia zigbee zmieniający jego stan skoro mówisz że tak się da?

Nie mogę, bo tych akcji nie wywołuje bramka a SuplaDevice - oczywiście mógłbym zmodyfikować kod SD na potrzeby bramki, ale później są kłopoty z utrzymaniem kompatybilności a @Krzysztof ciągle coś nowego tam dodaje

[vajera]

Łukasz sprawdź proszę jeśli to nie kłopot czy w moim czujniku jak go dopisywałeś do bramki bo on był DIY czy bramka przypisuje mu stan OFF jak przychodzi raport że jest w stanie 0.

To Ty możesz to sprawdzić, bo ja nie mam takiego czujnika - zrób logi ze startu bramki, to spojrzę.

Co do tych zaproponowanych akcji do dodania to ja sie przyznaję że nie do końca rozumiem ich zamysł.

Zamysł jest taki, że masz 100% pewności, że takie zdarzenie jest wywołane informacją z czujnika a nie np. restartem bramki albo inną akcją.

[klew]

Myślę że można zrobić tak aby odtworzenie stanu nie generowało reakcji lokalnych

[zzrr]

Myślę że można zrobić tak aby odtworzenie stanu nie generowało reakcji lokalnych

@klew Mów mów dalej, nie przerywaj...

[klew]

Myślę że można zrobić tak aby odtworzenie stanu nie generowało reakcji lokalnych

@klew Mów mów dalej, nie przerywaj...

Można też zrobić tak, aby nawet kolejne zmiany stanu nie generowały reakcji lokalnych

[vajera]

Można też zrobić tak, aby nawet kolejne zmiany stanu nie generowały reakcji lokalnych

Code: Select all

void SuplaDevice::Iterate() {

return;
...
}

[zzrr]

Myślę że można zrobić tak aby odtworzenie stanu nie generowało reakcji lokalnych

@klew Mów mów dalej, nie przerywaj...

Można też zrobić tak, aby nawet kolejne zmiany stanu nie generowały reakcji lokalnych

To już może lepiej zostańmy przy tym żeby zrobić a nie mówić dalej