RetroPie forum home
    • Recent
    • Tags
    • Popular
    • Home
    • Docs
    • Register
    • Login

    Help testing: Random Video Screensaver on Main Branch

    Scheduled Pinned Locked Moved Ideas and Development
    emulationstatiovideo screensavscreensavervideotesting
    146 Posts 5 Posters 50.3k Views
    Loading More Posts
    • Oldest to Newest
    • Newest to Oldest
    • Most Votes
    Reply
    • Reply as topic
    Log in to reply
    This topic has been deleted. Only users with topic management privileges can see it.
    • pjftP
      pjft @meleu
      last edited by

      @meleu Sounds good.

      Any further thoughts on the stretch defaults and subtitles, just so this is closed off?

      pjftP meleuM 2 Replies Last reply Reply Quote 0
      • pjftP
        pjft @pjft
        last edited by

        So, I've added the option to show the game info "Always", "Never" and "Start & End". The default is still "Never", as it can cause problems for some Pi resolutions, unless the user disables overscan in config.txt.

        I've left stretch as false by default. It will still cover the full screen, just with black bars, but keep the game's original proportions.

        I edited the option and help text as well, and added the ALSA condition.

        I need to confirm that this compiles and that all is good (blind coding, yay), but if all is good then that's the last of it.

        Thanks all for the inputs and testing! I'll update when I've confirmed it compiles.

        1 Reply Last reply Reply Quote 2
        • meleuM
          meleu @pjft
          last edited by

          @pjft

          Any further thoughts on the stretch defaults and subtitles, just so this is closed off?

          The vertical screen games example (e.g. DonPachi) convinced me, I'll let it disabled (maybe it should be always disabled and the option removed?).

          Regarding subtitles the options you provided are good. I'll choose the "ALWAYS" option.

          • Useful topics
          • joystick-selection tool
          • rpie-art tool
          • achievements I made
          pjftP 1 Reply Last reply Reply Quote 1
          • pjftP
            pjft @meleu
            last edited by

            @meleu Great.

            Yeah, I usually leave it NOT stretched, but some people - maybe with 4:3 monitors and such - asked for the stretch option, and I can see how it does look good if you do have the right resolution. It's just the other ones that are an eyesore, but it's down to each person's preference :)

            I'll update the PR later today, and then if a final round of testing can take place, that'd be perfect, though hopefully things should be good.

            Also, have you tried it on your Pi 1/0? OMX Player should make these pretty smooth, though I imagine you don't want to go through compiling these test builds for 4 hours or so before they're final. :)

            1 Reply Last reply Reply Quote 1
            • HexH
              Hex
              last edited by

              @pjft I would like to reconfirm that you are testing that the device starts with ALSA all caps and not equals.

              Sent from 20,000 leagues under the sea.

              Powersaver Emulation station : https://github.com/hex007/EmulationStation
              ES dev script : https://github.com/hex007/es-dev/blob/master/es-tests.sh

              pjftP 1 Reply Last reply Reply Quote 0
              • HexH
                Hex
                last edited by

                @pjft what happens to the subtitles if overscan is enabled? Are they not visible or just no video?

                Sent from 20,000 leagues under the sea.

                Powersaver Emulation station : https://github.com/hex007/EmulationStation
                ES dev script : https://github.com/hex007/es-dev/blob/master/es-tests.sh

                pjftP 1 Reply Last reply Reply Quote 0
                • pjftP
                  pjft @Hex
                  last edited by

                  @Hex correct, starts with, but I believe the option in the code is lower case? I'll re-check what's being used but I thought I saw it as lower case.

                  1 Reply Last reply Reply Quote 0
                  • pjftP
                    pjft @Hex
                    last edited by

                    @Hex it'll depend. In some cases it works perfect - my old TV is one such case. That being said, the fact that it requires an extra layer from OMX Player may cause the HDMI signal to drop - I believe you experienced it earlier?

                    Removing the subtitles or disabling overscan (which is, for the GPU pipeline's purpose, just an additional forced black layer to allow for a forced margin) will allow the subtitles to not cause the HDMI signal to drop.

                    It's only for OMX Player though.

                    1 Reply Last reply Reply Quote 0
                    • HexH
                      Hex
                      last edited by

                      I didnt know OMX had subtitles.

                      Sent from 20,000 leagues under the sea.

                      Powersaver Emulation station : https://github.com/hex007/EmulationStation
                      ES dev script : https://github.com/hex007/es-dev/blob/master/es-tests.sh

                      pjftP 1 Reply Last reply Reply Quote 0
                      • pjftP
                        pjft @Hex
                        last edited by

                        @Hex same as with VLC, if you choose to display the game info.

                        Actually, did you manage to get the screensaver working on your pi? Your signal was dropping if I recall correctly?

                        I'd like to know if you're still experiencing that. I'd hate for that to be the case, but I could consider allowing VLC to be used for the screensaver on the Pi, if there are cases where OMX Player just load signal and there's no workaround, though that's really a recipe for overheating...

                        1 Reply Last reply Reply Quote 0
                        • pjftP
                          pjft
                          last edited by

                          It compiles. Updated the PR.

                          Refactored the options - merged some, moved audio-related ones to the Sound menu, and added an explicit option on the Pi to use VLC as screensaver should people choose to (default: OFF).

                          TMNTturtlguyT 1 Reply Last reply Reply Quote 0
                          • TMNTturtlguyT
                            TMNTturtlguy @pjft
                            last edited by

                            @pjft I updated your build and I am not sure that I am getting the most up to date version? I made sure to do a clean install. I am not seeing any of the revised menu options on the pi 3? I believe above you mentioned a VLC option for the PI, as well as moving audio options under sound, and 3 different subtitle options. I am currently running on Pi3 4:3 monitor. Thanks

                            pjftP 1 Reply Last reply Reply Quote 0
                            • pjftP
                              pjft @TMNTturtlguy
                              last edited by

                              @TMNTturtlguy hm. It should but I'll check as I'm putting the boys to bed at the moment. What version does it claim to be? It should be 2.2.1 I think. If not, maybe it's the wrong branch, or maybe I messed something up - certainly possible.

                              TMNTturtlguyT 1 Reply Last reply Reply Quote 0
                              • TMNTturtlguyT
                                TMNTturtlguy @pjft
                                last edited by

                                @pjft It's 2.2.0RP - I am using the same branch that the PR was created with:
                                RetroPie-ScreenSaver-Merge

                                I have it saved in @meleu script. The first time i ran it, I had it ignore make clean to speed it up, I didn't see the updates, so I reran it and installed it clean, same results.

                                Thanks

                                pjftP 2 Replies Last reply Reply Quote 0
                                • pjftP
                                  pjft @TMNTturtlguy
                                  last edited by

                                  @TMNTturtlguy sounds good, I'll check. I must have messed something up then.

                                  1 Reply Last reply Reply Quote 0
                                  • pjftP
                                    pjft @TMNTturtlguy
                                    last edited by

                                    @TMNTturtlguy hm.

                                    The code seems to be there though:

                                    https://github.com/pjft/EmulationStation/blob/RetroPie-ScreenSaver-Merge/es-app/src/EmulationStation.h

                                    It's 2.2.1 that you should be seeing.

                                    Could you open that file locally on the Pi and check whether the content is the same?

                                    TMNTturtlguyT 1 Reply Last reply Reply Quote 0
                                    • TMNTturtlguyT
                                      TMNTturtlguy @pjft
                                      last edited by

                                      @pjft dumb question - but i am not sure where to find that! Your branch is currently in /opt/retropie/supplementary/es_pjft_retropie-screensaver-merge/ on the pi. I don't have a file named EmulationStation.h there.

                                      1 Reply Last reply Reply Quote 0
                                      • pjftP
                                        pjft
                                        last edited by

                                        Not a dumb question at all. I suppose @meleu would know where the code is compiled to. That's where you'd want to look into.

                                        I've just done a final push to the PR, updating a few things that had gone missing. All seems to be good to go on my end, but more testing is always encouraged and appreciated.

                                        @Hex : the Alsa test works fine, and actually when you change it and exit the options menu the video player is restarted immediately with the new audio option, so no worries about that. Do test it, but it seems to work well.

                                        I do get an error on my end - I believe when set to alsa:hw:1,0 for "ALSA lib pcm_hw.c:1667:(_snd_pcm_hw_open) Invalid value for card".

                                        Unsure if it's because it's not a valid option or if it's because I don't have an external card, but thought you'd like to know.

                                        @TMNTturtlguy one thing you may want to do is to just do it old school style.

                                        https://retropie.org.uk/forum/post/82820

                                        I didn't test those instructions but they should work.

                                        TMNTturtlguyT HexH 2 Replies Last reply Reply Quote 0
                                        • TMNTturtlguyT
                                          TMNTturtlguy @pjft
                                          last edited by TMNTturtlguy

                                          @pjft said in Help testing: Random Video Screensaver on Main Branch:

                                          @TMNTturtlguy one thing you may want to do is to just do it old school style.
                                          https://retropie.org.uk/forum/post/82820

                                          Yeah, i have to run for a few hours, but i am going to install the main branch es and then install yours, if that doesn't work, i will install it the old way! Thanks

                                          Edit, i am going to try the remove/remove source files from @meleu script and then try to reinstall.

                                          meleuM 1 Reply Last reply Reply Quote 1
                                          • meleuM
                                            meleu @TMNTturtlguy
                                            last edited by meleu

                                            @TMNTturtlguy the script saves sources at ~/src.

                                            Edit: typo

                                            • Useful topics
                                            • joystick-selection tool
                                            • rpie-art tool
                                            • achievements I made
                                            1 Reply Last reply Reply Quote 0
                                            • First post
                                              Last post

                                            Contributions to the project are always appreciated, so if you would like to support us with a donation you can do so here.

                                            Hosting provided by Mythic-Beasts. See the Hosting Information page for more information.