Posts made by Denis Renaud

Hi, yout post is quit old but I changed your code a bit. Now it sends the status of both relays:

long double last_heartbeat_time = millis();
long double HEARTBEAT_TIME = 30000;

int oldValue = 0;
bool state;

MyMessage msg1a(1, V_STATUS);
MyMessage msg1b;

MyMessage msg2a(2, V_STATUS);
MyMessage msg2b;

void before() {
  for (int sensor = 1, pin = RELAY_1; sensor <= NUMBER_OF_RELAYS; sensor++, pin++) {
    // Then set relay pins in output mode
    pinMode(pin, OUTPUT);
    // Set relay to last known state (using eeprom storage)
    digitalWrite(pin, loadState(sensor) ? RELAY_ON : RELAY_OFF);
  }
}

void setup() {

}

void presentation()
{
  // Send the sketch version information to the gateway and Controller
  sendSketchInfo("Relay", "1.0");

  for (int sensor = 1, pin = RELAY_1; sensor <= NUMBER_OF_RELAYS; sensor++, pin++) {
    // Register all sensors to gw (they will be created as child devices)
    present(sensor, S_BINARY);
  }
}


void loop()
{
  long double temp = (millis() - last_heartbeat_time);
  if (temp > HEARTBEAT_TIME) {
    // If it exceeds the heartbeat time then send a heartbeat
    sendHeartbeat();
    send(msg1a.set(msg1b.getBool() ? 1 : 0));
    send(msg2a.set(msg2b.getBool() ? 1 : 0));
    last_heartbeat_time = millis();
    Serial.println("Sent heartbeat" );
  }
}

void receive(const MyMessage &message) {

  // We only expect one type of message from controller. But we better check anyway.
  if (message.type == V_STATUS) {
    // Change relay state
    digitalWrite(message.sensor - 1 + RELAY_1, message.getBool() ? RELAY_ON : RELAY_OFF);
    // Store state in eeprom
    saveState(message.sensor, message.getBool());
    // Write some debug info
    Serial.print("Incoming change for sensor:");
    Serial.print(message.sensor);
    Serial.print(", New status: ");
    Serial.println(message.getBool());

    if (message.sensor == 1) {
      msg1b = message;
    }

    if (message.sensor == 2) {
      msg2b = message;
    }

  }
}

rg Denis