[Solved] Retroarch no longer respecting shader overrides

AndrewH

Here's the full /dev/shm/runcommand.log following that directory name change;

Parameters: 
Executing: /opt/retropie/emulators/retroarch/bin/retroarch -L /opt/retropie/libretrocores/lr-mame2003/mame2003_libretro.so --config /opt/retropie/configs/arcade/retroarch.cfg "/home/pi/RetroPie/roms/arcade/1943.zip" --verbose --appendconfig /dev/shm/retroarch.cfg'|'"/home/pi/RetroPie/roms/arcade/1943.zip.cfg"
[INFO] RetroArch 1.7.5 (Git c9c6c5a)
[INFO] Redirecting save file to "/home/pi/RetroPie/roms/arcade/1943.srm".
[INFO] Redirecting savestate to "/home/pi/RetroPie/roms/arcade/1943.state".
[INFO] === Build =======================================
Capabilities: NEON VFPv3 VFPv4 
Built: Oct  9 2018
[INFO] Version: 1.7.5
[INFO] Git: c9c6c5a
[INFO] =================================================
[INFO] Loading dynamic libretro core from: "/opt/retropie/libretrocores/lr-mame2003/mame2003_libretro.so"
[INFO] [overrides] no core-specific overrides found at /home/pi/.config/retroarch/config/mame2003/mame2003.cfg.
[INFO] [overrides] no content-dir-specific overrides found at /home/pi/.config/retroarch/config/mame2003/arcade.cfg.
[INFO] [overrides] game-specific overrides found at /home/pi/.config/retroarch/config/mame2003/1943.cfg.
[INFO] Config: appending config "/home/pi/.config/retroarch/config/mame2003/1943.cfg"

Here's the contents of /home/pi/.config/retroarch/config/mame2003/1943.cfg :

# Auto-generated zfast_crt_standard_vertical.glslp .cfg
# Game Title : 1943 , Width : 224, Height : 298, Aspect : 3:4
# Screen Width : 1280, Screen Height : 1024
# Place in /opt/retropie/configs/all/retroarch/config/MAME 2003/
video_shader_enable = "true"
video_shader = "/opt/retropie/configs/all/retroarch/shaders/zfast_crt_standard_vertical.glslp"
# To avoid horizontal rainbow artefacts, use integer scaling for the width
aspect_ratio_index = "22"
custom_viewport_width = "672"
custom_viewport_height = "1024"
custom_viewport_x = "304"
custom_viewport_y = "0"

The shader is being applied correctly now, but a screenshot verifies that the custom viewport width is not being used - the screenshot is 768 x 1024 instead of 672 x 1024

AndrewH

Taking a look at the commit history of lr-mame 2003 on github, it looks like the commit after the version I'm using fixes an issue introduced when the APPNAME macro was added (back in April)

Might be related, so I'm trying an "Update from Source" rather than "Update from Binary" to see what happens...

AndrewH

So... following the update from source, it's now necessary for that folder to be named MAME 2003 (0.78)
It still appears not to be respecting the custom viewport sizes.

pariziv

The aspect_ratio_index is changed. Custom is now "23" instead of 22.

AndrewH

@pariziv said in Retroarch no longer respecting shader overrides:

The aspect_ratio_index is changed. Custom is now "23" instead of 22.

Ah, okay. I’ll give that a try tomorrow and report back.

AndrewH

Okay, I've got it all fixed now. For anyone who finds themselves with the same problem, here's roughly what I did to fix it (just the relevant bits);

1 - Upgrade lr-mame2003 from source - this takes a while.

2 - locate /opt/retropie/configs/all/retroarch/config/crt_pi_configs.py and edit it. You can do this either by quitting Emulationstation to the console, SSHing in, or via the configs SMB share. Assuming you have the current version of the aforementioned script, these are the two changes that need to be made;
Line 21 needs to be changed from coreName = "MAME 2003" to coreName = "MAME 2003 (0.78)"
and line 164 needs to be changed from newCfgFile.write("aspect_ratio_index = \"22\"\n") to newCfgFile.write("aspect_ratio_index = \"23\"\n")

3 - With those edits made, run the script as follows; python3 crt_pi_configs.py mame2003 <screen width> <screen height> - this will generate new configs in a folder called MAME 2003 (0.78) within a subfolder named after the screen width x screen height, and you'll need to move this MAME 2003 (0.78) to the folder from which you ran the script.

That's pretty much it.

Thanks @mitu, @RedFarmer, and @pariziv for your inputs!

robertvb83

@AndrewH puh thank you very much Andrew. I am happy i found this thread. I understand what to do now. But i think many other People especially new to retropie will die trying to fix this.

But this is really a bad mess. Not only for this particular issue but many many references, docs, wikis etc. are telling people to use aspect_ratio_index ="22". How can they just change that like they dont care about anything…

I understand that additional aspect ratios are added from time to time, but this should be done in a non harmful way for example if they just use plain text instead of an index like custom_aspect_ratio_index="custom" would be suffficient… makes me want to scream :-(

Riverstorm

@robertvb83 - I think that's what @dankcushions was asking for in this open issue from last November before updating the script so isn't broken every time they make an amendment to the index list.

As it stands now I'm not sure how people are handling it. This thread or there's a good number of easy to use programs to search and replace the index and folder in existing configs.

robertvb83

@Riverstorm

As it stands now I'm not sure how people are handling it. This thread or there's a good number of easy to use programs to search and replace the index and folder in existing configs.

Thanks for the info about the open issue.

Thats exactly what i did. Search and replace everything at once with notepad++ took a couple of minutes

dankcushions

i encourage people to comment on the issue https://github.com/libretro/RetroArch/issues/7536 if they want to show their support :) if it's just me complaining then it's not much incentive for them to change how they operate.

Riverstorm

@dankcushions - It looks like were unable to add comments to the issue. It shows a padlock and the message "You can't perform this action at this time."