EmulationStation mod

jacobfk20

@meleu Thank you! It looks like I was correct. You went from a hair over 5 minutes to 9-10 seconds on exit now. It wasn't having much impact on my exit time (2 seconds to 1.4 seconds) but I'm glad to see it works on someone that has long exit times.

I did the math with the data given from the es_logs and the new method saves you 2,967,762 cycles in a while loop. That's insane!!

Is 10 seconds manageable? lol, I don't think I can get it any faster.

Anyway, I'm going to clean this branch up and may do a PR. Thanks again for doing the test. :)

lilbud

Where would be able to theme the carousel bar rank on your to do list?

jacobfk20

@lilbud I have a branch that I work on occasion that will allow that. So far all it does is just allow you to move the little bar beneath that shows game count and change its color. As for priority... I'm not sure. GridView has calmed down a bit and I just got OSK running good so it may get moved up.

meleu

@jacobfk20 said in EmulationStation mod:

Is 10 seconds manageable? lol, I don't think I can get it any faster.

10 seconds is great! :D

Could you put a message saying to the user that ES is updating the metadata or something similar? The first time I faced this problem I thought the RetroPie crashed ("Hey! It is not shutting down!"). A simple message would be enough to calm down the user. :-)

jacobfk20

@meleu I will consider adding a message on shutdown. I'd like for you to do one final test for right now if you wouldn't mind. I cleaned the branch up and changed how it works slightly and I just want to make sure it still gets you down to 10 seconds. It doesn't keep time this time but it keeps cycles.

You can get this test build in the usual place.

Please send a copy of the es_log when done :)

meleu

@jacobfk20 Done! It took 8 seconds to exit and the lastplayed/timesplayed were correctly updated.

I have noticed some lvl1 warnings about data type:

lvl1:    requested mismatched theme type for [detailed.md_lastplayed] - expected "datetime", got "text"

But the lastplayed info displayed in ES is correct.

You can see more details in the files I've sent.

jacobfk20

@meleu I get a mismatch warning from the official retropie build but it's to do with ratings. This could be from the theme you're using. Does it report that warning when you run the official retropie branch?

meleu

@jacobfk20
I checked more carefully now, with 3 different themes: Pixel, Simple and Carbon. The mismatch occurs only on Carbon. All the tests were made with the official RetroPie ES. So, it's not an issue with your branch. Phew~

BuZz

I caused the error with carbon - will fix.

quantum.hero

Looking forward to all the features being worked on here! Keep up the great work.

BuZz

@meleu please can you update the carbon theme and let me know if the error is gone now ?

meleu

@BuZz updated and tested the Carbon theme. The type mismatch error is gone.

Arcuza

How are things progressing with the grid view? I would love to have this game browsing mod installed.

lilbud

@Arcuza What is that specifically?

Also, I want folders to work in grid view

mediamogul

@lilbud said in EmulationStation mod:

Also, I want folders to work in grid view

I've been hoping that folder support would be possible as well.

trekdrop

Just so that I don't f*ck up. Do I need to update my manually? Or can I do it through the setup from retropie. I ask because I update though the setup menu and everything is weird... It's all white the text is everywhere. So I just want to know if this is normal or I need to update manually.

jacobfk20

@trekdrop To use this version of ES you need to install it manually.

@Arcuza Things are going fairly slow right now but I'm getting a good chunk of the weekend to work on it.

@lilbud, @mediamogul I haven't looked in to adding folders yet but It is on the todo.

mediamogul

@jacobfk20 said in EmulationStation mod:

@lilbud, @mediamogul I haven't looked in to adding folders yet but It is on the todo.

Very keen! Thanks for all your work here. It's very exciting and I look forward to trying it.

trekdrop

@jacobfk20

Do I need to copy the emulationstation.sh file from the download to the pi?

jacobfk20

@trekdrop There is no emulationstation.sh file in that link. It is just an emulationstation file. Unless you're downloading from the github page. But you just copy over the emulationstation file, make it executable with chmod +x emulationstation in terminal and then if you want to boot it instead of the normal ES you copy it to the /opt/retropie/supplementary/emulationstation directory. This will overwrite the ES currently in there so be sure to make a backup.