Zgłoszenia błędów [bramka ZigBee]

[klew]

W tych logach na starcie widać że odczytał korektę -3 stopnie. Potem jest zmiana na -2.5.
Natomiast korekta jest chyba aplikowana w momencie odczytu temperatury. Sprawdzę to jutro.

Ta metoda od korekcji wilgotności jest w termometrze, bo ona pochodzi z klasy bazowej i dla termometru jest ona ukryta i zastąpiona pustą implementacją.

[vajera]

W tych logach na starcie widać że odczytał korektę -3 stopnie. Potem jest zmiana na -2.5.
Natomiast korekta jest chyba aplikowana w momencie odczytu temperatury. Sprawdzę to jutro.

Ta metoda od korekcji wilgotności jest w termometrze, bo ona pochodzi z klasy bazowej i dla termometru jest ona ukryta i zastąpiona pustą implementacją.

Problem jest chyba tutaj?

https://github.com/SUPLA/supla-device/b ... er.cpp#L94

Code: Select all

// temperature correction
  auto cfgTempCorrection = getConfiguredTemperatureCorrection();
  double correction = 1.0 * cfgTempCorrection / 10.0;
  SUPLA_LOG_DEBUG("Channel[%d] temperature correction %.2f",
                  getChannelNumber(),
                  correction);
  // humidity correction
  auto cfgHumCorrection = getConfiguredHumidityCorrection();
  correction = 1.0 * cfgHumCorrection / 10.0;
  SUPLA_LOG_DEBUG(
      "Channel[%d] humidity correction %.2f", getChannelNumber(), correction);

  if (applyCorrections) {
    getChannel()->setCorrection(correction);    
    getChannel()->setCorrection(correction, true);
  }

if (applyCorrections) {
getChannel()->setCorrection(correction);
getChannel()->setCorrection(correction, true);
}

Zarówno temperatura jak i wilgotność dostają tę samą wartość korekcji, czyli zero gdy ustawiona jest tylko korekcja temperatury albo wartość korekcji wilgotności. To się zgadza z obserwacjami @endrju_88, @LukiSpajder i moimi własnymi.

EDIT - Potwierdzam - po poprawkach w kodzie SuplaDevice korekta działa prawidłowo, ale to oznacza, że @klew musiałby poprawić kod - to są 2 linie, więc nie bardzo jest sens PR?

EDIT2 - To oznacza również, że ta korekta nie działa prawidłowo na wszystkich czujnikach temperatury i wilgotności w SD.

[klew]

Zarówno temperatura jak i wilgotność dostają tę samą wartość korekcji, czyli zero gdy ustawiona jest tylko korekcja temperatury albo wartość korekcji wilgotności. To się zgadza z obserwacjami @endrju_88, @LukiSpajder i moimi własnymi.

EDIT - Potwierdzam - po poprawkach w kodzie SuplaDevice korekta działa prawidłowo, ale to oznacza, że @klew musiałby poprawić kod - to są 2 linie, więc nie bardzo jest sens PR?

EDIT2 - To oznacza również, że ta korekta nie działa prawidłowo na wszystkich czujnikach temperatury i wilgotności w SD.

Dzięki. Rzeczywiście to ostatnio zepsułem . Poprawka wrzucona do repo.
Dodałem teraz też bardziej rozbudowane testy i sprawdzam czy to faktycznie działa na kanałach VirtualThermometer i VirtualThermHygroMeter.

[vajera]

Cześć
Od jakiegoś czasu zaobserwowałem że moje gniazdka TS001F TZ3000 nie aktualizują mocy. Widać to na załączonych fotkach prąd się zmienia i jest w miarę prawidłowy.

Sprawdź proszę wersję 0.4.2.

[endrju_88]

@LukiSpajder
@endrju_88

Mam podejrzenie, że ma to związek z tym:

viewtopic.php?p=199712#p199712.

@Klew - widzialem sporo zmian w kodzie m. in. czujników temperatury I wilgotności a to jest jedyna różnica między wersjami 0.4.0 i 0.4.1 bramki Zigbee (pomijając ustawienia stosu Zigbee).
BTW. Funkcja

Code: Select all

void Supla::Sensor::Thermometer::setHumidityCorrection(int32_t correction);

w pliku thermometer.h/cpp na pewno powinna tam być?

@endrju_88, @bokk - To oznacza, że kompilator na GitHub nie zaciąga poprawek, które @klew zrobił dzisiaj w SuplaDevice - może trzeba podnieść wersję biblioteki SuplaDevice? U mnie kompiluje się prawidłowo, ale sam naniosłem poprawki w swojej kopii SD.

Czyli zostaje nam czekać na aktualizację oficjalną Supla Device ?

[vajera]

Czyli zostaje nam czekać na aktualizację oficjalną Supla Device ?

W katologu temp_bin masz 2 biny - merged dla 8 MB(z logami) i ino, który można wgrać po kablu na 4MB i 8MB albo jako OTA na 8MB

[endrju_88]

Czyli zostaje nam czekać na aktualizację oficjalną Supla Device ?

W katologu temp_bin masz 2 biny - merged dla 8 MB(z logami) i ino, który można wgrać po kablu na 4MB i 8MB albo jako OTA na 8MB

Wgrałem 8MB , korekty temperatur w Cloudzie działają i zmieniają wartości w Apk SUPLI. Dzięki

[wojtas567]

Sprawdź proszę wersję 0.4.2.

Wgrałem najnowszą 0.4.4 i nadal przekłamuje lub brak odczytu z licznika

[vajera]

Sprawdź proszę wersję 0.4.2.

Wgrałem najnowszą 0.4.4 i nadal przekłamuje lub brak odczytu z licznika

A na jakim obciążeniu robisz testy? Próbowałeś, np. suszarką?

[wojtas567]

A na jakim obciążeniu robisz testy? Próbowałeś, np. suszarką?

Większe akurat ekspresem i prąd pokazuje od razu jak grzałka grzeje a Waty są 0, chyba że potrzebuje dłuższej chwili żeby przeliczyć .
Jutro sprawdzę suszarką.