Upgrading to 2.0 sketch dificulty

[mfalkvidd]

@andmaster use the relay example for MySensors 2.0 (which is included with the library) and follow the instructions in the code comments and you should be all set.

[andmaster]

@mfalkvidd this is the example 😃

[mfalkvidd]

@andmaster no.

The sketch you posted does not have these instructions:

#define RELAY_1  3  // Arduino Digital I/O pin number for first relay (second on pin+1 etc)
#define NUMBER_OF_RELAYS 1 // Total number of attached relays

[andmaster]

@mfalkvidd the example is provided here: https://github.com/mysensors/MySensorsArduinoExamples/blob/master/examples/RelayWithButtonActuator/RelayWithButtonActuator.ino

what you are mentioning is the example where you can put multiple relays, but without any button actuator.

Thanks

[scalz]

@andmaster
in fact @mfalkvidd is right, you have to "mix" the two sketch if you want multi relay with multiple buttons.
could be fast to do, but i have no time.
if you want show us your non working sketch, so we can tell what to do ;)

[niccodemi]

@andmaster

I've used below sketch with 1.6 development version; it compiles also with 2.0.0 but I have not tested it. This sketch uses toggle buttons.

//MultiRelayButton Sketch, MySensors 1.6, toggle switch

#define MY_RADIO_NRF24

#define MY_REPEATER_FEATURE

#include <MySensors.h>
#include <SPI.h>
#include <Bounce2.h>
#define RELAY_ON 0                      // switch around for ACTIVE LOW / ACTIVE HIGH relay
#define RELAY_OFF 1                     // switch around for ACTIVE LOW / ACTIVE HIGH relay
//

#define noRelays 4                     //2-4, define number of relays
const int relayPin[] = {14,15,16,17};          //  relay pins - should match noRelays
const int buttonPin[] = {3,4,5,6};      //  button pins - should match noRelays

class Relay             // relay class, store all relevant data (equivalent to struct)
{
public:                                      
  int buttonPin;                   // physical pin number of button
  int relayPin;             // physical pin number of relay
  boolean relayState;               // relay status (also stored in EEPROM)
};

Relay Relays[noRelays]; 
Bounce debouncer[noRelays];
MyMessage msg[noRelays];

void setup(){
    sendHeartbeat();
    wait(100);
    // Initialize Relays with corresponding buttons
    for (int i = 0; i < noRelays; i++){
    Relays[i].buttonPin = buttonPin[i];              // assign physical pins
    Relays[i].relayPin = relayPin[i];
    msg[i].sensor = i;                                   // initialize messages
    msg[i].type = V_LIGHT;   
    pinMode(Relays[i].buttonPin, INPUT_PULLUP);
    wait(100);
    pinMode(Relays[i].relayPin, OUTPUT);
    Relays[i].relayState = loadState(i);                               // retrieve last values from EEPROM
    digitalWrite(Relays[i].relayPin, Relays[i].relayState? RELAY_ON:RELAY_OFF);   // and set relays accordingly
    send(msg[i].set(Relays[i].relayState? true : false));                  // make controller aware of last status  
    wait(50);
    debouncer[i] = Bounce();                        // initialize debouncer
    debouncer[i].attach(buttonPin[i]);
    debouncer[i].interval(20);
    wait(50);
    }
}
void presentation()  {
      sendSketchInfo("MultiRelayButton", "1.0");
      wait(100);
      for (int i = 0; i < noRelays; i++)
      present(i, S_LIGHT);                               // present sensor to gateway
      wait(100);
}
void loop()
{
    for (byte i = 0; i < noRelays; i++) {
        if (debouncer[i].update()) {
            Relays[i].relayState = !Relays[i].relayState;
            digitalWrite(Relays[i].relayPin, Relays[i].relayState?RELAY_ON:RELAY_OFF);
            send(msg[i].set(Relays[i].relayState? true : false));
            // save sensor state in EEPROM (location == sensor number)
            saveState( i, Relays[i].relayState );
        }                 
    }
    wait(20);
}
// process incoming message 
void receive(const MyMessage &message){        
   if (message.type == V_LIGHT){ 
   if (message.sensor <noRelays){            // check if message is valid for relays..... previous line  [[[ if (message.sensor <=noRelays){ ]]]
   Relays[message.sensor].relayState = message.getBool(); 
   digitalWrite(Relays[message.sensor].relayPin, Relays[message.sensor].relayState? RELAY_ON:RELAY_OFF); // and set relays accordingly
   saveState( message.sensor, Relays[message.sensor].relayState ); // save sensor state in EEPROM (location == sensor number)
   }
  }
  wait(20);
}

[andmaster]

hi @niccodemi that did the trick.

I've made a quick comparison and i guess the issue was that i've defined my button without a proper array.

this also has a change i think is important. i had a push button, and i wanted a toggle :)

Just made a change. i want relay on to be 1 (high) and off to be 0 (low).
This way, it starts every time at 0

many thanks!!

[andredts]

@niccodemi

Hi, I know that this is quite old topic, but only now I´m updating my home from version 1.5.4 to 2.*, most because reability issues. One thing I notice with you skecth is that if by any chance the gateway is not present the nodes can´t be used anymore after a power loss, I mean, one can not use the fisical buttons to switch the relays, because they will not respond. My question is if there is a way to keep the node up with out the gateway, in case it dies?

Thanks

[Boots33]

@andredts As you have discovered version 2.1 nodes while booting will not proceed to the loop part of the sketch if the gateway is not found. You can however force them to move on. The line of code you need is...

#define MY_TRANSPORT_WAIT_READY_MS 3000

This code tells the node how long to wait for the uplink to be established before moving on to the rest of the sketch. The number at the end is how long to wait in milliseconds, in this case 3000ms (3 seconds). You can set this to a time of your choosing.

if the node establishes the uplink it will move on without any delay.

The above line is placed near the top of your sketch somewhere before

#include <MySensors.h>

have a look at these for a bit more detail....

MY_TRANSPORT_DONT_CARE_MODE

Synchronising Light switch

[andredts]

@Boots33 Thanks, that was exactly what I needed. Synchronizing was a great bonus.