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 Offline
    T Offline
    torfinn
    wrote on last edited by
    #10

    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 1 Reply Last reply
    0
    • 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

                                          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


                                          12

                                          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