ES Gamelist Scroll Bug - Black boxes appear when rapidly scrolling through the game list

pjft

@TMNTturtlguy thanks for the time. If it's too much effort, don't worry, I'll test it out in the coming days. Thanks for looking into it!

TMNTturtlguy

@pjft No problem testing on my end. The update from source stalled out for 5 hours at 81%, but the good news is the card booted back up, so i am attempting it once again. 5 minutes in and we are at 23% and running. I am glad to do all the testing you need as you are more than overly helpful for with all my problems, issues, and requests! Keep the testing requests coming!

TMNTturtlguy

@jdrassa do you have any idea as to why my installation of emulationstation from source is stalling out. Yesterday I attempted an install and it stalled out at

[ 81% ] Building CXX object es-app/CMakeFiles/emulationstation.dir/src/GameList.cpp.o

Just now i have tried again to update from source and I am stalled at

[ 80% ] Building CXX object es-app/CMakeFiles/emulationstation.dir/src/SystemDatacpp.o

I am going to try to update my entire build next and then try to update ES from source after that. Any thoughts would be great.

Thank you

pjft

@TMNTturtlguy I had a similar experience recently, which I assumed was related to lack of memory/swap perhaps.

When I killed ES (F4) and ran it form the command line or SSH, via

cd ~/RetroPie-Setup
sudo ./retropie_setup.sh

(I think those are the right folder and script) it worked well.

Don't unplug the Pi, but rather SSH into it and do sudo reboot now to restart it.

TMNTturtlguy

@pjft Thanks! I am currently running the entire system update just to do it. It is my test card anyways so i am not concerned. Good to hear, i guess, that you had this issue as well. I will try your method if the install doesn't work.

As for unplugging, yes 100% never unplug, except when it has stalled out on me. For some reason, both times when it stalled I was unable to SSH in. At first i thought the stall was network related, but I am running the internet hardwire off the same connection as my computer, network is fine. It just wouldn't let me get into the pi to reboot. P.S. network connection is fine because as soon as i reboot it is connected and the entire update is running very smoothly so far.

Side note, how have you been lately? Hope you are doing well and your projects are coming along nicely.

pjft

@TMNTturtlguy Thank you very much for asking - I have been good. A bit busy at work, as well as with the kids at home - they're at an age where they're not yet particularly self sufficient (1 and 3) so I haven't had a lot of free time to work on side projects.

I was trying to push for a way to allow people to install and update the OMX ES version from RetroPie-Setup's experimental menu, but will probably be pursuing actually integrating it in the main ES, though that will certainly take a longer time. But at least it gives us some good outlook for the future!

Anyway, if you do want to download this emulationstation-omx.sh file and copy it to ~/RetroPie-Setup/scriptmodules/supplementary, it should show up in your experimental packages as an option.

Be aware that it installs to a different folder by default (/opt/retropie/supplementary/emulationstation-omx) rather than replacing your current installation. It shouldn't affect you in any way, as it's transparent for the user, but in case you decide to replace binaries in the future, you should first re-install the main ES for the shortcuts to all go back to "normal".

How are you doing sir? Hope you're doing well as well!

Cheers!

TMNTturtlguy

@pjft i think we live identical lives on the opposite side of the pond! I also have 2 daughters, 3 and 7 months. The 3 year old is having potty training issues (don't want to many details here but she doesn't like to do her "Business" on the potty) so she has been very constipated and making life interesting. I also have a huge work deadline and I have been trying to keep my pi on next to me and run tests and different trouble shooting things as breaks between the massive report i am working on.

I was trying to push for a way to allow people to install and update the OMX ES version from RetroPie-Setup's experimental menu, but will probably be pursuing actually integrating it in the main ES, though that will certainly take a longer time. But at least it gives us some good outlook for the future!

I think either is great, but the way you currently have your build set up where you can turn OMX on and off from the menu is perfect. Allows super easy options for everyone. If you maintain this, i think building it into the main ES would be perfect. Those who want overlays can turn it off, those who want the OMX performance can turn it on.

Anyway, if you do want to download this emulationstation-omx.sh file and copy it to ~/RetroPie-Setup/scriptmodules/supplementary, it should show up in your experimental packages as an option.

Be aware that it installs to a different folder by default (/opt/retropie/supplementary/emulationstation-omx) rather than replacing your current installation. It shouldn't affect you in any way, as it's transparent for the user, but in case you decide to replace binaries in the future, you should first re-install the main ES for the shortcuts to all go back to "normal".

I will give this a shot on my experimental card fist and see how it implements. Once i do this, copy it to the supplementary folder and install it from the experimental options, is there an easy option to switch back and forth between the main ES and the -OMX? Would that just be going back into the experimental packages and telling it to uninstall -OMX? Lastly, would this folder be removed the next time I would update retropie? For example if retropie 4.3 is released?

Thanks

TMNTturtlguy

This post is deleted!

TMNTturtlguy

@pjft

@TMNTturtlguy could you check if in the latest, updated from source, main ES the lower ribbon in system view is showing up (the background of the "xxx games available" text)?

It's not showing for me in your theme at the moment, after I merged the latest ES changes, so wanted to check if it's my merged version that's broken or the main one.

Just checked on the updated ES V2.1.8RP and i experience the same results as you do, no bar behind the games available. There is something new in the update between 2.1.6 and 2.1.8 which changes the the way ES works and breaks my current code.

I updated my comic_book.xml with the code i posted earlier and it corrected the issue:

  <text name="systemInfo">
	<backgroundColor>eeeeee99</backgroundColor>
	<color>000000</color>
	<fontPath>./art/Shaka_Pow.ttf</fontPath>
	<fontSize>0.035</fontSize>
  </text>

I will try to update the code over the weekend and add the <feature> coding so that it will function on both the udpated ES as well as non carousel ES. Thanks for the heads up. As ES evolves, it appears that sometimes our old theme codes are breaking!

TMNTturtlguy

@pjft said in ES Gamelist Scroll Bug - Black boxes appear when rapidly scrolling through the game list:

Anyway, if you do want to download this emulationstation-omx.sh file and copy it to ~/RetroPie-Setup/scriptmodules/supplementary, it should show up in your experimental packages as an option.

Can i bother you to resend the link to the .sh file? When i click on the link you posted it opens the code in a web browser and does not download the file. Alternatively, I suppose i could copy the code into a .xml and save as a .sh? Thank you!

pjft

@TMNTturtlguy Yeah, we're going through potty training as well right now, though it's going alright-ish. :) What a coincidence.

I tried taking the Pi to work and having it there, though I don't have a lot of time during the day as I wanted to. I might take it with me if I want to compile something, or back it up as those take more time, but are fairly hands off activities.

Definitely if it makes it to the main ES it will be as an option, just like it's in the current OMX build. I would not dare to force it on people, given its limitations, but it also has quite a lot of benefits, especially in regards to performance on the Pi.

In regards to switching easily back and forth, there are two.

One is to re-install the main ES from RetroPie-Setup.

The other is to edit the file at /usr/bin/emulationstation(for instance, via sudo nano /usr/bin/emulationstation) and you'll see one mention to the /opt/retropie/supplementary/emulationstation-omxfolder. Just remove the "-omx" part, and it should point to your original ES folder. Add "-omx" back to point to the new folder.

Might be the easiest.

If it's only for one-off tests, though, you can always just straight up run the right executable file in the respective folder. :)

pjft

@TMNTturtlguy Sure. Either way works, but you can use this link and right-click "Raw" and save as!

TMNTturtlguy

@pjft got it! With this installed in the experimental packages, will your updates to the project be able to be updated from the setup menu/experimental packages?

pjft

@jdrassa it looks like it's traced to this. Is this an intended consequence of the change in here, or is it an omission?

Will this change cause unintended issues in other themes? Carbon seemed to work well, so I wonder what causes it here, or whether it was the theme's omission.

@TMNTturtlguy correct. I take it you were able to compile things in the end, then?

TMNTturtlguy

@pjft haven't had a chance to compile, had to shut the pi down and work! I have a conference call coming up!

Will this change cause unintended issues in other themes? Carbon seemed to work well, so I wonder what causes it here, or whether it was the theme's omission.

I think it is a little of both, might have actually been a bug in the old ES code. I found with testing through @modmaster who was running an old ES that if i intentionally omitted the <backgroundcolor> from the theme code, the ES v2.1.6RP was able to see the correct comic text style and older versions were able to still see the text, but in the standard font type. Basically older versions that v2.1.6RP just ignored the code lines. If however, i had the <backgroundcolor> in the theme, it broke the theme for old versions. I was having trouble with the <feature> tag, so this was my fix.

This explains why it is not an issue with carbon, carbon does not try to modify the text from the standard out of the box ES text for games available.

jdrassa

@pjft It is not an intended consequence. I believe the cause is actually this line. Here I am resetting the render flag to false, because I was concerned about what would happen when switching themes. The problem is that the backroundColor theme element can be used on any text element, not just systemInfo. If a user switches from a theme that uses a background color to one that does not, without this, the backgroundColor set by the previous theme would still be rendered.

pjft

@jdrassa got it. I'm not quite familiar with that code, but it's good that at least you narrowed it down to what the cause might be.

I don't know enough about the themes landscape - are there many themes without background color set, or is it pretty much standard?

If it's quite standard and expected, then I won't worry about changing it. If on the other hand it's a more recent change, I'd be concerned about the compatibility of the older themes, so I might suggest we think about an alternative there. Happy to take a look at the code if it helps - though, admittedly, given that nobody really noticed it yet (whether because of lack of updating, lack of noticing or actually using themes that specify the color) I wouldn't suggest this is in any way a high priority.

Thanks, and have a great weekend!

jdrassa

@pjft I ability to theme the background color of a TextComponent is a new feature that was added as part of the carousel changes. Previously, themes did not have the ability to change the system info text. I believe this issue only presents itself for a theme that is making theme changes to the systemInfo element, but doesn't set a background color. As such it should not effect any older existing themes that have not been updated.

TMNTturtlguy

@jdrassa @pjft Correct, i did a poor job of trying to explain that above.

Basically older versions that v2.1.6RP just ignored the code lines. If however, i had the <backgroundcolor> in the theme, it broke the theme for old versions. I was having trouble with the <feature> tag, so this was my fix.
This explains why it is not an issue with carbon, carbon does not try to modify the text from the standard out of the box ES text for games available.

TMNTturtlguy

@pjft said in ES Gamelist Scroll Bug - Black boxes appear when rapidly scrolling through the game list:

@jdrassa it looks like it's traced to this. Is this an intended consequence of the change in here, or is it an omission?

Will this change cause unintended issues in other themes? Carbon seemed to work well, so I wonder what causes it here, or whether it was the theme's omission.

@TMNTturtlguy correct. I take it you were able to compile things in the end, then?

I was able to add it to my experimental packages and run it from there, however once again i got to 81% building CXX objetct es-app/CMakeFiles/emulationstation.dir/src/gamelist.cpp.o and it froze up. This time after about 10 minutes i go the following error.

[1094.865873] Write-error on swap-device (179:0 :3925616)

I wonder what the deal is. I was able to update the entire retropie update all packages. That must be from binary? It appears whenever i try to update from source i have an issue.

I ended up running the setup.sh from putty as you recommended and it ran just fine. Is the surprise me on the options menu new? i don't remember that being there last time!

Another thing that i have been meaning to ask you, this is low priority for sure and may not be possible, but when i am in screensaver mode with my keyboard plugged in, i can press left or right on the arrows and skip videos. When i am using my controller (ps4 wireless with USB dongle) pressing the left or right dpad exits me from the screensaver. 1. is there something on my end i can do to my end with my controller setup to allow the dpad to scroll through screensaver videos? 2. is there code that you can change to make the dpad = the computer arrows?

Thanks!