moonlight: request for comments and testing

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.

WarC0zes

@hhromic

My runcommand.log :

Parameters: 
Executing: /opt/retropie/supplementary/moonlight/moonlight.sh stream -config "/home/pi/RetroPie/roms/steam/Steam.ml"
Can't open configuration file: /opt/retropie/configs/all/moonlight/global.conf

my global.conf :

# global config file for moonlight
quitappafter = true
width = 1920
height = 1080
fps = 60
bitrate = 20000

my global.conf file is in /opt/retropie/configs/all/moonlight/

hhromic

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

Can't open configuration file

That's very weird indeed. Can you please show the output of:

ls -la /opt/retropie/configs/all/moonlight