Multisensor PIR problem [solved]

Sparkman

@n3ro Hard to help you with your sketch since you don't have it posted ;-)

n3ro

https://github.com/n3roGit/MySensors_n3ro/tree/master/MotionSensor_DHT_light_battery_mod

;)

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();
......

n3ro

@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.

n3ro

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>

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.

n3ro

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

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);

n3ro

@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.

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() ; 

}

n3ro

@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 :(

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 <<<<<<

n3ro

@n3ro said:

Serial.println(mytime);

yes :)
Serial.println(mytime);

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).

Sweebee

@rvendrame Pir sensor is defined as 3

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

rvendrame

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

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

n3ro

@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?

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 ).

n3ro

@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

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

Multisensor PIR problem [solved]

14

11.7k

11.2k

113.1k