Bugsounet and MMM-Pir

rkorell

@sdetweil @lif
For your information:

I was successful with installing MMM-Pir on my brand new Pi5/bookworm:

pi@MagicMirrorPi5:~ $ cd MagicMirror/modules
pi@MagicMirrorPi5:~/MagicMirror/modules $ tar -xf MMM-Pir.tar.gz 
pi@MagicMirrorPi5:~/MagicMirror/modules $ cd MMM-Pir/
pi@MagicMirrorPi5:~/MagicMirror/modules/MMM-Pir $ npm install

> MMM-Pir@2.4.4 preinstall
> echo ⚠ npm install will be deprecated on next release!.

⚠ npm install will be deprecated on next release!.

> MMM-Pir@2.4.4 install
> npm run setup


> MMM-Pir@2.4.4 presetup
> npm run dependencies && installer/preinstall.sh


> MMM-Pir@2.4.4 dependencies
> installer/dependencies.sh -d 'unclutter ddcutil cec-utils python3 python-is-python3 python3-gpiozero gpiod libgpiod2 libgpiod-dev'


Welcome to MMM-Pir v2.4.4

① ➤ dependencies installer

Checking OS...
OS Detected: linux-gnu (debian 12 aarch64)

Checking all dependencies...
All Dependencies needed are installed !

② ➤ Preinstall

No root checking...
Checking MMM-Pir: pi/pi
Checking MagicMirror: pi/pi

Installing @sdetweil sandbox fix...
fixing sandbox permissions

③ ➤ Install npm dependencies

> MMM-Pir@2.4.4 setup
> npm prune


up to date in 990ms

124 packages are looking for funding
  run `npm fund` for details

> MMM-Pir@2.4.4 postsetup
> installer/postinstall.sh -m -r


④ ➤ Postinstall

Minify Main code...
Found: 10 files to install and minify

Process File: MMM-Pir/MMM-Pir.js
Process File: MMM-Pir/node_helper.js
Process File: MMM-Pir/components/cronJob.js
Process File: MMM-Pir/components/governorLib.js
Process File: MMM-Pir/components/motion.js
Process File: MMM-Pir/components/motionLib.js
Process File: MMM-Pir/components/pirLib.js
Process File: MMM-Pir/components/screenDisplayer.js
Process File: MMM-Pir/components/screenLib.js
Process File: MMM-Pir/components/screenTouch.js
Done

Try to Disable screen saver...
Found: screen saver in gsettings
disable screensaver via gsettings was true and uint32 300
Found: screen saver in lightdm
disable screensaver via lightdm.conf
Found: screen saver in lxsession
disable screensaver via lxsession
Found: wayfire.ini

[WARN] There is some change for disable screen saver
[WARN] Please, don't forget to reboot your OS for apply the new configuration!
Done

Rebuild MagicMirror...
✔ Rebuild Complete
Done


MMM-Pir is now installed !

> MMM-Pir@2.4.4 postinstall
> echo ⚠ On next release, Please use: npm run setup

⚠ On next release, Please use: npm run setup

added 1 package in 31s

124 packages are looking for funding
  run `npm fund` for details
pi@MagicMirrorPi5:~/MagicMirror/modules/MMM-Pir $

I will have to play with the screen options (caused for sure by bookworm) because right now I get an error notification from MMM-Pir (screen error detected: dpms command error(mode 1) ) but there are several other modes to try …

Regards,
Ralf

sdetweil

@rkorell yeh, fun times

rkorell

@sdetweil yes! :-)

FrostByte

Hi everyone,

I didn’t quite get it right. Are there still problems with MMM-Pir?

I see that there are already two pull requests on https://github.com/Coernel82/MMM-Pir/pulls (one is from me). I would be willing to maintain the module, if no one else does.

Happy to hear from you @coernel

rkorell

@FrostByte .
Yes there are still “problems”.
@coernel has prepared a re-upload of bugsounet’s MMM-Pir.
But for some reasons this has failed.

Several entities are missing in his upload (including e.g. the core MMM-Pi.js and several other linked and necessary sub-programs (Described in more detail in this post).

I’ve copied a tar.gz file of my own local repository-copy here.

This copy is installable with npm install in the respective MMM-Pir directory in mirror Version 2.30 (NOT tested in brand new 2.31!).
It works without problems on a Pi4 with bullseye.
It does NOT work on a Raspi5/bookworm “out of the box”.
In the meantime I’ve identified and corrected the root cause for this (wrong parameter for the xrandr-call).
This correction is NOT reflected in the URL above because I’ve done this correction after the upload.
I can share this with you if you are interested in - only a very minor adoption and only for MY use-case (working without wayland and with x11).

From my side it would be really highly appreciated if somebody can take the responsibility for maintenance of this pretty module because I really like it very much.
I definitely NOT able to do this but I’m definitely willing to give any additional support in the course of my (tiny) abilities…

Let me know what you’re thinking about.

Warm regards,
Ralf

FrostByte

Okay, let’s go into detail. @rkorell

As already written by @sdetweil , the node_modules directory is created when npm run setup (or npm install as it should be called).

In addition, the ‘missing files’ are also created dynamically when this command is called. See:

lukas@smartmirror:~/MagicMirror/modules$ git clone https://github.com/Coernel82/MMM-Pir.git MMM-Pir-test
lukas@smartmirror:~/MagicMirror/modules$ cd MMM-Pir-test
lukas@smartmirror:~/MagicMirror/modules/MMM-Pir-test$ npm run setup
lukas@smartmirror:~/MagicMirror/modules/MMM-Pir-test$ ls -al components/
total 68
drwxr-xr-x  2 lukas lukas  4096 Apr  1 19:06 .
drwxr-xr-x 10 lukas lukas  4096 Apr  1 19:53 ..
-rw-r--r--  1 lukas lukas  4101 Apr  2 11:23 cronJob.js
-rw-r--r--  1 lukas lukas  2241 Apr  2 11:23 governorLib.js
-rw-r--r--  1 lukas lukas  3894 Apr  2 11:23 motion.js
-rw-r--r--  1 lukas lukas  1563 Apr  2 11:23 motionLib.js
-rw-r--r--  1 lukas lukas   757 Mar  3 19:09 MotionSensor.py
-rw-r--r--  1 lukas lukas  3755 Apr  2 11:23 pirLib.js
-rw-r--r--  1 lukas lukas  6068 Apr  2 11:23 screenDisplayer.js
-rw-r--r--  1 lukas lukas 19159 Apr  2 11:23 screenLib.js
-rw-r--r--  1 lukas lukas  1763 Apr  2 11:23 screenTouch.js

I use Wayland/labwc (Debian GNU/Linux 12 bookworm) on aarch64 Raspberry Pi 3B and everything is working for my Pir configuration. I did not use your copy.

So i am asking again, are there some runtime errrors? So please send me errrors logs.

The ‘build’ step in package.json is really very confusing. That’s why I’ve started to organise it: https://github.com/Coernel82/MMM-Pir/compare/main...LukasWestholt:MMM-Pir:main#diff-7ae45ad102eab3b6d7e7896acd08c427a9b25b346470d7bc6507b6481575d519

Best regards,
Lukas

FrostByte

@lif So in my experience, your error just means that you didn’t run ‘npm run setup’!

rkorell

@FrostByte said in Bugsounet and MMM-Pir:

As already written by @sdetweil , the node_modules directory is created when npm run setup (or npm install as it should be called).

This one I’ve missed or was to blonde, to see …

If the install is creating necessary files all “should” be fine.
Despite the fact that in the original screenLib.js is an error (for x11 - regarding handling because the calls are slightly different as coded).
Wayland does not work properly with MagicMirror on my Pi5 - so I’ve switched back to x11, found the above error and corrected this.
Not sure how to handle this in “release” and your maintenance of module.

What have I done?

See here:
This is line 311 (of screenLib.js ):

       exec("DISPLAY=:0 xrandr | grep 'connected primary'", //--rkorell
        // original : exec("xrandr | grep 'connected primary'",

this is line 489 (of screenLib.js ):

        // original exec(`xrandr --output ${this.screen.hdmiPort} --auto --rotate ${this.screen.xrandrRotation}`, (err) => {
          exec(`xrandr --output ${this.screen.hdmiPort} --primary --mode 2560x1440 --rate 59.951 --pos 0x0  --rotate ${this.screen.xrandrRotation}`, (err) => {

especially the second one was disturbing because the statement simply doesn’t work - this option “auto” is something one can find everywhere referenced but it never worked as far as I have tried to research. from own experience (see above) it don’t work. I guess “–auto” should recover the current settings -but it doesn’t (on bookworm!) . Hard coded screen resolution is definitely no candidate for broader distribution …

(First one “only” produces an error message)

These are my findings.
I’ve mistakenly (not aware of the “dynamically created” libraries/dependant modules) linked the problems of the thread owner with the missing libraries in HIS installation to the likewise missing modules in git clone…

Sorry for confusion!

Ralf

FrostByte

@rkorell Thanks for your answer! That makes total sense. Thanks for reporting your experience. I’ll keep that in mind.

With Wayland it took me tens of hours to rotate the display by 90 degrees via autostart and I often thought about migrating to X11 back. But in the end it worked.

lif

@FrostByte
I don’t know if it helps but the only way I could get my old DVI input monitor to turn on & off was with Wayland enabled and the commands

DISPLAY=:0.0 xrandr --output HDMI-1 --auto --rotate left
and
DISPLAY=:0.0 xrandr --output HDMI-1 --off

Nick