Let's Control It

manjh wrote: ↑15 Jul 2017, 18:08 I've set up a Sonoff 4CH with ESP Easy and am now fine tuning the rules.
Besides simple "on" and "off", I've also created rules for the buttons to toggle the relay.
Works fine.
But I have also set two of the relays to be switched on after reboot. It works, the two relays are activated. But after that the two corresponding buttons have no effect!
In detail: relais 1 and 4 are switched off, 2 and 3 are switched on after reboot.
Then buttons 1 and 4 work fine, buttons 2 and 3 not.
It can be solved by sending an "off" command, then buttons 2 and 3 are OK again.

I tested this using the command field: evoking my "toggle" rule has no effect, turning it "off" resets the state and from that moment on everything works.
It feels like turning the relay on after boot "freezes" that state until an explicite "off" is done.

What is wrong here? Am I doing something wrong or is this a bug?

Another thing I noticed: the relay devices are defined with both "send boot state" and "send data", but the two relays that are switched on after reboot show up in Domoticz as "off", even though they are really on.

LisaM wrote: ↑15 Jul 2017, 21:09 Rules + devices screenshot please!!!

grovkillen wrote: ↑16 Jul 2017, 10:32 Please study this information.

https://www.letscontrolit.com/wiki/inde ... figuration

grovkillen wrote: ↑16 Jul 2017, 14:44 Since you didn't mention stuff like this it is hard to help. But you could add a delay before setting the GPIO state. Right on boot might be a tad too fast?

manjh wrote: ↑16 Jul 2017, 09:37

LisaM wrote: ↑15 Jul 2017, 21:09 Rules + devices screenshot please!!!

Done.

LisaM wrote: ↑16 Jul 2017, 19:39

manjh wrote: ↑16 Jul 2017, 09:37

LisaM wrote: ↑15 Jul 2017, 21:09 Rules + devices screenshot please!!!

Done.

Screenshots of the button devices?

grovkillen wrote: ↑16 Jul 2017, 22:42 Those images are broken for me.

manjh wrote: ↑16 Jul 2017, 20:18 Done.

On Button2#State do
    If [Relais2#State]=1
        gpio,5,0
    Else
        gpio,5,1
    EndIf
EndOn

LisaM wrote: ↑17 Jul 2017, 00:16

manjh wrote: ↑16 Jul 2017, 20:18 Done.

I'm missing some toggle rules like this one:

Code: Select all

On Button2#State do
    If [Relais2#State]=1
        gpio,5,0
    Else
        gpio,5,1
    EndIf
EndOn

manjh wrote: ↑17 Jul 2017, 00:31

LisaM wrote: ↑17 Jul 2017, 00:16

manjh wrote: ↑16 Jul 2017, 20:18 Done.

I'm missing some toggle rules like this one:

Code: Select all

On Button2#State do
    If [Relais2#State]=1
        gpio,5,0
    Else
        gpio,5,1
    EndIf
EndOn

I did not think they are relevant.At bootup I only use on and off.... that puts the two 'on" relays in a very strange state.

LisaM wrote: ↑17 Jul 2017, 00:36

manjh wrote: ↑17 Jul 2017, 00:31

LisaM wrote: ↑17 Jul 2017, 00:16
I'm missing some toggle rules like this one:

Code: Select all

On Button2#State do
    If [Relais2#State]=1
        gpio,5,0
    Else
        gpio,5,1
    EndIf
EndOn

I did not think they are relevant.At bootup I only use on and off.... that puts the two 'on" relays in a very strange state.

At bootup you put the relay to the on state, then when pushing the corresponding button you want it to change state? Since it's a push button, there's no on or off state but only toggle...

on Toggle2 do
  if [relais2#state]=1
    Event TurnOff2
  else
    Event TurnOn2
  endif
endon
on button2#state do
  Event Toggle2
endon

papperone wrote: ↑17 Jul 2017, 12:13 Can you try like this:

Code: Select all

on System#Boot do
  Event TurnOff1
  Event TurnOff2
  Event TurnOff3
  Event TurnOff4
  Event Toggle2
  Event Toggle3
endon

I can imagine your toggle events get confused when you set channels manually On at boot so above should work properly

manjh wrote: ↑17 Jul 2017, 09:13

Code: Select all

on Toggle2 do
  if [relais2#state]=1
    Event TurnOff2
  else
    Event TurnOn2
  endif
endon
on button2#state do
  Event Toggle2
endon

on Toggle2=1 do
  if [relais2#state]=1
    Event TurnOff2
  else
    Event TurnOn2
  endif
endon

LisaM wrote: ↑17 Jul 2017, 21:45

manjh wrote: ↑17 Jul 2017, 09:13

Code: Select all

on Toggle2 do
  if [relais2#state]=1
    Event TurnOff2
  else
    Event TurnOn2
  endif
endon
on button2#state do
  Event Toggle2
endon

Did you try :

Code: Select all

on Toggle2=1 do
  if [relais2#state]=1
    Event TurnOff2
  else
    Event TurnOn2
  endif
endon

manjh wrote: ↑17 Jul 2017, 22:12

LisaM wrote: ↑17 Jul 2017, 21:45

manjh wrote: ↑17 Jul 2017, 09:13

Code: Select all

on Toggle2 do
  if [relais2#state]=1
    Event TurnOff2
  else
    Event TurnOn2
  endif
endon
on button2#state do
  Event Toggle2
endon

Did you try :

Code: Select all

on Toggle2=1 do
  if [relais2#state]=1
    Event TurnOff2
  else
    Event TurnOn2
  endif
endon

No, I did not try that. But I don't see the point: all rules work just fine. The only problem is when I switch a relay on after boot. It then goes into a strange state, which can only be corrected by an explicit "off" command.

LisaM wrote: ↑17 Jul 2017, 22:20

manjh wrote: ↑17 Jul 2017, 22:12

LisaM wrote: ↑17 Jul 2017, 21:45
Did you try :

Code: Select all

on Toggle2=1 do
  if [relais2#state]=1
    Event TurnOff2
  else
    Event TurnOn2
  endif
endon

No, I did not try that. But I don't see the point: all rules work just fine. The only problem is when I switch a relay on after boot. It then goes into a strange state, which can only be corrected by an explicit "off" command.

Set a timer at boot and set state in the timer on a later time?

grovkillen wrote: ↑18 Jul 2017, 06:32 Since it is the two I2C pins it might be that these are still being active as such right at boot. (Just my wild guess, I haven't studied the code.)

grovkillen wrote: ↑18 Jul 2017, 10:48 Yes, that's considered a bug. Will you file it as an issue on GitHub?