Communication problem (maybe)

[hek]

@Hausner said:

//gw.sleep(SLEEP_TIME);

Why did you comment away this line?

[Hausner]

@hek

Because it looks like that the sleep function is messing with the repeater function. With sleep enabled, almost all messages to the relay actuator node gets lost.

[hek]

Hmm.. shouldn't affect transmissions. When you removed this line you will get several hundreds of temp readings per second. It is a very big chance a few of them differs which means they will send it to gateway.

You could try adding:
gw.powerUp();
delay(10)
Just before gw.send() to add a small some additional delay after radio powers up.

If you want to keep node awake another option is to use millis() keep track of send frequency. See here:
https://github.com/mysensors/Arduino/blob/master/libraries/MySensors/examples/TimeAwareSensor/TimeAwareSensor.ino#L47

[Hausner]

@hek said:

delay(10)

I assume that is milliseconds, but does that mean that the radio will be powered on during that delay, and will the node repeat incoming messages during that?

If so, I think a much bigger delay would be better, like 30000 or even more.

[hek]

This node isn't configured to be a repeater. So no, it won't.

[Hausner]

It isn't??

hmm... now i'm puzzled!

Cause this is 2 temp nodes

0;0;3;0;9;read: 24-23-0 s=0,c=1,t=0,pt=7,l=5:23.8

My relay actuator is 22

[niccodemi]

@Hausner do you have any issues with relay node if you are NOT using it also as repeater?

[hek]

If so, I think a much bigger delay would be better, like 30000 or even more.

Yes, thats is actually a third option like you say. Just add a delay(30000) at the end of loop(). (If this node just acts a as a temp sensor and nothing else).

[pgo]

@Hausner I send the temp changes only if change is > 1 deg C and light changes only > +/- 40 lux, have this in main loop:

void loop()     
{     
  // Process incoming messages (like config from server)
  gw.process(); 

  // Fetch temperatures from Dallas sensors
  sensors.requestTemperatures(); 

  // Read temperatures and send them to controller 
  for (int i=0; i<numSensors && i<MAX_ATTACHED_DS18B20; i++) {

    // Fetch and round temperature to one decimal
    float temperature = static_cast<float>(static_cast<int>((gw.getConfig().isMetric?sensors.getTempCByIndex(i):sensors.getTempFByIndex(i)) * 10.)) / 10.;

    // Only send data if temperature has changed more then 1 degC and no error
    if (int(lastTemperature[i]) != int(temperature) && temperature != -127.00) { //added integer

      // Send in the new temperature
      gw.send(msg.setSensor(i).set(temperature,1));
      lastTemperature[i]=temperature;
    }
  }
  // Read lux from TEMT6000
  float lightLevel = analogRead(LIGHT_SENSOR_ANALOG_PIN) * 9.765625;  // 1000/1024
  if (int(lightLevel - lastLightLevel) > 40 || int(lastLightLevel - lightLevel) > 40) {
      gw.send(msg.setSensor(CHILD_ID_LIGHT).set(int(lightLevel),1));
      lastLightLevel = lightLevel;
  }
  //gw.sleep(SLEEP_TIME); //no sleep for relays!!!!
}

Hope this helps, Pego

[daulagari]

As you can see i'm getting way more than 1 reading a second. I think this is happening because of the rounding up/down of the temperature readings.

Yes, clear, if you do not want this you need to change the if to something like:

if (abs(lastTemperature[i] - temperature) >= 0.5 && temperature != -127.00) {

[Hausner]

@daulagari

Thank you. I'm gonna try with this

if (abs(lastTemperature[i] - temperature) >= 0.2 && temperature != -127.00) {