moonlight: request for comments and testing

hhromic

@hhromic said in moonlight: request for comments and testing:

Also, you may want to know that Moonlight gets properly terminated by Multi Switch Shutdown Script, but terminating it this way does not send quit message to the host. I'm not sure if it's a bug or expected behaviour, but I'll just patch up the script on my end to do that for me!

Umm this seems like a bug. If moonlight is terminated using SIGTERM it should gracefully finish (and send the quitapp request). I just tested and is not responding to the SIGTERM, I will debug and submit a PR upstream to fix this.

@Silent I found out the problem now. Indeed it was a bug in the code. I submitted already a PR upstream and once merged, Moonlight will gracefully terminate when receiving SIGTERM signals, for example from the switch shutdown script.
This means it will send the quit-app request on this situation too.

Thanks for reporting this!

Silent

@hhromic Nice updates, thank you! I'll make sure I test them soon, after I've restored the SD card to pre-sixaxis changes (due to new changes I don't want to uninstall, but rather start fresh) and then will give those a spin.

hhromic

@Silent the SIGTERM issue is now fixed upstream, although not from my PR as a better solution was proposed by the author :D.
Anyway, recompile moonlight and it should now gracefully exit from the shutdown script.

Silent

Good! Will try to test it tonight.

Also FYI I also posted an issue about this DS3 mapping on SDL bugtracker, but it's hard to tell when will they react to that.

Silent

@hhromic I just gave the newest scriptmodule a try and slugify option is a bit disappointing - while of course it works as intended and ThegamesDB handles these names relatively well, MobyGames seems terrible at it (so far I have 0 matches for games which names consist of multiple words). Will try IGDB next and report.

I think I'll stick to non-slugified names for now.

BuZz

@hhromic thanks for your work on this. I'm happy to accept this when you are. The old limelight module is so out of date I'm keen to remove it.

In regards to splitting out functions. Certainly may be useful, but I reckon accepting first and can refactor out later if useful for other modules.

Thanks again! Looks excellent.

llamadestroyer

Can be used with steam link? I think they are using same rom folder. Your script works awesome! I asked just for curious.

hhromic

@llamadestroyer said in moonlight: request for comments and testing:

Can be used with steam link? I think they are using same rom folder. Your script works awesome! I asked just for curious.

Yes it can be used at the same time. Steam Link is installed under "ports" (and doesn't use any "roms") while Moonlight is installed as an "emulator" for the "steam" platform (using ".ml" "roms"). These two are 100% independent.

Thanks for the feedback, glad you like the scriptmodule!

hhromic

@Silent said in moonlight: request for comments and testing:

@hhromic I just gave the newest scriptmodule a try and slugify option is a bit disappointing - while of course it works as intended and ThegamesDB handles these names relatively well, MobyGames seems terrible at it (so far I have 0 matches for games which names consist of multiple words). Will try IGDB next and report.

I think I'll stick to non-slugified names for now.

Oh that's a boomer to hear about the slugify solution. May I ask how exactly are you scraping these? I might give it a try myself to sort out what can be further done. Perhaps after slugifying changing "-" to " " might help, but I don't want to introduce more noise to the filename mangling if not necessary. Thanks for testing!

Silent

@hhromic said in moonlight: request for comments and testing:

May I ask how exactly are you scraping these?

Exclusively via Skyscraper. I'm yet to try the aliasMap for those - seems like it'd be a good solution, but I think changing - to would indeed help a lot.

jackalblood

Hi there Ive just installed moonlight for the first time on my retro pie and got everything setup fine right up to after I generated the configs all my games list and I restart retropie, but I'm not seeing any new menu items in emulation station and nothing but kodi in ports have I missed a step somewhere?

I've read the instructions a few times and I don't think I missed anything.

Any help would be greatly appreciated

Thank you

hhromic

@jackalblood thanks for testing the scriptmodule and confirming it's working fine for you (at least until game file generation).

The scriptmodule creates a "Steam" system where it places all the moonlight config files (with .ml extension), contrary to Kodi or Steam that only have a single entry in the "Ports" system. You should be able to find a "Steam" system in your emulationstation alongside others such as "SNES" or "NES".

Maybe you are using a theme that doesn't support this system? In that case you should still be able to find it I think. If you are not using the default theme, can you switch to the Carbon included theme just to check if it's your theme or something else?

jackalblood

@hhromic I was actually using carbon already.

However I did get the menu item to show with the scripts present by running this script after your embedded one.

[link text]https://github.com/TechWizTime/moonlight-retropie(link url)

This made the menu item appear and the scripts work perfectly although streaming performance sucked as I was away and using VPN.

All in all thanks so much for your work I've tried everything to get moonlight working in the past (hence my remembering the above script)

hhromic

@Silent I improved the filename mangling functionality now. The option is no longer called "slugify" alone but more general "file name mangling". There are three options now for it: NONE, SLUGIFY and WINDOWS. The latter is less agressive than "slugify" and only removes/replaces characters to make the names windows-compatible (SMB shares and FAT32 filesystems). Give it a try and see if it improves scraping for you. This should be the last change before submitting to official RP :)

@jackalblood said in moonlight: request for comments and testing:

However I did get the menu item to show with the scripts present by running this script after your embedded one.
[link text]https://github.com/TechWizTime/moonlight-retropie(link url)
This made the menu item appear and the scripts work perfectly although streaming performance sucked as I was away and using VPN.

Umm that's strange. Maybe you are using a custom es_systems.cfg that overrides the system one in /etc/emulationstation?

I would strongly advise you to re-install moonlight using this scriptmodule here because the TechWiz version uses an outdated pre-built Moonlight binary.

Thanks for testing and the positive feedback, much appreciated!

hhromic

Update

Recently, preliminary rumble support was added to Moonlight in upstream!
To give it a try, just re-build Moonlight from source using this scriptmodule :)

Silent

@hhromic Thanks! Now scraping works well as usual :D
I even managed to scrape streamed PS2 games from screenscraper by giving them sha1 hashes manually, good stuff :)

hhromic

@Silent I'm glad it's finally working good for you and probably others too!
I think now is time to graduate the scriptmodule to the Official RP repo :)
Any other further issue you would like to report?

Silent

Seems all good now! I was unable to get rumble working with my DualShock 3 yet, but naturally that's nothing a scriptmodule can do :D

As far as I can tell, it should be good to be merged! I am able to use it both for "steam" and "ps2" platforms just fine myself. Time to say goodbye to limelight.

WarC0zes

Hi,
I can not start steam in big picture mode, it comes back in emulationstation.

my Steam.ml file:

config = /opt/retropie/configs/all/moonlight/global.conf
address = 192.168.1.3

What's the problem?

hhromic

@WarC0zes thanks for testing.

To further debug, can you please give the contents of the /dev/shm/runcommand.log file just after it goes back to emulationstation? It might be a long file so please use pastebin or similar.