Multi Button Relay switch

Terence Faul

This new one had the relays correct, but the buttons did not work. I compared the old with the new and below now works.

Thanks again a million for all the help

#include <MySensor.h>
#include <SPI.h>
#include <Bounce2.h>

#define RELAY_ON 0
#define RELAY_OFF 1

//
MySensor gw;
const int relayPin[] = {7, 8, A0, A1};
const int buttonPin[sizeof(relayPin) / sizeof(relayPin[0])] = {3, 4, 5, 6};
byte oldValue[sizeof(relayPin) / sizeof(relayPin[0])];
boolean relayState[sizeof(relayPin) / sizeof(relayPin[0])];

Bounce debouncer[sizeof(relayPin) / sizeof(relayPin[0])];

MyMessage msg[sizeof(relayPin) / sizeof(relayPin[0])];//(sensor,type);

void setup()  
{  
  Serial.begin(115200);
  gw.begin(incomingMessage, AUTO, false);
  //
  // or you can try:
  // gw.begin(incomingMessage, <Your_Node_ID>, false); // where Your_Node_ID is a number from 1 to 254
  //
  delay(250);
  gw.sendSketchInfo("MultiRelayButton", "0.9b");// <<<<<<<<<<<<<<<<<<< I forgot this, here
  //
  delay(250);
  for (int i = 0; i < sizeof(relayPin) / sizeof(relayPin[0]); i++)
  {
    msg[i].sensor = i;
    msg[i].type = V_LIGHT;
    debouncer[i] = Bounce();
    debouncer[i].attach(buttonPin[i]);
    debouncer[i].interval(5);
    pinMode(buttonPin[i], INPUT_PULLUP);
    digitalWrite(relayPin[i], LOW);
    pinMode(relayPin[i], OUTPUT);
    gw.present(i, S_LIGHT); // <<<<<<<<<<<<<<<<<<<<<<< I fixed this too...
    delay(250);
  }
  //retreive from EEPROM last states
  for (int i = 0; i < sizeof(relayPin) / sizeof(relayPin[0]); i++)
  {
    relayState[i] = gw.loadState(i);
    digitalWrite(relayPin[i], relayState[i]? RELAY_ON : RELAY_OFF);
    gw.send(msg[i].set(relayState[i]? true : false), true);
    delay(250);
  }
}
//
void loop() 
{
  gw.process();
  for (byte i = 0; i < sizeof(relayPin) / sizeof(relayPin[0]); i++)
  {
    debouncer[i].update();
    byte value = debouncer[i].read();
    if (value != oldValue[i] && value == 0)
    {
      relayState[i] = !relayState[i];
      digitalWrite(relayPin[i], relayState[i]);
      gw.send(msg[i].set(relayState[i]? true : false), true);
    }
    oldValue[i] = value;
  }
}
//
void incomingMessage(const MyMessage &message)
{
  if (message.isAck()) 
  {
    Serial.println(F("This is an ack from gateway"));
  }
  for (byte i = 0; i< sizeof(relayPin) / sizeof(relayPin[0]); i++)
  {
    if (message.sensor == i)
    {
      if (message.type == V_LIGHT)
      {
        relayState[i] = message.getBool();
        digitalWrite(relayPin[i], relayState[i]? RELAY_ON : RELAY_OFF); 
        gw.saveState(relayPin[i], relayState[i]);
      }
    }
  }
}

Terence Faul

@Terence-Faul
Only one other thing.

I notice that when the sensor starts up, each of the relays turn on and then off.

This ia all good and well, however i assume in practice this would mean what ever is connected to them will turn on and off as well in the initialisation phase.

Also I assume that retrieve from EEPROM means that if the sensor loses power while a relay is on, when it powers up this relay will return to the on state?

Can you point me to to code that controls this, if these are lights and the power goes out a better option may be to have the relays all start in the off position so that the lights do not come on in the middle of the night

Is this correct?

Terence Faul

Bulldog

So this is as far as I got.

Managed to get the relays to start in the off position by Changing LOW for the defined RELAY_OFF.

Commented out the EEPROM Section and now when the sensor reboots is starts with all relays in off position.

I do not know how to change the EEPROM section I am sur I need to maybe include the RELAY_OFF somewhere in that line.

#include <MySensor.h>
#include <SPI.h>
#include <Bounce2.h>

#define RELAY_ON 0  //switch around for realy HIGH/LOW state
#define RELAY_OFF 1

//
MySensor gw;
const int relayPin[] = {7, 8, A0, A1};
const int buttonPin[sizeof(relayPin) / sizeof(relayPin[0])] = {3, 4, 5, 6};
byte oldValue[sizeof(relayPin) / sizeof(relayPin[0])];
boolean relayState[sizeof(relayPin) / sizeof(relayPin[0])];

Bounce debouncer[sizeof(relayPin) / sizeof(relayPin[0])];

MyMessage msg[sizeof(relayPin) / sizeof(relayPin[0])];//(sensor,type);

void setup()  
{  
  Serial.begin(115200);
  gw.begin(incomingMessage, AUTO, false);
  //
  // or you can try:
  // gw.begin(incomingMessage, <Your_Node_ID>, false); // where Your_Node_ID is a number from 1 to 254
  //
  delay(250);
  gw.sendSketchInfo("MultiRelayButton", "0.9b");// <<<<<<<<<<<<<<<<<<< I forgot this, here
  //
  delay(250);
  for (int i = 0; i < sizeof(relayPin) / sizeof(relayPin[0]); i++)
  {
    msg[i].sensor = i;
    msg[i].type = V_LIGHT;
    debouncer[i] = Bounce();
    debouncer[i].attach(buttonPin[i]);
    debouncer[i].interval(5);
    pinMode(buttonPin[i], INPUT_PULLUP);
    digitalWrite(relayPin[i], RELAY_OFF); //Start State of Relays
    pinMode(relayPin[i], OUTPUT);
    gw.present(i, S_LIGHT); // <<<<<<<<<<<<<<<<<<<<<<< I fixed this too...
    delay(250);
  }
  //retreive from EEPROM last states
//  for (int i = 0; i < sizeof(relayPin) / sizeof(relayPin[0]); i++)//>>>>>>> I'm sure that this is causing all relays to turn on and off at startup
//  {
//    relayState[i] = gw.loadState(i);
//    digitalWrite(relayPin[i], relayState[i]? RELAY_ON:RELAY_OFF);
//    gw.send(msg[i].set(relayState[i]? true : false), true);
//    delay(250);
//  }
//  
}
//
void loop() 
{
  gw.process();
  for (byte i = 0; i < sizeof(relayPin) / sizeof(relayPin[0]); i++)
  {
    debouncer[i].update();
    byte value = debouncer[i].read();
    if (value != oldValue[i] && value == 0)
    {
      relayState[i] = !relayState[i];
      digitalWrite(relayPin[i], relayState[i]);
      gw.send(msg[i].set(relayState[i]? true : false), true);
    }
    oldValue[i] = value;
  }
}
//
void incomingMessage(const MyMessage &message)
{
  if (message.isAck()) 
  {
    Serial.println(F("This is an ack from gateway"));
  }
  for (byte i = 0; i< sizeof(relayPin) / sizeof(relayPin[0]); i++)
  {
    if (message.sensor == i)
    {
      if (message.type == V_LIGHT)
      {
        relayState[i] = message.getBool();
        digitalWrite(relayPin[i], relayState[i]? RELAY_ON : RELAY_OFF); 
        gw.saveState(relayPin[i], relayState[i]);
      }
    }
  }
}

BulldogLowell

@Terence-Faul said:

gw.saveState(relayPin[i], relayState[i]);

if you are not relying on settings saved to EEPROM to recover its state after a power cycle, you can also comment our or remove this line:

gw.saveState(relayPin[i], relayState[i]);

it saves the state of the relay each time it gets changed.

glad to see you are learning and have it working just the way you like.

Terence Faul

Thanks a stack, it is slow.

If I did want to restore the relay state how would I change the code?

At the moment if I leave that code in when the arduino boots it cycles each relay on and then off, instead of just restoring the previous state

Anthony Straw

go for SSR instead of hk mechical relay it will stop working after 10,000 clicked ... cheap brand

jeylites

I got the same question as @Terence-Faul, How to change the code to have the Relay to restore its previous state. I tried a couple of things but can't seem to get it to work. Thanks @BulldogLowell for the script !

BulldogLowell

@jeylites ,

so, try to look over the example just above the one that worked for @Terence-Faul

It included saving states to EEPROM and retrieving them in setup( ).

C.r.a.z.y.

How do you seperate relay power?

What is your solution to drop 220v AC to 5v DC?

jeylites

@C.r.a.z.y. You could get a USB power adapter... see Ebay link below. On my current setup, I'm using a 2.1A+1A Dual USB 2-Ports. I use the the 1A port for the Arduino and the 2.1A port for the relays. On the relay board, you have VCC, IN1 & GND. IN1 gets connected to the Arduino, VCC to the second power supply or power port. The GND is shared between both power ports. Hope that helps!

http://www.ebay.com/sch/i.html?_from=R40&_trksid=p2141725.m570.l1313.TR12.TRC2.A0.H0.XUSB+Wall+Charger+.TRS0&_nkw=USB+Wall+Charger+&ghostText=&_sacat=0

jeylites

@BulldogLowell

I have gone through the script like 8 times with no luck at all. Are you suggesting the mistake in in the EEPROM retrieval section ?

#include <MySensor.h>
#include <SPI.h>
#include <Bounce2.h>

#define RELAY_ON 1 //switch around for realy HIGH/LOW state
#define RELAY_OFF 0

//
MySensor gw;
const int relayPin[] = {7, 8, A0, A1};
const int buttonPin[sizeof(relayPin) / sizeof(relayPin[0])] = {3, 4, 5, 6};
byte oldValue[sizeof(relayPin) / sizeof(relayPin[0])];
boolean relayState[sizeof(relayPin) / sizeof(relayPin[0])];

Bounce debouncer[sizeof(relayPin) / sizeof(relayPin[0])];

MyMessage msg[sizeof(relayPin) / sizeof(relayPin[0])];//(sensor,type);

void setup()
{
Serial.begin(115200);
gw.begin(incomingMessage, AUTO, true);
//
// or you can try:
// gw.begin(incomingMessage, <Your_Node_ID>, false); // where Your_Node_ID is a number from 1 to 254
//
delay(250);
gw.sendSketchInfo("MultiRelayButton", "0.9b");

delay(250);
for (int i = 0; i < sizeof(relayPin) / sizeof(relayPin[0]); i++)
{
msg[i].sensor = i;
msg[i].type = V_LIGHT;
debouncer[i] = Bounce();
debouncer[i].attach(buttonPin[i]);
debouncer[i].interval(5);
pinMode(buttonPin[i], INPUT_PULLUP);
digitalWrite(relayPin[i], RELAY_OFF); //Start State of Relays
pinMode(relayPin[i], OUTPUT);
gw.present(i, S_LIGHT);
delay(250);
}
//retreive from EEPROM last states
for (int i = 0; i < sizeof(relayPin) / sizeof(relayPin[0]); i++)//>>>>>>> I'm sure that this is causing all relays to turn on and off at startup
{
relayState[i] = gw.loadState(i);
digitalWrite(relayPin[i], relayState[i]? RELAY_ON:RELAY_OFF);
//gw.send(msg[i].set(relayState[i]? true : false), true);
delay(250);
}

}

void loop()
{
gw.process();
for (byte i = 0; i < sizeof(relayPin) / sizeof(relayPin[0]); i++)
{
debouncer[i].update();
byte value = debouncer[i].read();
if (value != oldValue[i] && value == 0)
{
relayState[i] = !relayState[i];
digitalWrite(relayPin[i], relayState[i]);
gw.send(msg[i].set(relayState[i]? true : false), true);
}
oldValue[i] = value;
}
}
//
void incomingMessage(const MyMessage &message)
{
if (message.isAck())
{
Serial.println(F("This is an ack from gateway"));
}
for (byte i = 0; i< sizeof(relayPin) / sizeof(relayPin[0]); i++)
{
if (message.sensor == i)
{
if (message.type == V_LIGHT)
{
relayState[i] = message.getBool();
digitalWrite(relayPin[i], relayState[i]? RELAY_ON : RELAY_OFF);
gw.saveState(relayPin[i], relayState[i]);
}
}
}
}```





#include <MySensor.h>
#include <SPI.h>
#include <Bounce2.h>

#define RELAY_ON 1  //switch around for realy HIGH/LOW state
#define RELAY_OFF 0

//
MySensor gw;
const int relayPin[] = {7, 8, A0, A1};
const int buttonPin[sizeof(relayPin) / sizeof(relayPin[0])] = {3, 4, 5, 6};
byte oldValue[sizeof(relayPin) / sizeof(relayPin[0])];
boolean relayState[sizeof(relayPin) / sizeof(relayPin[0])];

Bounce debouncer[sizeof(relayPin) / sizeof(relayPin[0])];

MyMessage msg[sizeof(relayPin) / sizeof(relayPin[0])];//(sensor,type);

void setup()  
{  
  Serial.begin(115200);
  gw.begin(incomingMessage, AUTO, true);
  //
  // or you can try:
  // gw.begin(incomingMessage, <Your_Node_ID>, false); // where Your_Node_ID is a number from 1 to 254
  //
  delay(250);
  gw.sendSketchInfo("MultiRelayButton", "0.9b");
  
  delay(250);
  for (int i = 0; i < sizeof(relayPin) / sizeof(relayPin[0]); i++)
  {
    msg[i].sensor = i;
    msg[i].type = V_LIGHT;
    debouncer[i] = Bounce();
    debouncer[i].attach(buttonPin[i]);
    debouncer[i].interval(5);
    pinMode(buttonPin[i], INPUT_PULLUP);
    digitalWrite(relayPin[i], RELAY_OFF); //Start State of Relays
    pinMode(relayPin[i], OUTPUT);
    gw.present(i, S_LIGHT);
    delay(250);
  }
  //retreive from EEPROM last states
  for (int i = 0; i < sizeof(relayPin) / sizeof(relayPin[0]); i++)//>>>>>>> I'm sure that this is causing all relays to turn on and off at startup
  {
    relayState[i] = gw.loadState(i);
    digitalWrite(relayPin[i], relayState[i]? RELAY_ON:RELAY_OFF);
    //gw.send(msg[i].set(relayState[i]? true : false), true);
    delay(250);
  }
  
}

void loop() 
{
  gw.process();
  for (byte i = 0; i < sizeof(relayPin) / sizeof(relayPin[0]); i++)
  {
    debouncer[i].update();
    byte value = debouncer[i].read();
    if (value != oldValue[i] && value == 0)
    {
      relayState[i] = !relayState[i];
      digitalWrite(relayPin[i], relayState[i]);
      gw.send(msg[i].set(relayState[i]? true : false), true);
    }
    oldValue[i] = value;
  }
}
//
void incomingMessage(const MyMessage &message)
{
  if (message.isAck()) 
  {
    Serial.println(F("This is an ack from gateway"));
  }
  for (byte i = 0; i< sizeof(relayPin) / sizeof(relayPin[0]); i++)
  {
    if (message.sensor == i)
    {
      if (message.type == V_LIGHT)
      {
        relayState[i] = message.getBool();
        digitalWrite(relayPin[i], relayState[i]? RELAY_ON : RELAY_OFF); 
        gw.saveState(relayPin[i], relayState[i]);
      }
    }
  }
}

AWI

@jeylites You are initializing the relays with the "Start State of Relays" and afterwards with the state from EEPROM. That could cause the on/off switching at startup.

Chaotic

To Elaborate on what AWI said

@jeylites said:

digitalWrite(relayPin[i], RELAY_OFF); //Start State of Relays
...
digitalWrite(relayPin[i], relayState[i]? RELAY_ON:RELAY_OFF);

You have both of those lines in your Setup() The first one is setting the relay to off the second one is checking the EEPROM value

Remove the first one and the problems should be fixed (assuming the EEPROM data is correct.

jeylites

@Chaotic , @AWI

//digitalWrite(relayPin[i], RELAY_OFF); //Start State of Relays
I did as you say and removed the above line but still no luck. The relay turns on/off on every start up. And it doesn't hold the last state. I compared My Sensor's script with Bulldogs's and found lots of similarities but can't seem to find the fault.

AWI

@jeylites I took a deeper look at your sketch and think I found the cause. You are writing and reading from different EEPROM locations. The "standard" way is to use the "Sensor number" as location reference. Whereas the sketch also used "relayPin[i]" to store values. I took the liberty of adapting your sketch as I found it hard to read. Also added storing the value in EEPROM after pushing a button.
I have not build the circuit so were not able to test it in real life :) Have fun and let me know if it worked for you.

https://codebender.cc/sketch:92964

jeylites

@AWI Thank you so much. This is awesome! I bet so many people is going to benefit from this script.

I had some time off today and ran the script. It didn't work the fist time, but I made some minor changes to "Relay_ON & Relay_OFF and also activated " Start State of Relays" and it worked. I'm guessing it didn't the first time because my relays are Active Low.

Also, while toying around with it, I realize that it doesn't send back the status of the relay & actuator to the Controller (Homeseer). On Homeseer, suppose you have it at “ON” there is a maker that indicates its state and vice versa. I noticed that worked on Binary Switch & Motion Sensor but not on Relay & Relay Button Actuator sketch. Don't know if it's a script or Homeseer issue though.

Again, thanks, everyone who helped pointing out the problem. I'm new to My Sensor and I'm glad to be here and making progress.

#define RELAY_ON 0 //switch around for relay HIGH/LOW state
#define RELAY_OFF 1

pinMode(Relays[i].relayPin, OUTPUT);

jeylites

@AWI I just realize another thing. The is a long delay for the sensor to sends its presentation during inclusion mode. Is there something to do with the number of sensors in one node or sketch? Essentially, I would have to reset the sensor several times to get it discovered.
It's pretty quick on a single or double sensor node.

AWI

@jeylites I don't have experience with Homeseer as controller.and don't fully understand your questions, so just give it a shot..

The multi relay sketch is firing a large amount of messages to the gateway/ controller during presentation. If you have transmission problems and/or a slow controller it could get into trouble. I suggest you monitor on the serial port of the sensor if there are "fail" messages. (need to turn on DEBUG = default) If there are it could be a power problem or...

The real fun (and headache) is in the debugging :) The serial monitor can give you lost of clues

Depending on how long is "long". there are also 250 ms delays between the individual presentations of the relays. ..'
have fun ;)

jeylites

@AWI

I think you're right on the controller. I've been getting some glitch running HS3 on Mac. I don't know if its a plugin or software issue, but I will like to get to the bottom of it. In addition, a buddy of mine is loaning his Vera edge so I might run some test on that too.

Meanwhile, will take your advice on serial monitor and will let you know the outcome. Cheers!

quocanhcgd

Thanks for sharing.
I have a problem with this scripts. My relay auto start then off every gateway check node. How can i fix it?
Thanks

Multi Button Relay switch

16

11.7k

11.2k

113.0k