Last Played Collection - Possible to limit?

mediamogul

Everything seems to be working well. From a user experience point of view, the list now definitely feels less cumbersome and more manageable. However, your first instinct to include 100 items may have been more appropriate, as 50 bottoms out a lot faster than I would have thought. The best number for this is always going to be subjective, but it'll be interesting to hear a consensus after more people have had a chance to try it. Fantastic work!

pjft

@mediamogul thanks for testing. I'd definitely be interested in hearing others' thoughts after a week or so of use, and then we can certainly adjust.

I have another fix for ES to keep me busy before lr-fuse, so it'll still be easy to change if needed.

Best.

cyperghost

@pjft Wow nice add-on
I vote for 100 items as default.

Clyde

I'm amazed what an idea plus some collaboration can accomplice. I love such communities like this one. :)

pjft

Hi all.

Just a note for those of you who moved to the development branch because of this. I seem to have run into an unrelated issue in that branch whereby the metadata of the games you play, if you're saving it on exit, may write incorrect paths for video and Marquee.

Check the games you've played (if any) and see if the metadata on reloading is still ok. I'll try to see if I can fix it (or others as well) soon, but you might be better off with the main branch right now, or at least not saving the metadata. I'd check the metadata for the games you played since updating.

Best.

Alturis

@pjft said in Last Played Collection - Possible to limit?:

Hi all.

Just a note for those of you who moved to the development branch because of this. I seem to have run into an unrelated issue in that branch whereby the metadata of the games you play, if you're saving it on exit, may write incorrect paths for video and Marquee.

Yeah even though I was the one that instigated the request for this feature I have decided to be patient and wait for it in the next stable drop. Experience has taught me never to grab latest until its been tested. ;)

pjft

@alturis it's not related to this particular change - though it also won't graduate to the stable branch if it isn't tested by people.

My comment was that there's this specific issue in the development branch for now, unrelated to this, that I ran into and thought I'd let others know.

Alturis

@pjft said in Last Played Collection - Possible to limit?:

it's not related to this particular change

Right. Sorry didn't mean to imply that. Just saying that who knows what other changes are in the latest along with it.

mediamogul

Ha, looks like this bit me in the backside on a few gamelist files. I suppose it's all part of walking the razor's edge of open source advancement. Actually, I've never felt more alive. I might even take up sky diving next, or perhaps record a ball game without the express written consent of Major League Baseball.

Anyone looking to check if any files were affected, look for any gamelist.cfg files that have a modification date past when you updated ES from source and look/search for any image tags starting with:

<image>./ownloaded_images/

and replace that part with

<image>/home/pi/.emulationstation/downloaded_images/

Unfortunately, I don't use videos on my setup, so I don't know exactly how they would have been affected.

pjft

@mediamogul interesting. Do you confirm that those are the exact before and after versions of one such path?

In my case the images stayed well. I was looking through the code and I think I suspect what might be the issue, but I don't know that code well enough to pass a judgement without effectively testing it.

Welcome to the bleeding edge of development:)

mediamogul

@pjft said in Last Played Collection - Possible to limit?:

Do you confirm that those are the exact before and after versions of one such path?

Yes. The image tag for every game that was played after the update was affected in that same way. I corrected them all with a find and replace.

pjft

@mediamogul thank you. Interesting.

What's your ROM path in the gamelist files for one of those ROMs, can you also share it?

mediamogul

@pjft

Sure. Here's 'Cosmo Gang: The Video' from my arcade system gamelist.xml.

<path>./AdvanceMAME/cosmogng.zip</path>

hiulit

Just wanted to let you all know that I've released the script I was talking about on this thread.

RetroPie Limit Last Played Games: A tool for RetroPie to limit the number of 'last played' games. https://retropie.org.uk/forum/topic/16608/script-retropie-limit-last-played-games

pjft

@mediamogul thank you. I take it you don't have one exact before and after path for the images in your specific scenario anymore as you fixed them, but I'll try to dig into it.

Thanks.

mediamogul

@pjft

Actually, the post here details one such before and after example. Outside of that, everything has been fixed, but the image paths in each affected gamelist.cfg file were all altered in exactly the same way.

<image>/home/pi/.emulationstation/downloaded_images/

Was altered in each occurrence to

<image>./ownloaded_images/

Clyde

@mediamogul said in Last Played Collection - Possible to limit?:

Was altered in each occurrence to
<image>./ownloaded_images/

I think you've a typo there (although „ownloaded“ could grow into a nice gamer slang word ;) ).

mediamogul

@clyde

Ha, it does look like it would be a typo, but that's just how it was truncated. However, the next time I hear a gamer tell me I was "ownloaded" over my headset, I'm gonna have to ask if his name is Clyde.

pjft

@clyde no, that's actually correct, it's a big that's changing the paths because it's shortening those strings. I was just trying to figure out if there was a pattern, but mine was different. :)

I'll need to test the code.

pjft

I just submitted a fix for the behavior and it has been accepted. You may now update from the dev branch and see if all works well.

Thanks.