Relay with button example sketch, works...but not as expected

[BulldogLowell]

@roffy said:

gw.send(msg.set(state?false:true), true);

also, are you aware that you are not changing the variable "state" with a button press?

nor are you affecting the relay, fyi...

void loop() 
{
  gw.process();
  debouncer.update();
  // Get the update value
  int value = debouncer.read();
  if (value != oldValue && value==0)
  {
      state = !state;
      digitalWrite(RELAY_PIN, state);
      gw.send(msg.set(state, true)); // Send new state and request ack back
  }
  oldValue = value;
} 

[BulldogLowell]

@roffy said:

gw.saveState(CHILD_ID, state);

void loop() 
{
  gw.process();
  debouncer.update();
  // Get the update value
  int value = debouncer.read();
  if (value != oldValue && value==0)
  {
      state = !state;
      digitalWrite(RELAY_PIN, state);
      gw.saveState(CHILD_ID, state);// forgot to save the new state to EEPROM
      gw.send(msg.set(state, true)); // Send new state and request ack back
  }
  oldValue = value;
} 

gw.saveState(CHILD_ID, state);

[BulldogLowell]

@rvendrame said:

Try changing this line

if (value != oldValue && value==0) {

to this

if (value != oldValue) {

I don't think you want to affect a state change on both sides of the button press....

[rvendrame]

@BulldogLowell said:

I don't think you want to affect a state change on both sides of the button press....

... if you are thinking in a pushbutton , yes I agree with you --- But what about a regular on-off switch?

[BulldogLowell]

@rvendrame said:

@BulldogLowell said:

I don't think you want to affect a state change on both sides of the button press....

... if you are thinking in a pushbutton , yes I agree with you --- But what about a regular on-off switch?

For an on/off switch? well you would probably not have that on a device that is also controlled remotely/wirelessly.

[rvendrame]

@BulldogLowell said:

For an on/off switch? well you would probably not have that on a device that is also controlled remotely/wirelessly.

Why not? I do have the original on/off switch from my lights for example. I want to control them via automation but also via wall switch...

[BulldogLowell]

@rvendrame said:

@BulldogLowell said:

For an on/off switch? well you would probably not have that on a device that is also controlled remotely/wirelessly.

Why not? I do have the original on/off switch from my lights for example. I want to control them via automation but also via wall switch...

yeah, but every wall switch I have, which are integrated into my home automation, are not two position switches. Anyways, the OP has to chime in here... in the rare event he's using a two-position switch, well your code may still need to change. Time to hear from the OP.

[rvendrame]

@BulldogLowell said:

also, are you aware that you are not changing the variable "state" with a button press?

Just to mention, state is being changed:

 if (value != oldValue && value==0)
  {
      state = !state;   <<<<<<<<<<<<<<<<<<<<<<<<<<<<<
      digitalWrite(RELAY_PIN, state);
      gw.saveState(CHILD_ID, state);// forgot to save the new state to EEPROM
      gw.send(msg.set(state, true)); // Send new state and request ack back
  }

[BulldogLowell]

@rvendrame

@rvendrame said:

Just to mention, state is being changed:

yes, I added that... it was missing from OP's code

[hek]

Yes, as you are aware not the WithButton example was created to be used with a monostable button.

Haven't really thought about bi-stable but should be doable.

The state is actually not stored in EEPROM until ack message comes back from gateway. It is also only then the actual realy is flipped.

As you already mention above removing value==0 would send of a change when flipping button. Not sure if one should pick up oldValue from eeprom at startup as well.

[roffy]

wow thanks for all the suggestions guys! sorry for the slow reply, this is the first time iv actually been able to sit at my computer since i posted, if only we didnt have to work to pay bills id be here all day...ahhh if only..... anyway!

Yes i am using a pushbutton not a rocker switch so i would need it to only do something on the down press and ignore the upward contact on the release of the button.

Ill try changing the script as explained above and see what that does, (as you have probably guessed by now my arduino tinkering only covered the first 10 or so lessons in the basic tutorial, then i went off and only looked up specific coding for the small projects i was working on....and most of which i have forgotten as i haven't used them for a long time :( need to brush up on what i should already know i guess)

anyway i shall post results as soon as i try out the suggestions. i think you all get what im trying to achieve here! thanks again,