Czujniki w bibliotece SuplaDevice

[dogu18]

mam problem ze znikającymi pomiarami z DTH22.
wcześniej ten problem miałem na mutiboard. po napisaniu nowego programu do sonoff basic z nową biblioteką miesiąc był spokój a od kilku dni znowu gubi się pomiar. Wyłączenie z zasilania i ponowne podpięcia rozwiązuje problem.

czy jest jakieś rozwiązanie tego problemu żeby ciągłe był pomiar?

pozdrawiam

[klew]

mam problem ze znikającymi pomiarami z DTH22.
wcześniej ten problem miałem na mutiboard. po napisaniu nowego programu do sonoff basic z nową biblioteką miesiąc był spokój a od kilku dni znowu gubi się pomiar. Wyłączenie z zasilania i ponowne podpięcia rozwiązuje problem.

czy jest jakieś rozwiązanie tego problemu żeby ciągłe był pomiar?

pozdrawiam

Masz jakieś logi z Seriala?
Odczyty znikają na stałe Aż do resetu, czy raz dział raz nie?

[dogu18]

Logów żadnych niemam niestety. Zanika stale aż do wyłączenia.

[klew]

Logów żadnych niemam niestety. Zanika stale aż do wyłączenia.

Jakiej płytki używasz? Wszystko inne działa poprawnie, tylko DHT przestaje czytać?
Jak masz podłączonego DHT? Jaka długość przewodów (mniej więcej)?

[dogu18]

Sonoff Basic pierwszej generacji. Przewód ekranowany około metra. Pod gpio1 podpięte teraz sprawdzałem

[lukfud]

@klew, co można/należy tu poprawić?

Kod: Zaznacz cały

#ifndef _bh1750_h
#define _bh1750_h

#include <BH1750.h>

#include "thermometer.h"

namespace Supla {
namespace Sensor {
class LightBH1750 : public Thermometer {
 public:
  LightBH1750() : sensorStatus(false) {
  }

  double getIntensity() {
    float value = TEMPERATURE_NOT_AVAILABLE;
    bool retryDone = false;
    do {
      if (!sensorStatus || isnan(value)) {
        sensorStatus = lightSensor.begin();
        retryDone = true;
      }
      value = TEMPERATURE_NOT_AVAILABLE;
      if (sensorStatus) {
        value = lightSensor.readLightLevel();
      }
    } while (isnan(value) && !retryDone);
    return value;
  }

  void onInit() {
    sensorStatus = lightSensor.begin();
    channel.setNewValue(getIntensity());
  }

 protected:
  bool sensorStatus;
  BH1750 lightSensor;  // I2C
};

};  // namespace Sensor
};  // namespace Supla

#endif

[klew]

@klew, co można/należy tu poprawić?

Zamień "getIntensity" na "getValue", bo tej metody używa termometr do robienia pomiaru co 10 s.

Ten "ifndef/define" - z reguły define'a robie zgodnego z nazwą pliku (z podkreślnikiem). U Ciebie wygląda tak jakby ten plik miał się nazywać tak samo jako include ten zewnętrznej biblioteki.

Dodaj w komentarzu opis/link skąd można pobrać zewnętrzną bibliotekę.

Ta metoda "getIntensity" może zwrócić NAN. Powinieneś na końcu jeszcze sprawdzić czy isnan(value) i wtedy value napidać tym TEMPERATURE_NOT_AVAILABLE.

W ostatnim kroku trzeba będzie podmienić kanał termometru na kanał ogólnego przeznaczenia - ale dopiero jak pojawi się dla niego wsparcie

[lukfud]

Faktycznie nazwa pliku jest tożsama z nazwą z biblioteki - zmienię (choć błędu przy kompilacji nie było)

Ta metoda "getIntensity" może zwrócić NAN. Powinieneś na końcu jeszcze sprawdzić czy isnan(value) i wtedy value napidać tym TEMPERATURE_NOT_AVAILABLE.

Metodę skopiowałem z BME, tam nie użyłeś tego warunku, nie był potrzebny?

Po kosmetycznych poprawkach.
Odczyt jest

Kod: Zaznacz cały

//https://github.com/claws/BH1750

#ifndef _lsbh1750_h
#define _lsbh1750_h

#include <BH1750.h>

#include "thermometer.h"

namespace Supla {
namespace Sensor {
class LSBH1750 : public Thermometer {
 public:
  LSBH1750() : sensorStatus(false) {
  }

  double getValue() {
    float lux = TEMPERATURE_NOT_AVAILABLE;
    bool retryDone = false;
    do {
      if (!sensorStatus || isnan(lux)) {
        sensorStatus = lightSensor.begin();
        retryDone = true;
      }
      lux = TEMPERATURE_NOT_AVAILABLE;
      if (sensorStatus) {
        lux = lightSensor.readLightLevel();
	  }
    } while (isnan(lux) && !retryDone);
    return lux;
  }

  void onInit() {
    sensorStatus = lightSensor.begin();
    channel.setNewValue(getValue());
  }

 protected:
  bool sensorStatus;
  BH1750 lightSensor;  // I2C
};

};  // namespace Sensor
};  // namespace Supla

#endif

[klew]

Metodę skopiowałem z BME, tam nie użyłeś tego warunku, nie był potrzebny?

Jest potrzebny. Dodam to

[lukfud]

Metodę skopiowałem z BME, tam nie użyłeś tego warunku, nie był potrzebny?

Jest potrzebny. Dodam to

Wstawiłem jeszcze korektę czasu pomiaru:

Kod: Zaznacz cały

/*

https://github.com/claws/BH1750

*/

#ifndef _lsbh1750_h
#define _lsbh1750_h

#include <BH1750.h>
#include "thermometer.h"

namespace Supla {
namespace Sensor {
class LSBH1750 : public Thermometer {
 public:
  LSBH1750() : sensorStatus(false) {
  }

  double getValue() {
    float lux = TEMPERATURE_NOT_AVAILABLE;
    bool retryDone = false;
    do {
      if (!sensorStatus || isnan(lux)) {
        sensorStatus = lightSensor.begin();
        retryDone = true;
      }
      lux = TEMPERATURE_NOT_AVAILABLE;
      if (sensorStatus) {
        lux = lightSensor.readLightLevel();
	  }
    } while (isnan(lux) && !retryDone);
	if (isnan(lux)) {
	  lux = TEMPERATURE_NOT_AVAILABLE;
	}
	if (lux > 40000.00) { 
	  lightSensor.setMTreg(32);
	}
	else if (lux > 10.00) {
	  lightSensor.setMTreg(69);
	}
    else if (lux <= 10.00) {
	  lightSensor.setMTreg(138);
	}
    return lux;
  }

  void onInit() {
    sensorStatus = lightSensor.begin();
    channel.setNewValue(getValue());
  }

 protected:
  bool sensorStatus;
  BH1750 lightSensor;  // I2C
};

};  // namespace Sensor
};  // namespace Supla

#endif