MMM-PIR doesn´t turn off monitor since update MM 2.7

toucherts

@shgmongohh did you fix it ? TutuApp Adam4adam AppValley

shgmongohh

I get a failure message and I don´t know, how to fix it. npm audit fix doesnt work.

pi@raspberrypi:~/MagicMirror/modules/MMM-PIR-Sensor $ npm install electron-rebuild
+ electron-rebuild@1.8.4
updated 1 package and audited 301 packages in 17.922s
found 1 high severity vulnerability
  run `npm audit fix` to fix them, or `npm audit` for details

pi@raspberrypi:~/MagicMirror/modules/MMM-PIR-Sensor $ npm audit

                       === npm audit security report ===


                                 Manual Review
             Some vulnerabilities require your attention to resolve

          Visit https://go.npm.me/audit-guide for additional guidance


  High            Arbitrary File Overwrite

  Package         tar

  Patched in      >=4.4.2

  Dependency of   electron-rebuild [dev]

  Path            electron-rebuild > node-gyp > tar

  More info       https://nodesecurity.io/advisories/803

found 1 high severity vulnerability in 301 scanned packages
  1 vulnerability requires manual review. See the full report for details.

djay07

I did the recommended above, but it’s still just nothing happening. Any further ideas?

I have a RCWL-0516 connected right now and a small python script which shows me it’s activity. but the module won’t shutdown the screen after minutes without motion (delay set to 10 secs).

Edit:
Maybe i should add:
I tried this with MMM-PIR -> interface not starting, error:

WARNING! Could not validate config file. Starting with default configuration. Please correct syntax errors at or above this line: ReferenceError: NodeHelper is not defined
    at Object.<anonymous> (/home/pi/MagicMirror/modules/MMM-PIR/node_helper.js:11:18)
    at Object.<anonymous> (/home/pi/MagicMirror/modules/MMM-PIR/node_helper.js:67:3)
    at Module._compile (internal/modules/cjs/loader.js:711:30)
    at Object.Module._extensions..js (internal/modules/cjs/loader.js:722:10)
    at Module.load (internal/modules/cjs/loader.js:620:32)
    at tryModuleLoad (internal/modules/cjs/loader.js:559:12)
    at Function.Module._load (internal/modules/cjs/loader.js:551:3)
    at Module.require (internal/modules/cjs/loader.js:658:17)
    at require (internal/modules/cjs/helpers.js:20:18)
    at loadModule (/home/pi/MagicMirror/js/app.js:127:17)
App threw an error during load
ReferenceError: NodeHelper is not defined
    at Object.<anonymous> (/home/pi/MagicMirror/modules/MMM-PIR/node_helper.js:11:18)
    at Object.<anonymous> (/home/pi/MagicMirror/modules/MMM-PIR/node_helper.js:67:3)
    at Module._compile (internal/modules/cjs/loader.js:711:30)
    at Object.Module._extensions..js (internal/modules/cjs/loader.js:722:10)
    at Module.load (internal/modules/cjs/loader.js:620:32)
    at tryModuleLoad (internal/modules/cjs/loader.js:559:12)
    at Function.Module._load (internal/modules/cjs/loader.js:551:3)
    at Module.require (internal/modules/cjs/loader.js:658:17)
    at require (internal/modules/cjs/helpers.js:20:18)
    at loadModule (/home/pi/MagicMirror/js/app.js:127:17)

And MMM-PIR-Sensor is not reacting at all.

sdetweil

@djay07 after you did the git pull update, did you do npm install in the MagicMirror folder
AND every module that has a package.json file???

you must

djay07

@sdetweil
morning,
well i didn’t do it in every single subfolder where a package.json is placed, since i didn’t expect this to be necessary.

I did:

git pull
in the module folder “npm install”
npm install electron-rebuild
./node_modules/.bin/electron-rebuild

the result is the above.
Currently i solved it with external scripts running in system.d, but if you tell me i need to run “npm install” in each folder with a package.json, i would give it a try, just to report back.

lxne

@djay07 Hi, have you been able to solve the MMM-PIR problem? I am stuck with the same thing.

FruityBebbles

@lxne @djay07 Same…

sdetweil

@FruityBebbles sorry, could you tell me more?? same what?

can’t tell how far back u are responding…

sdetweil

@shgmongohh just fyi… the npm audit fix will NOT fix everything… these are warnings anyhow… so, no problems

sdetweil

@djay07 @lxne @FruityBebbles ok, node_helper missing… heres how to fix

cd ~/MagicMirror
git checkout modules/node_modules/node_helper/index.js

then restart your magic mirror however you do it… reboot, pm2, some script. npm start