[SCRIPT] RetroPie Limit Last Played Games

Clyde

@hiulit Great work! (without having tested it)

MadHorse

@hiulit, first, thank you so much for your work, your work is really appreciated.
Second, yout script is awesome, it's working perfectly.
Installed the script on the Retropie menu, launched the script, choosed to keep 1 last played game, at the end of the execution you need to restart Emulationstation to see the results.

Now my last played list contain only 40 games, one per system :)

Thank you again, you did a great job.

cyperghost

@hiulit I don't want to do a PR but you can add these lines to perform a restart. You can write a small yes/no dialog to perform a restart to ES.

# Restart ES!
touch /tmp/es-restart
chown pi:pi /tmp/es-restart
kill $(pidof emulationstation)

Best wishes ;)

hiulit

@madhorse Hey! You're welcome! I'm glad you find it useful :D

And you made a good point about EmulationStation having to be restarted to see the changes. I'll add a message to the script .

hiulit

@cyperghost hey, I was just thinking about adding a message but I like your idea better. Thanks for the code ;)

hiulit

@cyperghost btw, can this code be run under 'sudo'? Because my script has an option be installed in the RetroPie menu and I think the scripts executed from there are run under 'sudo'.

cyperghost

@hiulit Yes but then the file has to be chowend to pi:pi
Added to the codebox

hiulit

@cyperghost the file being 'tmp/es-restart' or my script?

cyperghost

@hiulit The restart file
Because ES entitiy deletes the file on restart and if it was created by root it won't work

hiulit

@cyperghost got it. Thanks! I'll try it ;)

MadHorse

@hiulit I've found a little "bug", if there are more games that have the exactly same lastplayed it will give you this errore:

Number of 'last played' games to limit is set to '1'.

psx
---
'gamelist.xml' for 'psx' found!
> Creating 'gamelist-backups.xml' for 'psx' ...
'gamelist-backups.xml' for 'psx' created successfully!
./retropie-limit-last-played-games.sh: line 124: [[: 0
1: syntax error in expression (error token is "1")
./retropie-limit-last-played-games.sh: line 124: [[: 0
1: syntax error in expression (error token is "1")
No 'last played' games to remove.

All done!

After i change the time, i just add 1 second, everything is working without a problem.
I think it's almost impossible to have more games with the same lastplayed date and time unless someone manually edit the file and make some mistakes, so probably is not a big deal.

hiulit

@madhorse Hmmm... I'll take a look at it. But what are the odds of having the same date on different lastplayed tags? How is it even possible to open 2 different games at the same time? :P hehe! Anyway, I'll look into it.

MadHorse

@hiulit said in [SCRIPT] RetroPie Limit Last Played Games:

@madhorse Hmmm... I'll take a look at it. But what are the odds of having the same date on different lastplayed tags? How is it even possible to open 2 different games at the same time? :P hehe! Anyway, I'll look into it.

Exactly, in fact i manually edited the gamelist.xml to add some games because i didn't want to scrape it, copy and paste the <game> section and then changed the name and the path, by doing this i also duplicated the lastplayed date and time.
I just pointed out because if someone needs help for the error

./retropie-limit-last-played-games.sh: line 124: [[: 0
1: syntax error in expression (error token is "1")

you know why, but i don't think you need to spend much time over this problem.

hiulit

@madhorse I figured it out. I'll test it a bit and I'll upload a new version. It would be cool if you could help me test once this fix is uploaded. I'll let you know. Thanks!

P.S. This will be on Wednesday. Tomorrow is a holiday for me :P

MadHorse

@hiulit No problem mate, I will be happy to test the new version.
Thank you and happy holiday :)

hiulit

Version 2.1.0 of RetroPie Limit Last Played Games is here!

Added

Dialog asking if the user wants to restart EmulationStation when the script is done.
Output message for the games that will be shown in the 'last played' section.

Fixed

When there are duplicated <lastplayed> tags in different games the script crashes. Now, if the scripts find duplicates, it looks for the game's name (it should be different).

Thanks to @cyperghost for the restart function and @MadHorse for testing the script.

hiulit

Updated the documentation for v2.1.0 - 2018-09-12 in the first post.

MadHorse

Hi @hiulit,
I'm so sorry, but checking the name introduced two more trouble.
If the name contain a ' single quote, it will complain because it Failed to compile select expression

Number of 'last played' games to limit is set to '10'.

daphne
------
'gamelist.xml' for 'daphne' found!
> Creating 'gamelist-backups.xml' for 'daphne' ...
'gamelist-backups.xml' for 'daphne' created successfully!
Invalid predicate: /gameList/game[name='Dragon's Lair II: Time Warp (US v3.19)']/playcount
compilation error: element with-param
XSLT-with-param: Failed to compile select expression '/gameList/game[name='Dragon's Lair II: Time Warp (US v3.19)']/playcount'
Invalid predicate: /gameList/game[name='Dragon's Lair']/playcount
compilation error: element with-param
XSLT-with-param: Failed to compile select expression '/gameList/game[name='Dragon's Lair']/playcount'
> Removing the 'last played' games surplus for 'daphne' ...
ERROR: There aren't enough 'last played' games to remove.
Try lowering the '--nth' number.
Now it's set to '10' and there are only 4 games in 'daphne'.

Daphne gamelist.xml contains 6 played games, but 2 games are not counted beacuse of the XSLT-with-param: Failed to compile select expression.

The other error is, if there are more games with the same name, it will give you another error

Number of 'last played' games to limit is set to '10'.

amiga
-----
'gamelist.xml' for 'amiga' found!
> Creating 'gamelist-backups.xml' for 'amiga' ...
'gamelist-backups.xml' for 'amiga' created successfully!
./retropie-limit-last-played-games.sh: line 125: [[: 1
1: syntax error in expression (error token is "1")
./retropie-limit-last-played-games.sh: line 125: [[: 1
1: syntax error in expression (error token is "1")
> Removing the 'last played' games surplus for 'amiga' ...
ERROR: There aren't enough 'last played' games to remove.
Try lowering the '--nth' number.
Now it's set to '10' and there is only 1 game in 'amiga'.

and instead of the same lastplayed date and time, finding more games witht he same name is easier, you may have multiple version of the same game, for example you may have the same Amiga games on CD32,CDTV, ADF or for example in many languages, but when you scrape the games, you will have the same name for every version of the game unless you don't manually change it.
One las note, the 2 amiga games with the same name, didn't have the same lastplayed date

	<game>
		<path>./Turrican2_v1.4_CDTV.lha</path>
		<name>Turrican II : The Final Fight</name>
		<desc>Turrican II: The Final Fight is a platformer/shooter as typically seen on consoles, but designed for home computers from the ground up.

Its plot is - naturally - very simple: an evil mega-robot called The Machine attacks the United Planets Ship Avalon 1, slaughtering all who resist. All - except Bren McGuire, who manages to escape and slips into the experimental Turrican bionic armor. As Turrican, he strives to show The Machine who's the boss.

Unlike other games of its type, this game contains three levels of horizontal shooter action in the spirit of R-Type or the developers' own Katakis. Six large worlds are to be explored, where you are basically free to go everywhere you want, since there is no automatic scrolling and the levels are packed with hidden extra lives and weapons. All those levels are very different: the first one in the rock desert is pretty colorful with parallaxing rainbow background and happy music, while the following worlds get darker and darker.</desc>
		<image>./media/images/Turrican 2 (CDTV).png</image>
		<rating>0.95</rating>
		<releasedate>19910101T000000</releasedate>
		<developer>Factor 5</developer>
		<publisher>Rainbow Arts Software</publisher>
		<genre>Platform</genre>
		<playcount>1</playcount>
		<lastplayed>20180719T150228</lastplayed>
	</game>
	<game>
		<path>./Turrican2_v1.7_0029.lha</path>
		<name>Turrican II : The Final Fight</name>
		<desc>Turrican II: The Final Fight is a platformer/shooter as typically seen on consoles, but designed for home computers from the ground up.

Its plot is - naturally - very simple: an evil mega-robot called The Machine attacks the United Planets Ship Avalon 1, slaughtering all who resist. All - except Bren McGuire, who manages to escape and slips into the experimental Turrican bionic armor. As Turrican, he strives to show The Machine who's the boss.

Unlike other games of its type, this game contains three levels of horizontal shooter action in the spirit of R-Type or the developers' own Katakis. Six large worlds are to be explored, where you are basically free to go everywhere you want, since there is no automatic scrolling and the levels are packed with hidden extra lives and weapons. All those levels are very different: the first one in the rock desert is pretty colorful with parallaxing rainbow background and happy music, while the following worlds get darker and darker.</desc>
		<image>./media/images/Turrican 2 (CDTV).png</image>
		<rating>0.95</rating>
		<releasedate>19910101T000000</releasedate>
		<developer>Factor 5</developer>
		<publisher>Rainbow Arts Software</publisher>
		<genre>Platform</genre>
		<playcount>1</playcount>
		<lastplayed>20180720T005157</lastplayed>
	</game>

I beg your pardon, if it wasn't for me, you made a perfectly working script unless someone don't mess with the gamelist.sml file.
I'm so sorry, may I help you in any way ?

hiulit

@madhorse whooops, it's getting tricky! Haha! Would you mind copying a couple of your gamelists into pastebin.com or something similar so I can test with real game lists? In the virtual machine where I develop the scripts I only have a couple of games to test.

The first issue involving the characters, I think I can escape them so it doesn't crash. The second one needs a little bit more of testing.

Thanks a lot for helping me test the script, btw!

hiulit

@madhorse your Daphne and Amiga game lists would be enough, I think.