Aktualizacja SuplaDevice

pzygmunt · Post Mon Oct 28, 2019 8:44 am

Serwer przy tej odpowiedzi zawsze dostosowuje się (nawet sztucznie) do wersji klienta tak aby mógł on zrozumieć odpowiedź.

https://github.com/SUPLA/supla-core/blo ... on.cpp#L64
https://github.com/SUPLA/supla-core/blo ... pc.c#L1243
https://github.com/SUPLA/supla-core/blo ... pc.c#L1177

Min i Max w odpowiedzi są zahardkodowane. Może ktoś coś przy tej kompilacji zmienił
https://github.com/SUPLA/supla-core/blo ... pc.c#L1240
https://github.com/SUPLA/supla-core/blo ... roto.h#L82

klew · Post Mon Oct 28, 2019 9:26 am

pzygmunt wrote: Mon Oct 28, 2019 8:44 am Min i Max w odpowiedzi są zahardkodowane. Może ktoś coś przy tej kompilacji zmienił

Dzięki za naprowadzenie

. To problematyczne "min_version" jest w "register device result". Jest chyba błąd w kodzie, bo jest tam wpisane SUPLA_PROTO_VERSION:
https://github.com/SUPLA/supla-core/blo ... e.cpp#L309

pzygmunt · Post Mon Oct 28, 2019 9:34 am

Jak już klient się zarejestruje to serwer wymaga od niego pracy na jednej wersji.

klew · Post Mon Oct 28, 2019 10:10 am

pzygmunt wrote: Mon Oct 28, 2019 9:34 am Jak już klient się zarejestruje to serwer wymaga od niego pracy na jednej wersji.

Ale tutaj serwer zwraca zawsze "SUPLA_PROTO_VERSION" (w version i w min_version), a nie wersję z którą się zarejestrowało urządzenie.
Zresztą, zdaje się, że ta kontrola wersji jest prawie nie sprawdzana. Przykładowo, wiele urządzeń rejestruje się z "7", a w odpowiedzi na rejestrację dostaje "version = 11, min_version = 11" i działa ;P

pzygmunt · Post Mon Oct 28, 2019 11:58 am

Rzeczywiście, akurat konkretnie ta odpowiedź w zakresie wersji jest mało istotna (raczej nigdzie nie jest weryfikowana). Nie przyjrzałem się temu, a pisałem ten fragment prawie 4 lata temu.
Zmieniłem.

cino111 · Post Tue Oct 29, 2019 6:34 am

@klew daj znać jak coś wymodzisz. Będę testerem, bo bardzo mi zależy na tej bibliotece pod Arduino Mega

Może dodasz od razu przyciski monostabilne i bistabilne, to przetestuje w pakiecie

klew · Post Tue Oct 29, 2019 8:23 am

cino111 wrote: Tue Oct 29, 2019 6:34 am @klew daj znać jak coś wymodzisz. Będę testerem, bo bardzo mi zależy na tej bibliotece pod Arduino Mega
Może dodasz od razu przyciski monostabilne i bistabilne, to przetestuje w pakiecie

Poprawka na wersję protokołu jest już na moim branchu develop. Jest też pull request otwarty, więc powinno wskoczyć do develop na oficjalnym repo.
Na przyciski trzeba będzie chwilę poczekać. Chcę to trochę ładniej zrobić, a to wymaga lekkiego przeorania całej biblioteki

.

cino111 · Post Tue Oct 29, 2019 9:50 am

Pobiorę sobie z https://github.com/klew/arduino/tree/de ... uplaDevice
a czy byś mógł połączyć już ze swoją biblioteką https://github.com/klew/arduino czyli licznikami impulsów i poprawkami DHT i DS10B20?

klew · Post Tue Oct 29, 2019 10:31 am

cino111 wrote: Tue Oct 29, 2019 9:50 am Pobiorę sobie z https://github.com/klew/arduino/tree/de ... uplaDevice
a czy byś mógł połączyć już ze swoją biblioteką https://github.com/klew/arduino czyli licznikami impulsów i poprawkami DHT i DS10B20?

Mogę wrzucić licznik impulsów. Obsługę DHT chcę się wstyrzmać, bo chcę zmienić cały interfejs do sensorów. Będą to zmiany, które będą wymagały przepisania przykładów i zmian w obecnych programach użytkowników, więc chciałbym ilość tych koniecznych poprawek w programach ograniczyć do minium. Jeśli teraz wrzucę "moje DHT", a potem to "docelowe DHT", to trzeba będzie dwa razy aktualizować programy

cino111 · Post Tue Oct 29, 2019 11:44 am

No to dodaj proszę liczniki. Potestuję sobie liczniki i przekaźniki na prywatnej instancji.