MMM-iFrame-Ping

bolish

Review :

Using a list of several URL is working. Seems more “slow / laggy” than with iFrame-Ping and only one URL. Strange but OK.
URL rotation is working great. (maybe a bit slow because of reloading needs, but I guess than there is not much you can do for that?)
URL NEXT / PREV / PLAY / PAUSE is working like a charm. Which was my initial need, so perfect.

So all in all, a major and well managed update. Good job @AgP42 .

One thing :

Even by using the same size (% or px) parameter as in previous iFrame-Ping, the picture is much more smaller (tiny!!!) in this new module. Strange but nothing critical.

I’m displying youtube channels and “middle_center” parameter.

Nevertheless, when I increase the size of it, the centering is not working well. Indeed, the margin remains the same on left side, meaning the frame is off-centered on the right side of the screen.

Hope this can be fixed easily (I tried to look at it myself, .css, etc… but I’m not skilled enough). Maybe it’s something I’m doing wrong on my side…

AgP42

@bolish, thanks for your very fast review ! I just upload an update that (should) solve the lagging issue (and on the same time this issue solved the problem of several instances !)

And your display issue was coming from the fact that I send on GitHub my css file with modifs for me, I update it too, so you should be able to get back your previous params

bolish

@AgP42
Centering frame is OK now.
Lagging seems to be much better.
But it seems that new issue is happening. I get constant (uncontrolled) change between status Playing and Stopped which is stopping the video.

Strange.

bolish

I think it’s because of PIR Sensor “USER PRESENCE” notification!!!
(I don’t wnat any interaction between PIR and video), I only use PIR to turn screen ON / OFF.

And this issue is not new since your morning correction…It’s new for me because I installed PIR this morning!!!

Regards

AgP42

@bolish said in MMM-iFrame-Ping:

I think it’s because of PIR Sensor “USER PRESENCE” notification!!!
(I don’t wnat any interaction between PIR and video), I only use PIR to turn screen ON / OFF.

And this issue is not new since your morning correction…It’s new for me because I installed PIR this morning!!!

Regards

But why do you want your video to run if your screen is off ?

bolish

No you’re right I don’t.

I believe I understood the issue :

It seems the PIR sensor I’m using is very sensitive and triggering “USER PRESENCE FALSE” very easily.
I’m able to manage it in MMM-PIR sensor module thanks to the additional delay option which I put to 30 seconds which is preventing the screen to turn ON / OFF every 2 seconds.

But this is then triggering the issue I’m referring to above for MMM-SmartWebDisplay.

Does that makes sense? It’s the only explanation I found for now.

bolish

maybe you should let your module more generic that what you currently did.
If someone wants to STOP the video based on any other notification (like USER PRESENCE, false) he will always have the possibility to use MMM-NotificationTrigger to convert user presence to stop.

Up to you.

If you’re not keen to update that point (which I could understand), I believe I can maybe modify things a little bit myself to manage it.

Maybe the modification should be done directly in MMM-PIR Sensor, which should only trigger PRESENCE FALSE once the additional delay is over and not before…

AgP42

@bolish
Yes I perfectly understand the issue you are facing, in fact I also did some contribution to the module PIR sensor :smiling_face_with_open_mouth_closed_eyes:

Yes the delay on the module apply only about the screen off but the “USER_PRESENCE” is send anyway directly according to the sensor input.

Normaly on your sensor you have 2 screws, one to change the sensibility of the sensor and the other one to adjust the time delay more infos here

Did you check that ?

But you are also right that I could let the choice to the user to use this USER_PRESENCE or not, I will check what I can do for that.

bolish

The sensor I’m using doesn’t offer this option, unfortunatly.
link text

Thinking of it again :

Would it not be more “sustainable” to PR MMM-PIRSensor by putting a second notification trigger “USER_PRESENCE, Screen Off” after the timeout delay.
Then, we could play within MMM-SmartWebDisplay with that one instead of the only USER PRESENCE false, true.
By doing that + adding the option to use it or not, it would make the full thing more reliable and generic.

If I believe it’s somewhere here :

                else if (value == valueOff) {
                    self.sendSocketNotification('USER_PRESENCE', false);
                    if (!self.config.powerSaving){
                        return;
                    }

                    self.deactivateMonitorTimeout = setTimeout(function() {
                        self.deactivateMonitor();
                    }, self.config.powerSavingDelay * 1000);
                }
            });

AgP42

@bolish
On the link you sent, this sensor is exactly the same as mine and its allow to change the timer ! ;-)

The best way to make that clean will be to change the PIR-sensor module to send the notification only at the end of the timer, but I have no time to do so now…

What you can do for now to desactivate the Stop at the reception of a USER_PRESENCE = false notification is to go on the file MMM-SmartWebDisplay.js, on the function “GestionUpdateIntervalSWD”, then comment (//) the line "this.StopDisplay();

In fact I have no time to change the module PIR-Sensor now because I am about to finish an update of the module MMM-RemoteControl to be able to use "Next/Prev/Play/Pause/Stop directly on RemoteControl :

bolish

oh! I will look on the module again as I didn’t saw the famous screws you’re referring too…

Don’t worry @AgP42 , you already did a great job! Thanks again for that.

For the time being I will leave it like that (with your workaround) and if I have time on my side, I will maybe try to update MMM-PIRSensor. Maybe I could do it with trial and error method.

Thanks to the latest update of your module and MMM-GroveGestures (from @Sean ) , I end up with a perfect user experience :

On my page 1 (main MM page), up / down gestures are handling news RSS details, etc…
On my page 2, I have your iFrame and I’m able to change youtube live channels by up / down gestures also.

Perfect!

The Remote Control you are working on looks great also!

A bientôt et bonne continuation!! (tu as laissé qques indices dans ton code :-)

AgP42

:winking_face:

Here is the new post about MMM-SmartWebDisplay, and the modification of MMM-RemoteControl to control it !

lpgjg

Hey everyone,
Hoping I can jump in here at the beginners level. Been trying to load a number of YouTube modules for a week to no avail.

I’m more old school so this stuff doesn’t come natural to me and I’ve been trying to get iFrame-Ping to work the last few days.

Long story short, all I get is YouTube trying to load on the MM and then “Video unavailable” where the video should be.

The rest of the MM and its modules are working as should be. Strangely enough, I can click on the iframe-Ping module and the whole magic mirror disappears and the Pi switches to YouTube, full screen?

The only way to get out of youtube at this point is with “ctrl Q”.

Can anyone help?

AgP42

@lpgjg
Hello,

can you send your configuration ?

And also the log ? (when the MagicMirror is running, Ctrl+i and then filter to “iFrame-Ping” and copy the data°

Thanks,
AgP42

lpgjg

Will try to do so when I get home from work tonight. May take a few attempts as this very alien to me given that I’m probably older than most all of you…or your dads.

May be worth noting: After a few attempts I noticed if I used my companies playlist, or that of Bombardier, Nord-Gear or any other company / commercial play list, it works fine. Always begins at the same place when started but the module shows up and plays the videos.

But if I try to add a Pop music playlist or any of the popular artist out their, it gives me the “video unavailable.”

As if it’s a copy right issue when going through the mirror.

BTW: I’m in Montreal, Canada

sdetweil

@lpgjg I’m 67… some of us are older…

AgP42

@lpgjg @sdetweil,

the good news is that technology don’t care about the user age !

and I also don’t care, old or young, gives me your logs and I will help you ! :smiling_face_with_halo: :face_with_tears_of_joy:

lpgjg

Nothing happens when I hit “ctrl” and then the letter ”i” while the mirror is running. Or was I actually suppose to add the plus sign “+”…and I tried that as well. Only thing that works is ctrl Q.

As to my configuration. I assume this is what you are asking for:

{
module: ‘MMM-iFrame-Ping’,
position: ‘bottom_right’, // This can be any of the regions.
config: {
// See ‘Configuration options’ for more information.
url: “https://www.youtube.com/embed/LH4Y1ZUUx2g?list=PLw-VjHDlEOguL2K_NwIl_tYcs9_FSt5v_&autoplay=1&modestbranding=1&iv_load_policy=3”, //url to display
height:“80%”,
width:“80%”,
autoRefresh: false, //set to false for video
updateInterval: 1, //in min. Only if autoRefresh: true
displayLastUpdate: true,
width: “100%”, // Optional. Default: 100%
height: “400px”, //Optional. Default: 100px
scrolling: “no”
}

sdetweil

@lpgjg ctrl-shift-i (letter I) to open the dev window, then select the console tab

AgP42

@lpgjg
This video is not available on youtube, so yes it is normal that the magicmirror cannot load it.

Is it a private video ?