Skip to content
  • MySensors
  • OpenHardware.io
  • Categories
  • Recent
  • Tags
  • Popular
Skins
  • Light
  • Brite
  • Cerulean
  • Cosmo
  • Flatly
  • Journal
  • Litera
  • Lumen
  • Lux
  • Materia
  • Minty
  • Morph
  • Pulse
  • Sandstone
  • Simplex
  • Sketchy
  • Spacelab
  • United
  • Yeti
  • Zephyr
  • Dark
  • Cyborg
  • Darkly
  • Quartz
  • Slate
  • Solar
  • Superhero
  • Vapor

  • Default (No Skin)
  • No Skin
Collapse
Brand Logo
  1. Home
  2. Troubleshooting
  3. Multisensor PIR problem [solved]

Multisensor PIR problem [solved]

Scheduled Pinned Locked Moved Troubleshooting
53 Posts 7 Posters 26.3k Views 3 Watching
  • Oldest to Newest
  • Newest to Oldest
  • Most Votes
Reply
  • Reply as topic
Log in to reply
This topic has been deleted. Only users with topic management privileges can see it.
  • T torfinn

    Hi ! I had the same problem. Seems like gw.sleep(some_number_of_mills); lets you loose the init of the dht. Just put dht.begin(); in your loop before you read the value and it should work.

    example.
    void loop ()
    {
    dht.begin();
    float temperature = dht.readTemperature();
    ......

    N Offline
    N Offline
    n3ro
    wrote on last edited by
    #11

    @torfinn said:

    dht.begin();

    Hey torfinn,

    i get this error when compiling:
    MotionSensor_DHT_light_battery_mod.ino: In function 'void loop()':
    MotionSensor_DHT_light_battery_mod.ino:109:7: error: 'class DHT' has no member named 'begin'
    Fehler beim Kompilieren.

    pimatic + MySensors + Homeduino + z-way
    https://github.com/n3roGit/MySensors_n3ro

    1 Reply Last reply
    0
    • N Offline
      N Offline
      n3ro
      wrote on last edited by n3ro
      #12

      I have a idea,

      is it possible to build a second loop like this?
      But now i have one problem... the loop2 never run...

      #include <MySensor.h>
      #include <readVcc.h>
      #include <SPI.h>
      #include <DHT.h>
      
      #include <Scheduler.h>
      
      
      
      
      #define NODE_ID 11                       // ID of node
      unsigned long SLEEP_TIME = 600000;        // Sleep time between reports (in milliseconds)
      
      #define CHILD_ID_PIR 1                   // Id of the sensor PIR
      #define CHILD_ID_HUM 2                   // Id of the sensor HUM
      #define CHILD_ID_TEMP 3                  // Id of the sensor TEMP
      #define CHILD_ID_LIGHT 4                 // Id of the sensor LIGHT
      
      #define PIR_SENSOR_DIGITAL 3           // The digital input you attached your motion sensor.  (Only 2 and 3 generates interrupt!)
      #define INTERRUPT PIR_SENSOR_DIGITAL-2 // Usually the interrupt = pin -2 (on uno/nano anyway)
      #define HUMIDITY_SENSOR_DIGITAL_PIN 2
      #define LIGHT_SENSOR_ANALOG_PIN 0
      
      MySensor gw;
      // Initialize Variables
      MyMessage msgPir(CHILD_ID_PIR, V_TRIPPED);
      MyMessage msgHum(CHILD_ID_HUM, V_HUM);
      MyMessage msgTemp(CHILD_ID_TEMP, V_TEMP);
      MyMessage msgLight(CHILD_ID_LIGHT, V_LIGHT_LEVEL);
      
      DHT dht;
      float lastTemp;
      float lastHum;
      boolean metric = true;
      int oldBatteryPcnt;
      int lastLightLevel;
      unsigned long time;
      
      
      int MIN_V = 2700; // empty voltage (0%)
      int MAX_V = 3200; // full voltage (100%)
      
      void setup()
      {
        gw.begin(NULL, NODE_ID, false);
      
        //PIR
        // Send the sketch version information to the gateway and Controller
        gw.sendSketchInfo("Motion Sensor", "1.0");
      
        pinMode(PIR_SENSOR_DIGITAL, INPUT);      // sets the motion sensor digital pin as input
        digitalWrite(PIR_SENSOR_DIGITAL, HIGH);
        // Register all sensors to gw (they will be created as child devices)
        gw.present(CHILD_ID_PIR, S_MOTION);
      
        //DHT
        dht.setup(HUMIDITY_SENSOR_DIGITAL_PIN);
      
        // Send the Sketch Version Information to the Gateway
        gw.sendSketchInfo("Humidity", "1.0");
      
        // Register all sensors to gw (they will be created as child devices)
        gw.present(CHILD_ID_HUM, S_HUM);
        gw.present(CHILD_ID_TEMP, S_TEMP);
      
        metric = gw.getConfig().isMetric;
      
        //LIGHT
        // Send the sketch version information to the gateway and Controller
        gw.sendSketchInfo("Light Sensor", "1.0");
      
        // Register all sensors to gateway (they will be created as child devices)
        gw.present(CHILD_ID_LIGHT, S_LIGHT_LEVEL);
        
        Scheduler.startLoop(loop2);
      
      }
      
      void loop()
      {
        // Read digital motion value
        boolean tripped = digitalRead(PIR_SENSOR_DIGITAL) == HIGH;
        Serial.print("---------- PIR: ");
        Serial.println(tripped);
        gw.send(msgPir.set(tripped ? "1" : "0")); // Send tripped value to gw
      
        // Sleep until interrupt comes in on motion sensor. Send update every two minute.
        gw.sleep(INTERRUPT, CHANGE, SLEEP_TIME);
      }
      void loop2()
      {
        //DHT
        //dht.begin();
        delay(dht.getMinimumSamplingPeriod());
        //delay(1000);
        //millis(dht.getMinimumSamplingPeriod());
        //Serial.print(dht.getMinimumSamplingPeriod());
      
        float temperature = dht.getTemperature();
        if (isnan(temperature)) {
          Serial.println("Failed reading temperature from DHT");
        } else if (temperature != lastTemp) {
          lastTemp = temperature;
          if (!metric) {
            temperature = dht.toFahrenheit(temperature);
          }
          gw.send(msgTemp.set(temperature, 1));
          Serial.print("---------- Temp: ");
          Serial.println(temperature);
        }
      
        float humidity = dht.getHumidity();
        if (isnan(humidity)) {
          Serial.println("Failed reading humidity from DHT");
        } else if (humidity != lastHum) {
          lastHum = humidity;
          gw.send(msgHum.set(humidity, 1));
          Serial.print("---------- Humidity: ");
          Serial.println(humidity);
        }
      
        // Light
        int lightLevel = (1023 - analogRead(LIGHT_SENSOR_ANALOG_PIN)) / 10.23;
        //Serial.println(lightLevel);
        if (lightLevel != lastLightLevel) {
          gw.send(msgLight.set(lightLevel));
          lastLightLevel = lightLevel;
          Serial.print("---------- Light: ");
          Serial.println(lightLevel);
        }
      
        // Measure battery
        float batteryV = readVcc();
        int batteryPcnt = (((batteryV - MIN_V) / (MAX_V - MIN_V)) * 100 );
        if (batteryPcnt > 100) {
          batteryPcnt = 100;
        }
      
        if (batteryPcnt != oldBatteryPcnt) {
          gw.sendBatteryLevel(batteryPcnt); // Send battery percentage
          oldBatteryPcnt = batteryPcnt;
        }
        Serial.print("---------- Battery: ");
        Serial.println(batteryPcnt);
        gw.sleep(SLEEP_TIME);
      }
      

      i have downloaded THIS

      But i get this error when compiling:

      In file included from MotionSensor_DHT_light_battery_mod.ino:6:0:
      Arduino\libraries\Scheduler/Scheduler.h:33:22: fatal error: WProgram.h: No such file or directory
      #include <WProgram.h>

      pimatic + MySensors + Homeduino + z-way
      https://github.com/n3roGit/MySensors_n3ro

      1 Reply Last reply
      0
      • R Offline
        R Offline
        rvendrame
        Hero Member
        wrote on last edited by
        #13

        @n3ro , you can have only one loop().

        You need to merge the two logics into into a single loop. You might also want to remove the second sleep, otherwise PIR will not trigger there.

        Home Assistant / Vera Plus UI7
        ESP8266 GW + mySensors 2.3.2
        Alexa / Google Home

        N 1 Reply Last reply
        0
        • R rvendrame

          @n3ro , you can have only one loop().

          You need to merge the two logics into into a single loop. You might also want to remove the second sleep, otherwise PIR will not trigger there.

          N Offline
          N Offline
          n3ro
          wrote on last edited by
          #14

          @rvendrame do you have an idea how to do it?

          pimatic + MySensors + Homeduino + z-way
          https://github.com/n3roGit/MySensors_n3ro

          1 Reply Last reply
          0
          • R Offline
            R Offline
            rvendrame
            Hero Member
            wrote on last edited by
            #15

            Maybe you can start by to removing these 3 lines:

            }
            void loop2()
            {
            
            

            ... and move the 1st sleep to the end of loop (where the 2nd sleep is):

            // Sleep until interrupt comes in on motion sensor. Send update every two minute.
             gw.sleep(INTERRUPT, CHANGE, SLEEP_TIME);
            
            

            .... and remove the 2nd sleep:

            gw.sleep(SLEEP_TIME);
            

            Home Assistant / Vera Plus UI7
            ESP8266 GW + mySensors 2.3.2
            Alexa / Google Home

            N 1 Reply Last reply
            0
            • R rvendrame

              Maybe you can start by to removing these 3 lines:

              }
              void loop2()
              {
              
              

              ... and move the 1st sleep to the end of loop (where the 2nd sleep is):

              // Sleep until interrupt comes in on motion sensor. Send update every two minute.
               gw.sleep(INTERRUPT, CHANGE, SLEEP_TIME);
              
              

              .... and remove the 2nd sleep:

              gw.sleep(SLEEP_TIME);
              
              N Offline
              N Offline
              n3ro
              wrote on last edited by
              #16

              @rvendrame
              if i use this:

              #include <MySensor.h>
              #include <readVcc.h>
              #include <SPI.h>
              #include <DHT.h>
              
              
              #define NODE_ID 10                       // ID of node
              unsigned long SLEEP_TIME = 600000;        // Sleep time between reports (in milliseconds)
              
              #define CHILD_ID_PIR 1                   // Id of the sensor PIR
              #define CHILD_ID_HUM 2                   // Id of the sensor HUM
              #define CHILD_ID_TEMP 3                  // Id of the sensor TEMP
              #define CHILD_ID_LIGHT 4                 // Id of the sensor LIGHT
              
              #define PIR_SENSOR_DIGITAL 3           // The digital input you attached your motion sensor.  (Only 2 and 3 generates interrupt!)
              #define INTERRUPT PIR_SENSOR_DIGITAL-2 // Usually the interrupt = pin -2 (on uno/nano anyway)
              #define HUMIDITY_SENSOR_DIGITAL_PIN 2
              #define LIGHT_SENSOR_ANALOG_PIN 0
              
              MySensor gw;
              // Initialize Variables
              MyMessage msgPir(CHILD_ID_PIR, V_TRIPPED);
              MyMessage msgHum(CHILD_ID_HUM, V_HUM);
              MyMessage msgTemp(CHILD_ID_TEMP, V_TEMP);
              MyMessage msgLight(CHILD_ID_LIGHT, V_LIGHT_LEVEL);
              
              DHT dht;
              float lastTemp;
              float lastHum;
              boolean metric = true;
              int oldBatteryPcnt;
              int lastLightLevel;
              
              
              int MIN_V = 2700; // empty voltage (0%)
              int MAX_V = 3200; // full voltage (100%)
              
              void setup()
              {
                gw.begin(NULL, NODE_ID, false);
              
                //PIR
                // Send the sketch version information to the gateway and Controller
                gw.sendSketchInfo("Motion Sensor", "1.0");
              
                pinMode(PIR_SENSOR_DIGITAL, INPUT);      // sets the motion sensor digital pin as input
                digitalWrite(PIR_SENSOR_DIGITAL, HIGH);
                // Register all sensors to gw (they will be created as child devices)
                gw.present(CHILD_ID_PIR, S_MOTION);
                
                //DHT
                dht.setup(HUMIDITY_SENSOR_DIGITAL_PIN);
              
                // Send the Sketch Version Information to the Gateway
                gw.sendSketchInfo("Humidity", "1.0");
              
                // Register all sensors to gw (they will be created as child devices)
                gw.present(CHILD_ID_HUM, S_HUM);
                gw.present(CHILD_ID_TEMP, S_TEMP);
              
                metric = gw.getConfig().isMetric;
                
                //LIGHT
                // Send the sketch version information to the gateway and Controller
                gw.sendSketchInfo("Light Sensor", "1.0");
              
                // Register all sensors to gateway (they will be created as child devices)
                gw.present(CHILD_ID_LIGHT, S_LIGHT_LEVEL);
              
              }
              
              void loop()
              {
              
                // Measure battery
                float batteryV = readVcc();
                int batteryPcnt = (((batteryV - MIN_V) / (MAX_V - MIN_V)) * 100 );
                if (batteryPcnt > 100) {
                  batteryPcnt = 100;
                }
              
                if (batteryPcnt != oldBatteryPcnt) {
                  gw.sendBatteryLevel(batteryPcnt); // Send battery percentage
                  oldBatteryPcnt = batteryPcnt;
                }
                Serial.print("---------- Battery: ");
                Serial.println(batteryPcnt);
              
                // Read digital motion value
                boolean tripped = digitalRead(PIR_SENSOR_DIGITAL) == HIGH;
                Serial.print("---------- PIR: ");
                Serial.println(tripped);
                gw.send(msgPir.set(tripped ? "1" : "0")); // Send tripped value to gw
              
              
                //DHT
                //delay(dht.getMinimumSamplingPeriod());
              
                float temperature = dht.getTemperature();
                if (isnan(temperature)) {
                  Serial.println("Failed reading temperature from DHT");
                } else if (temperature != lastTemp) {
                  lastTemp = temperature;
                  if (!metric) {
                    temperature = dht.toFahrenheit(temperature);
                  }
                  gw.send(msgTemp.set(temperature, 1));
                  Serial.print("---------- Temp: ");
                  Serial.println(temperature);
                }
              
                float humidity = dht.getHumidity();
                if (isnan(humidity)) {
                  Serial.println("Failed reading humidity from DHT");
                } else if (humidity != lastHum) {
                  lastHum = humidity;
                  gw.send(msgHum.set(humidity, 1));
                  Serial.print("---------- Humidity: ");
                  Serial.println(humidity);
                }
                // Light
                int lightLevel = (1023 - analogRead(LIGHT_SENSOR_ANALOG_PIN)) / 10.23;
                //Serial.println(lightLevel);
                if (lightLevel != lastLightLevel) {
                  gw.send(msgLight.set(lightLevel));
                  lastLightLevel = lightLevel;
                  Serial.print("---------- Light: ");
                  Serial.println(lightLevel);
                }
                // Sleep until interrupt comes in on motion sensor. Send update every two minute.
                gw.sleep(INTERRUPT, CHANGE, SLEEP_TIME);
              }
              

              the pir works fine. but the dht doesn't work.

              when delay(dht.getMinimumSamplingPeriod()); is used the dht works and the pir fails.

              pimatic + MySensors + Homeduino + z-way
              https://github.com/n3roGit/MySensors_n3ro

              1 Reply Last reply
              0
              • R Offline
                R Offline
                rvendrame
                Hero Member
                wrote on last edited by
                #17

                It makes sense ;-) Try this:

                create (outside the loop) a variable to count the time:

                long mytime = 0; 
                

                .. and replace the delay by :

                 if ( milis() > mytime )  { 
                
                   <  Put the dht stuff here > 
                
                  mytime = millis() + dht.getMinimumSamplingPeriod() ; 
                
                }
                

                Home Assistant / Vera Plus UI7
                ESP8266 GW + mySensors 2.3.2
                Alexa / Google Home

                N 1 Reply Last reply
                0
                • R rvendrame

                  It makes sense ;-) Try this:

                  create (outside the loop) a variable to count the time:

                  long mytime = 0; 
                  

                  .. and replace the delay by :

                   if ( milis() > mytime )  { 
                  
                     <  Put the dht stuff here > 
                  
                    mytime = millis() + dht.getMinimumSamplingPeriod() ; 
                  
                  }
                  
                  N Offline
                  N Offline
                  n3ro
                  wrote on last edited by
                  #18

                  @rvendrame

                  hmm :)
                  if is use this:

                  #include <MySensor.h>
                  #include <readVcc.h>
                  #include <SPI.h>
                  #include <DHT.h>
                  
                  
                  #define NODE_ID 10                       // ID of node
                  unsigned long SLEEP_TIME = 10000;        // Sleep time between reports (in milliseconds)
                  
                  #define CHILD_ID_PIR 1                   // Id of the sensor PIR
                  #define CHILD_ID_HUM 2                   // Id of the sensor HUM
                  #define CHILD_ID_TEMP 3                  // Id of the sensor TEMP
                  #define CHILD_ID_LIGHT 4                 // Id of the sensor LIGHT
                  
                  #define PIR_SENSOR_DIGITAL 3           // The digital input you attached your motion sensor.  (Only 2 and 3 generates interrupt!)
                  #define INTERRUPT PIR_SENSOR_DIGITAL-2 // Usually the interrupt = pin -2 (on uno/nano anyway)
                  #define HUMIDITY_SENSOR_DIGITAL_PIN 2
                  #define LIGHT_SENSOR_ANALOG_PIN 0
                  
                  MySensor gw;
                  // Initialize Variables
                  MyMessage msgPir(CHILD_ID_PIR, V_TRIPPED);
                  MyMessage msgHum(CHILD_ID_HUM, V_HUM);
                  MyMessage msgTemp(CHILD_ID_TEMP, V_TEMP);
                  MyMessage msgLight(CHILD_ID_LIGHT, V_LIGHT_LEVEL);
                  
                  DHT dht;
                  float lastTemp;
                  float lastHum;
                  boolean metric = true;
                  int oldBatteryPcnt;
                  int lastLightLevel;
                  long mytime = 0;
                  
                  
                  int MIN_V = 2700; // empty voltage (0%)
                  int MAX_V = 3200; // full voltage (100%)
                  
                  void setup()
                  {
                    gw.begin(NULL, NODE_ID, false);
                  
                    //PIR
                    // Send the sketch version information to the gateway and Controller
                    gw.sendSketchInfo("Motion Sensor", "1.0");
                  
                    pinMode(PIR_SENSOR_DIGITAL, INPUT);      // sets the motion sensor digital pin as input
                    digitalWrite(PIR_SENSOR_DIGITAL, HIGH);
                    // Register all sensors to gw (they will be created as child devices)
                    gw.present(CHILD_ID_PIR, S_MOTION);
                  
                    //DHT
                    dht.setup(HUMIDITY_SENSOR_DIGITAL_PIN);
                  
                    // Send the Sketch Version Information to the Gateway
                    gw.sendSketchInfo("Humidity", "1.0");
                  
                    // Register all sensors to gw (they will be created as child devices)
                    gw.present(CHILD_ID_HUM, S_HUM);
                    gw.present(CHILD_ID_TEMP, S_TEMP);
                  
                    metric = gw.getConfig().isMetric;
                  
                    //LIGHT
                    // Send the sketch version information to the gateway and Controller
                    gw.sendSketchInfo("Light Sensor", "1.0");
                  
                    // Register all sensors to gateway (they will be created as child devices)
                    gw.present(CHILD_ID_LIGHT, S_LIGHT_LEVEL);
                  
                  }
                  
                  void loop()
                  {
                  
                  
                  
                    // Read digital motion value
                    boolean tripped = digitalRead(PIR_SENSOR_DIGITAL) == HIGH;
                    Serial.print("---------- PIR: ");
                    Serial.println(tripped);
                    gw.send(msgPir.set(tripped ? "1" : "0")); // Send tripped value to gw
                  
                  
                    if ( millis() > mytime )  {
                  
                      //DHT
                      delay(dht.getMinimumSamplingPeriod());
                  
                      float temperature = dht.getTemperature();
                      if (isnan(temperature)) {
                        Serial.println("Failed reading temperature from DHT");
                      } else if (temperature != lastTemp) {
                        lastTemp = temperature;
                        if (!metric) {
                          temperature = dht.toFahrenheit(temperature);
                        }
                        gw.send(msgTemp.set(temperature, 1));
                        Serial.print("---------- Temp: ");
                        Serial.println(temperature);
                      }
                  
                      float humidity = dht.getHumidity();
                      if (isnan(humidity)) {
                        Serial.println("Failed reading humidity from DHT");
                      } else if (humidity != lastHum) {
                        lastHum = humidity;
                        gw.send(msgHum.set(humidity, 1));
                        Serial.print("---------- Humidity: ");
                        Serial.println(humidity);
                      }
                      // Light
                      int lightLevel = (1023 - analogRead(LIGHT_SENSOR_ANALOG_PIN)) / 10.23;
                      //Serial.println(lightLevel);
                      if (lightLevel != lastLightLevel) {
                        gw.send(msgLight.set(lightLevel));
                        lastLightLevel = lightLevel;
                        Serial.print("---------- Light: ");
                        Serial.println(lightLevel);
                      }
                      // Measure battery
                      float batteryV = readVcc();
                      int batteryPcnt = (((batteryV - MIN_V) / (MAX_V - MIN_V)) * 100 );
                      if (batteryPcnt > 100) {
                        batteryPcnt = 100;
                      }
                  
                      if (batteryPcnt != oldBatteryPcnt) {
                        gw.sendBatteryLevel(batteryPcnt); // Send battery percentage
                        oldBatteryPcnt = batteryPcnt;
                      }
                      Serial.print("---------- Battery: ");
                      Serial.println(batteryPcnt);
                  
                      mytime = millis() + dht.getMinimumSamplingPeriod();
                   
                    }
                       Serial.println(mytime);
                    // Sleep until interrupt comes in on motion sensor. Send update every two minute.
                    gw.sleep(INTERRUPT, CHANGE, SLEEP_TIME);
                  }
                  

                  i get this output:
                  ---------- PIR: 0
                  send: 10-10-0-0 s=1,c=1,t=16,pt=0,l=1,st=fail:0
                  6608
                  ---------- PIR: 0
                  send: 10-10-0-0 s=1,c=1,t=16,pt=0,l=1,st=fail:0
                  6608
                  ---------- PIR: 0
                  send: 10-10-0-0 s=1,c=1,t=16,pt=0,l=1,st=fail:0
                  6608
                  ---------- PIR: 0
                  send: 10-10-0-0 s=1,c=1,t=16,pt=0,l=1,st=fail:0
                  6608
                  ---------- PIR: 0
                  send: 10-10-0-0 s=1,c=1,t=16,pt=0,l=1,st=fail:0
                  send: 10-10-255-255 s=255,c=3,t=7,pt=0,l=0,st=fail:
                  send: 10-10-0-0 s=2,c=1,t=1,pt=7,l=5,st=fail:41.7
                  ---------- Humidity: 41.70
                  send: 10-10-0-0 s=4,c=1,t=23,pt=2,l=2,st=fail:38
                  ---------- Light: 38
                  ---------- Battery: 100
                  11148
                  ---------- PIR: 0
                  send: 10-10-0-0 s=1,c=1,t=16,pt=0,l=1,st=fail:0
                  11148
                  ---------- PIR: 0
                  send: 10-10-0-0 s=1,c=1,t=16,pt=0,l=1,st=fail:0
                  11148
                  ---------- PIR: 0
                  send: 10-10-0-0 s=1,c=1,t=16,pt=0,l=1,st=fail:0
                  11148
                  ---------- PIR: 0
                  send: 10-10-0-0 s=1,c=1,t=16,pt=0,l=1,st=fail:0
                  11148
                  ---------- PIR: 0
                  send: 10-10-0-0 s=1,c=1,t=16,pt=0,l=1,st=fail:0
                  send: 10-10-255-255 s=255,c=3,t=7,pt=0,l=0,st=fail:
                  ---------- Battery: 100
                  15441
                  ---------- PIR: 0
                  send: 10-10-0-0 s=1,c=1,t=16,pt=0,l=1,st=fail:0
                  15441
                  ---------- PIR: 0
                  send: 10-10-0-0 s=1,c=1,t=16,pt=0,l=1,st=fail:0
                  15441
                  ---------- PIR: 0
                  send: 10-10-0-0 s=1,c=1,t=16,pt=0,l=1,st=fail:0
                  15441
                  ---------- PIR: 1
                  send: 10-10-0-0 s=1,c=1,t=16,pt=0,l=1,st=ok:1

                  the pir sends 0 and didnt sleep right :(

                  pimatic + MySensors + Homeduino + z-way
                  https://github.com/n3roGit/MySensors_n3ro

                  1 Reply Last reply
                  0
                  • R Offline
                    R Offline
                    rvendrame
                    Hero Member
                    wrote on last edited by
                    #19

                    What is this number? Did you print the milis()?

                    send: 10-10-0-0 s=1,c=1,t=16,pt=0,l=1,st=fail:0
                    6608 <<<<<<

                    Home Assistant / Vera Plus UI7
                    ESP8266 GW + mySensors 2.3.2
                    Alexa / Google Home

                    N 1 Reply Last reply
                    0
                    • R rvendrame

                      What is this number? Did you print the milis()?

                      send: 10-10-0-0 s=1,c=1,t=16,pt=0,l=1,st=fail:0
                      6608 <<<<<<

                      N Offline
                      N Offline
                      n3ro
                      wrote on last edited by
                      #20

                      @n3ro said:

                      Serial.println(mytime);

                      yes :)
                      Serial.println(mytime);

                      pimatic + MySensors + Homeduino + z-way
                      https://github.com/n3roGit/MySensors_n3ro

                      1 Reply Last reply
                      0
                      • R Offline
                        R Offline
                        rvendrame
                        Hero Member
                        wrote on last edited by rvendrame
                        #21
                        #define PIR_SENSOR_DIGITAL 3           // The digital input you attached your motion sensor.  (Only 2 and 3 generates interrupt!)
                        #define INTERRUPT PIR_SENSOR_DIGITAL-2 // Usually the interrupt = pin -2 (on uno/nano anyway)
                        #define HUMIDITY_SENSOR_DIGITAL_PIN 2
                        #define LIGHT_SENSOR_ANALOG_PIN 0
                        

                        Are you connecting both DHT and PIR at same pin 2 ? That must be the cause (DHT is triggering the interruption, I think you want only the PIR to trigger it).

                        Edit: looks like your PIR is at pin 3 , and DHT on pin 2 --- Should be the inverse? The PIR should be on pin 2 (to trigger the interrupt).

                        Home Assistant / Vera Plus UI7
                        ESP8266 GW + mySensors 2.3.2
                        Alexa / Google Home

                        SweebeeS 1 Reply Last reply
                        0
                        • R rvendrame
                          #define PIR_SENSOR_DIGITAL 3           // The digital input you attached your motion sensor.  (Only 2 and 3 generates interrupt!)
                          #define INTERRUPT PIR_SENSOR_DIGITAL-2 // Usually the interrupt = pin -2 (on uno/nano anyway)
                          #define HUMIDITY_SENSOR_DIGITAL_PIN 2
                          #define LIGHT_SENSOR_ANALOG_PIN 0
                          

                          Are you connecting both DHT and PIR at same pin 2 ? That must be the cause (DHT is triggering the interruption, I think you want only the PIR to trigger it).

                          Edit: looks like your PIR is at pin 3 , and DHT on pin 2 --- Should be the inverse? The PIR should be on pin 2 (to trigger the interrupt).

                          SweebeeS Offline
                          SweebeeS Offline
                          Sweebee
                          wrote on last edited by
                          #22

                          @rvendrame Pir sensor is defined as 3

                          interrupt pin is defined as INT1: 3 - 2 = 1

                          1 Reply Last reply
                          0
                          • R Offline
                            R Offline
                            rvendrame
                            Hero Member
                            wrote on last edited by rvendrame
                            #23

                            So PIR must connect on PIN 1 and not PIN 3.

                            Edit: Better, how did you connect DHT and PIR?

                            Home Assistant / Vera Plus UI7
                            ESP8266 GW + mySensors 2.3.2
                            Alexa / Google Home

                            N SweebeeS 2 Replies Last reply
                            0
                            • R rvendrame

                              So PIR must connect on PIN 1 and not PIN 3.

                              Edit: Better, how did you connect DHT and PIR?

                              N Offline
                              N Offline
                              n3ro
                              wrote on last edited by n3ro
                              #24

                              @rvendrame The DHT is on pin 2 and the pir on pin 3. i use the MYS PCB.

                              i suspect the delay function disturb the sleep function. if i dont use the delay, the dht cant "heat up".

                              far as I know the interrupt is only an internal value. right?

                              pimatic + MySensors + Homeduino + z-way
                              https://github.com/n3roGit/MySensors_n3ro

                              1 Reply Last reply
                              0
                              • R Offline
                                R Offline
                                rvendrame
                                Hero Member
                                wrote on last edited by
                                #25

                                Yes, it looks fine. What exactly error are you having? From your first serial monitor, it looks like your sensor woke on every 10min aprox, ( 6608 / 11148 / 15441 ).

                                Home Assistant / Vera Plus UI7
                                ESP8266 GW + mySensors 2.3.2
                                Alexa / Google Home

                                N 1 Reply Last reply
                                0
                                • R rvendrame

                                  Yes, it looks fine. What exactly error are you having? From your first serial monitor, it looks like your sensor woke on every 10min aprox, ( 6608 / 11148 / 15441 ).

                                  N Offline
                                  N Offline
                                  n3ro
                                  wrote on last edited by n3ro
                                  #26

                                  @rvendrame
                                  if i use this:

                                  #include <MySensor.h>
                                  #include <readVcc.h>
                                  #include <SPI.h>
                                  #include <DHT.h>
                                  
                                  
                                  #define NODE_ID 10                       // ID of node
                                  unsigned long SLEEP_TIME = 600000;        // Sleep time between reports (in milliseconds)
                                  
                                  #define CHILD_ID_PIR 1                   // Id of the sensor PIR
                                  #define CHILD_ID_HUM 2                   // Id of the sensor HUM
                                  #define CHILD_ID_TEMP 3                  // Id of the sensor TEMP
                                  #define CHILD_ID_LIGHT 4                 // Id of the sensor LIGHT
                                  
                                  #define PIR_SENSOR_DIGITAL 3           // The digital input you attached your motion sensor.  (Only 2 and 3 generates interrupt!)
                                  #define INTERRUPT PIR_SENSOR_DIGITAL-2 // Usually the interrupt = pin -2 (on uno/nano anyway)
                                  #define HUMIDITY_SENSOR_DIGITAL_PIN 2
                                  #define LIGHT_SENSOR_ANALOG_PIN 0
                                  
                                  MySensor gw;
                                  // Initialize Variables
                                  MyMessage msgPir(CHILD_ID_PIR, V_TRIPPED);
                                  MyMessage msgHum(CHILD_ID_HUM, V_HUM);
                                  MyMessage msgTemp(CHILD_ID_TEMP, V_TEMP);
                                  MyMessage msgLight(CHILD_ID_LIGHT, V_LIGHT_LEVEL);
                                  
                                  DHT dht;
                                  float lastTemp;
                                  float lastHum;
                                  boolean metric = true;
                                  int oldBatteryPcnt;
                                  int lastLightLevel;
                                  
                                  
                                  int MIN_V = 2700; // empty voltage (0%)
                                  int MAX_V = 3200; // full voltage (100%)
                                  
                                  void setup()
                                  {
                                    gw.begin(NULL, NODE_ID, false);
                                  
                                    //PIR
                                    // Send the sketch version information to the gateway and Controller
                                    gw.sendSketchInfo("Motion Sensor", "1.0");
                                  
                                    pinMode(PIR_SENSOR_DIGITAL, INPUT);      // sets the motion sensor digital pin as input
                                    digitalWrite(PIR_SENSOR_DIGITAL, HIGH);
                                    // Register all sensors to gw (they will be created as child devices)
                                    gw.present(CHILD_ID_PIR, S_MOTION);
                                    
                                    //DHT
                                    dht.setup(HUMIDITY_SENSOR_DIGITAL_PIN);
                                  
                                    // Send the Sketch Version Information to the Gateway
                                    gw.sendSketchInfo("Humidity", "1.0");
                                  
                                    // Register all sensors to gw (they will be created as child devices)
                                    gw.present(CHILD_ID_HUM, S_HUM);
                                    gw.present(CHILD_ID_TEMP, S_TEMP);
                                  
                                    metric = gw.getConfig().isMetric;
                                    
                                    //LIGHT
                                    // Send the sketch version information to the gateway and Controller
                                    gw.sendSketchInfo("Light Sensor", "1.0");
                                  
                                    // Register all sensors to gateway (they will be created as child devices)
                                    gw.present(CHILD_ID_LIGHT, S_LIGHT_LEVEL);
                                  
                                  }
                                  
                                  void loop()
                                  {
                                  
                                    // Measure battery
                                    float batteryV = readVcc();
                                    int batteryPcnt = (((batteryV - MIN_V) / (MAX_V - MIN_V)) * 100 );
                                    if (batteryPcnt > 100) {
                                      batteryPcnt = 100;
                                    }
                                  
                                    if (batteryPcnt != oldBatteryPcnt) {
                                      gw.sendBatteryLevel(batteryPcnt); // Send battery percentage
                                      oldBatteryPcnt = batteryPcnt;
                                    }
                                    Serial.print("---------- Battery: ");
                                    Serial.println(batteryPcnt);
                                  
                                    // Read digital motion value
                                    boolean tripped = digitalRead(PIR_SENSOR_DIGITAL) == HIGH;
                                    Serial.print("---------- PIR: ");
                                    Serial.println(tripped);
                                    gw.send(msgPir.set(tripped ? "1" : "0")); // Send tripped value to gw
                                  
                                  
                                    //DHT
                                    //delay(dht.getMinimumSamplingPeriod());
                                  
                                    float temperature = dht.getTemperature();
                                    if (isnan(temperature)) {
                                      Serial.println("Failed reading temperature from DHT");
                                    } else if (temperature != lastTemp) {
                                      lastTemp = temperature;
                                      if (!metric) {
                                        temperature = dht.toFahrenheit(temperature);
                                      }
                                      gw.send(msgTemp.set(temperature, 1));
                                      Serial.print("---------- Temp: ");
                                      Serial.println(temperature);
                                    }
                                  
                                    float humidity = dht.getHumidity();
                                    if (isnan(humidity)) {
                                      Serial.println("Failed reading humidity from DHT");
                                    } else if (humidity != lastHum) {
                                      lastHum = humidity;
                                      gw.send(msgHum.set(humidity, 1));
                                      Serial.print("---------- Humidity: ");
                                      Serial.println(humidity);
                                    }
                                    // Light
                                    int lightLevel = (1023 - analogRead(LIGHT_SENSOR_ANALOG_PIN)) / 10.23;
                                    //Serial.println(lightLevel);
                                    if (lightLevel != lastLightLevel) {
                                      gw.send(msgLight.set(lightLevel));
                                      lastLightLevel = lightLevel;
                                      Serial.print("---------- Light: ");
                                      Serial.println(lightLevel);
                                    }
                                    // Sleep until interrupt comes in on motion sensor. Send update every two minute.
                                    gw.sleep(INTERRUPT, CHANGE, SLEEP_TIME);
                                  }
                                  

                                  i get this output after starting:
                                  sensor started, id 10
                                  send: 10-10-0-0 s=255,c=0,t=17,pt=0,l=5,st=ok:1.4.1
                                  send: 10-10-0-0 s=255,c=3,t=6,pt=1,l=1,st=ok:0
                                  read: 0-0-10 s=255,c=3,t=6,pt=0,l=1:M
                                  send: 10-10-0-0 s=255,c=3,t=11,pt=0,l=13,st=ok:Motion Sensor
                                  send: 10-10-0-0 s=255,c=3,t=12,pt=0,l=3,st=ok:1.0
                                  send: 10-10-0-0 s=1,c=0,t=1,pt=0,l=5,st=ok:1.4.1
                                  send: 10-10-0-0 s=255,c=3,t=11,pt=0,l=8,st=ok:Humidity
                                  send: 10-10-0-0 s=255,c=3,t=12,pt=0,l=3,st=ok:1.0
                                  send: 10-10-0-0 s=2,c=0,t=7,pt=0,l=5,st=ok:1.4.1
                                  send: 10-10-0-0 s=3,c=0,t=6,pt=0,l=5,st=ok:1.4.1
                                  send: 10-10-0-0 s=255,c=3,t=11,pt=0,l=12,st=ok:Light Sensor
                                  send: 10-10-0-0 s=255,c=3,t=12,pt=0,l=3,st=ok:1.0
                                  send: 10-10-0-0 s=4,c=0,t=16,pt=0,l=5,st=ok:1.4.1
                                  send: 10-10-0-0 s=255,c=3,t=0,pt=1,l=1,st=ok:100
                                  ---------- Battery: 100
                                  ---------- PIR: 1
                                  send: 10-10-0-0 s=1,c=1,t=16,pt=0,l=1,st=ok:1
                                  send: 10-10-0-0 s=3,c=1,t=0,pt=7,l=5,st=ok:16.6
                                  ---------- Temp: 16.60
                                  send: 10-10-0-0 s=2,c=1,t=1,pt=7,l=5,st=ok:43.8
                                  ---------- Humidity: 43.80
                                  send: 10-10-0-0 s=4,c=1,t=23,pt=2,l=2,st=ok:39
                                  ---------- Light: 39
                                  ---------- Battery: 100
                                  ---------- PIR: 0
                                  send: 10-10-0-0 s=1,c=1,t=16,pt=0,l=1,st=ok:0
                                  send: 10-10-0-0 s=4,c=1,t=23,pt=2,l=2,st=ok:37
                                  ---------- Light: 37
                                  ---------- Battery: 100
                                  ---------- PIR: 1
                                  send: 10-10-0-0 s=1,c=1,t=16,pt=0,l=1,st=ok:1
                                  send: 10-10-0-0 s=4,c=1,t=23,pt=2,l=2,st=ok:34
                                  ---------- Light: 34
                                  ---------- Battery: 100
                                  ---------- PIR: 0
                                  send: 10-10-0-0 s=1,c=1,t=16,pt=0,l=1,st=ok:0
                                  send: 10-10-0-0 s=4,c=1,t=23,pt=2,l=2,st=ok:36
                                  ---------- Light: 36
                                  ---------- Battery: 100
                                  ---------- PIR: 1
                                  send: 10-10-0-0 s=1,c=1,t=16,pt=0,l=1,st=ok:1
                                  send: 10-10-0-0 s=4,c=1,t=23,pt=2,l=2,st=ok:37
                                  ---------- Light: 37
                                  ---------- Battery: 100
                                  ---------- PIR: 0
                                  send: 10-10-0-0 s=1,c=1,t=16,pt=0,l=1,st=ok:0
                                  send: 10-10-0-0 s=4,c=1,t=23,pt=2,l=2,st=ok:38
                                  ---------- Light: 38

                                  The DHT gives only output the first start and the pir interrupt works perfect.

                                  When delay(dht.getMinimumSamplingPeriod()); is activated the DHT works and the pit interrupts the whole time :(
                                  In this configuration the arduino never sleeps ..

                                  sensor started, id 10
                                  send: 10-10-0-0 s=255,c=0,t=17,pt=0,l=5,st=ok:1.4.1
                                  send: 10-10-0-0 s=255,c=3,t=6,pt=1,l=1,st=ok:0
                                  read: 0-0-10 s=255,c=3,t=6,pt=0,l=1:M
                                  send: 10-10-0-0 s=255,c=3,t=11,pt=0,l=13,st=ok:Motion Sensor
                                  send: 10-10-0-0 s=255,c=3,t=12,pt=0,l=3,st=ok:1.0
                                  send: 10-10-0-0 s=1,c=0,t=1,pt=0,l=5,st=ok:1.4.1
                                  send: 10-10-0-0 s=255,c=3,t=11,pt=0,l=8,st=ok:Humidity
                                  send: 10-10-0-0 s=255,c=3,t=12,pt=0,l=3,st=ok:1.0
                                  send: 10-10-0-0 s=2,c=0,t=7,pt=0,l=5,st=ok:1.4.1
                                  send: 10-10-0-0 s=3,c=0,t=6,pt=0,l=5,st=ok:1.4.1
                                  send: 10-10-0-0 s=255,c=3,t=11,pt=0,l=12,st=ok:Light Sensor
                                  send: 10-10-0-0 s=255,c=3,t=12,pt=0,l=3,st=ok:1.0
                                  send: 10-10-0-0 s=4,c=0,t=16,pt=0,l=5,st=ok:1.4.1
                                  send: 10-10-0-0 s=255,c=3,t=0,pt=1,l=1,st=ok:100
                                  ---------- Battery: 100
                                  ---------- PIR: 1
                                  send: 10-10-0-0 s=1,c=1,t=16,pt=0,l=1,st=ok:1
                                  send: 10-10-0-0 s=3,c=1,t=0,pt=7,l=5,st=ok:16.5
                                  ---------- Temp: 16.50
                                  send: 10-10-0-0 s=2,c=1,t=1,pt=7,l=5,st=ok:43.9
                                  ---------- Humidity: 43.90
                                  send: 10-10-0-0 s=4,c=1,t=23,pt=2,l=2,st=ok:38
                                  ---------- Light: 38
                                  ---------- Battery: 100
                                  ---------- PIR: 1
                                  send: 10-10-0-0 s=1,c=1,t=16,pt=0,l=1,st=ok:1
                                  send: 10-10-0-0 s=3,c=1,t=0,pt=7,l=5,st=ok:16.6
                                  ---------- Temp: 16.60
                                  send: 10-10-0-0 s=2,c=1,t=1,pt=7,l=5,st=ok:43.8
                                  ---------- Humidity: 43.80
                                  send: 10-10-0-0 s=4,c=1,t=23,pt=2,l=2,st=ok:37
                                  ---------- Light: 37
                                  ---------- Battery: 100
                                  ---------- PIR: 0
                                  send: 10-10-0-0 s=1,c=1,t=16,pt=0,l=1,st=ok:0
                                  send: 10-10-0-0 s=3,c=1,t=0,pt=7,l=5,st=ok:16.5
                                  ---------- Temp: 16.50
                                  ---------- Battery: 100
                                  ---------- PIR: 0
                                  send: 10-10-0-0 s=1,c=1,t=16,pt=0,l=1,st=ok:0
                                  ---------- Battery: 100
                                  ---------- PIR: 0
                                  send: 10-10-0-0 s=1,c=1,t=16,pt=0,l=1,st=ok:0
                                  send: 10-10-0-0 s=4,c=1,t=23,pt=2,l=2,st=ok:38

                                  EDIT: i have changed the sleeptime for testing to a shorter time

                                  pimatic + MySensors + Homeduino + z-way
                                  https://github.com/n3roGit/MySensors_n3ro

                                  1 Reply Last reply
                                  0
                                  • R rvendrame

                                    So PIR must connect on PIN 1 and not PIN 3.

                                    Edit: Better, how did you connect DHT and PIR?

                                    SweebeeS Offline
                                    SweebeeS Offline
                                    Sweebee
                                    wrote on last edited by
                                    #27

                                    @rvendrame When the PIR is connected to PIN3 the interrupt is INT1. Pin 1 does not have an interrupt.

                                    Only pin 2 and pin 3 have an interrupt.

                                    Pin 2 = int0
                                    pin 3 = int1

                                    N 1 Reply Last reply
                                    0
                                    • R Offline
                                      R Offline
                                      rvendrame
                                      Hero Member
                                      wrote on last edited by
                                      #28

                                      if you put the delay() inside that 'if ( mills() > mytime ) ' , what happens?

                                      Home Assistant / Vera Plus UI7
                                      ESP8266 GW + mySensors 2.3.2
                                      Alexa / Google Home

                                      N 1 Reply Last reply
                                      0
                                      • SweebeeS Sweebee

                                        @rvendrame When the PIR is connected to PIN3 the interrupt is INT1. Pin 1 does not have an interrupt.

                                        Only pin 2 and pin 3 have an interrupt.

                                        Pin 2 = int0
                                        pin 3 = int1

                                        N Offline
                                        N Offline
                                        n3ro
                                        wrote on last edited by
                                        #29

                                        @Sweebee is it possible to fix the problem when DHT is connected to pin 4?

                                        pimatic + MySensors + Homeduino + z-way
                                        https://github.com/n3roGit/MySensors_n3ro

                                        1 Reply Last reply
                                        0
                                        • R Offline
                                          R Offline
                                          rvendrame
                                          Hero Member
                                          wrote on last edited by
                                          #30

                                          @n3ro said:

                                          s it possible to fix the problem when DHT is connected to pin 4?

                                          It worth a try ...

                                          Home Assistant / Vera Plus UI7
                                          ESP8266 GW + mySensors 2.3.2
                                          Alexa / Google Home

                                          1 Reply Last reply
                                          0

                                          Hello! It looks like you're interested in this conversation, but you don't have an account yet.

                                          Getting fed up of having to scroll through the same posts each visit? When you register for an account, you'll always come back to exactly where you were before, and choose to be notified of new replies (either via email, or push notification). You'll also be able to save bookmarks and upvote posts to show your appreciation to other community members.

                                          With your input, this post could be even better 💗

                                          Register Login
                                          Reply
                                          • Reply as topic
                                          Log in to reply
                                          • Oldest to Newest
                                          • Newest to Oldest
                                          • Most Votes


                                          3

                                          Online

                                          12.1k

                                          Users

                                          11.2k

                                          Topics

                                          113.4k

                                          Posts


                                          Copyright 2025 TBD   |   Forum Guidelines   |   Privacy Policy   |   Terms of Service
                                          • Login

                                          • Don't have an account? Register

                                          • Login or register to search.
                                          • First post
                                            Last post
                                          0
                                          • MySensors
                                          • OpenHardware.io
                                          • Categories
                                          • Recent
                                          • Tags
                                          • Popular