Thanks for your pointers.

So clearTimeout will throw an error because self.timer is not defined so I defined it as null at the top, and didn’t bother checking if it was null or not before clearing it.

Once this was done we were in the same boat as before with a pir detection of 1 before the timeout has fired resulting in the mirror turning off and on again which is annoying. This is fixed by adding an onstate variable and simply not executing the activatemirror if its already activated. I just moved cleartimeout outside of that check otherwise it won’t work.

My working code below since I find it annoying when people don’t come back when they solve a problem:

// Subclass socketNotificationReceived received.
socketNotificationReceived: function(notification, payload) {
if (notification === ‘CONFIG’ && this.started == false) {
const self = this;
this.config = payload;
self.timer = null;
self.onState = 0;

  //Log.info('PIR: ' + this.name);

  //Setup pins
  this.pir = new Gpio(this.config.sensorPIN, 'in', 'both');

  if (this.config.relayPIN) {
    this.relay = new Gpio(this.config.relayPIN, 'out');
    this.relay.writeSync(this.config.relayOnState);
    exec("/opt/vc/bin/tvservice --preferred && sudo chvt 6 && sudo chvt 7", null);
  }

  //Detected movement
  this.pir.watch(function(err, value) {
    if (value == 1) {
     clearTimeout(self.timer);
    if(self.onState != 1){
      self.sendSocketNotification("USER_PRESENCE", true);
      if (self.config.powerSaving){
        self.activateMonitor();
        self.onState = 1;
      }
      }
     }
    else if (value == 0) {
      if(self.onState != 0){
       self.timer = setTimeout(function(){
          self.sendSocketNotification("USER_PRESENCE", false);
          if (self.config.powerSaving){
            self.deactivateMonitor();
            self.onState = 0;
          }
      }, self.config.offDelay);
      }
    }

});

  this.started = true;

} else if (notification === 'SCREEN_WAKEUP') {
  this.activateMonitor();
}

}

});

@lsfourwheeler yes passed it in. Assumed that was how it was supposed to be done.

@bibi it does. Perfectly.

Sorry I didn’t quote the code properly.

Thanks for your pointers.

So clearTimeout will throw an error because self.timer is not defined so I defined it as null at the top, and didn’t bother checking if it was null or not before clearing it.

Once this was done we were in the same boat as before with a pir detection of 1 before the timeout has fired resulting in the mirror turning off and on again which is annoying. This is fixed by adding an onstate variable and simply not executing the activatemirror if its already activated. I just moved cleartimeout outside of that check otherwise it won’t work.

My working code below since I find it annoying when people don’t come back when they solve a problem:

// Subclass socketNotificationReceived received.
socketNotificationReceived: function(notification, payload) {
if (notification === ‘CONFIG’ && this.started == false) {
const self = this;
this.config = payload;
self.timer = null;
self.onState = 0;

  //Log.info('PIR: ' + this.name);

  //Setup pins
  this.pir = new Gpio(this.config.sensorPIN, 'in', 'both');

  if (this.config.relayPIN) {
    this.relay = new Gpio(this.config.relayPIN, 'out');
    this.relay.writeSync(this.config.relayOnState);
    exec("/opt/vc/bin/tvservice --preferred && sudo chvt 6 && sudo chvt 7", null);
  }

  //Detected movement
  this.pir.watch(function(err, value) {
    if (value == 1) {
     clearTimeout(self.timer);
    if(self.onState != 1){
      self.sendSocketNotification("USER_PRESENCE", true);
      if (self.config.powerSaving){
        self.activateMonitor();
        self.onState = 1;
      }
      }
     }
    else if (value == 0) {
      if(self.onState != 0){
       self.timer = setTimeout(function(){
          self.sendSocketNotification("USER_PRESENCE", false);
          if (self.config.powerSaving){
            self.deactivateMonitor();
            self.onState = 0;
          }
      }, self.config.offDelay);
      }
    }

});

  this.started = true;

} else if (notification === 'SCREEN_WAKEUP') {
  this.activateMonitor();
}

}

});

I’ve been using the PIR sensor module and if I want to change my PIR on delay in hardware its a pain to take a really neat installation down. Instead I wanted to set the delay to the minimum and then have a configurable offDelay of say 30 seconds.

I added offDelay:30000 to the config and instead of calling deactivateMonitor I used timerID = setInterval(deactivateMonitor,self.config.offDelay). If detecting on again before this runs I called clearInterval(); It works but isn’t quite right because if the PIR becomes 0 and then 1 again before the timer is activated the monitor goes off and on again.

I’m not really understanding the node_helper.js script and wondered if someone could point me in the right direction.

1. the socketreceivednotification gets called on startup with config set, this sets up the pirwatcher. - but does the pir watcher block or when its state changes it simply calls itself again?
2. im watching .pm2/log/ errors and output and only really see when something is catastrophically wrong with my code. Is there a more convenient way to pipe something out from the node_helper?
3. I am persisting state information by adding it to the config because I thought that gets passed back in the “main loop”. I looked at some other modules for a hint but couldn’t really find the answer.
4. the activate or deactivate functions cannot see config if they are called async so I guess I’ve fallen into the typical trap of not really getting the self. this. thing in this context.

So I’m obviously missing something bit fundamental, I write C normally and frequently misinterpret inline JS functions. Its also hard to concentrate on this in the evening with a 2 year old swinging on my head.