Scripting help for replacing retroarch.cfg

FlashPC

@tmntturtlguy I decided to use 4 options so I could use an overlay with an additional transparent filter which gives a shine or glare on the screen.

meleu

Can you guys post your scripts on ghostbin (or something similar) instead of meganz? I'm unable to see the script on my phone when it's on meganz.

cyperghost

@meleu https://ghostbin.com/paste/ytrxt

It's ... well functional but a huge list with cp commands.
I would prefer your approach as it is the easier way without copy file -->> much better.

@FlashPC
Use submenu and case selection to get it functional calls

Mean...
Main list is SEGA CD -> Select overlay, shader..... for SegaCD

#### THIS IS PSEUDOCODE!!!

modemenu ()
{
until [[ $shaderchoice == "<--" ]]
do
    shaderchoice=$(whiptail --backtitle " Choose " --title " Choose " --menu "Choose an option" 0 0 0 \
        "<--" "... back to mainmenu" \
        "standard" "Reset retroarch.cfg" \
        "shaders" "Use Shaders...." \
       .....
done

case "$shaderchoice" in
    standard) cp -f /opt/retropie/configs/all/retroswap/$1/standard/retroarch.cfg /opt/retropie/configs/$1/ ;;
    shaders) cp -f /opt/retropie/configs/all/retroswap/$1/shaders/retroarch.cfg /opt/retropie/configs/$1/ ;;
esac
done
}

#----------------------------

while true
do
    syschoice=$(whiptail --backtitle " Choose " --title " Choose " --menu "Choose an option" 0 0 0 \
        "nes" "Nintendo Entertainment System" \
        "md" "Sega Megadrive" \
       .....
done


case "$syschoice"
    *) modemenu "$syschoice" ;;
esac
done

This will shrink your code to minimum :)

meleu

@cyperghost said in Scripting help for replacing retroarch.cfg:

@meleu https://ghostbin.com/paste/ytrxt

Wow! Pretty huge. It can be simplified, but if it works, then fine. I could help, but it seems to be a very specific script to @FlashPC needs. I'm not sure about the details in his config...

FlashPC

Hey guys I've filled in the script that @cyperghost posted and I kind of understand what is trying to happen, but Iam stuck as to what I am missing, I don't know how to add the nice black boxes but I've entered it into ghost paste here

cyperghost

https://ghostbin.com/paste/t7ws7

Here is a working script!
As I said --- it's was pseudocode with some errors
2x done without do, break condition was set wrong, dialogboxes showed unneeded options....

If you run this script you will see just 3do standard for ex. as output.. Therefore
You must exchange the commands in line 49-52 with your copy commands.

So you stripped down from 30kB to 3kB and can faster navigate ;)

Greetings fly out to @meleu (<--- this guy is responsible for my rising interest in bash) and @TMNTturtlguy (<--- this guy is responsible that I get in need to use bash ... EUREKA!)

FlashPC

@cyperghost Thanks, its so tidy

cyperghost

@flashpc Does it work for you?
Can you post the full script with all alterings please?

You can even alter the titles and backtitles of each menu for ex. like this --backtitle " Your selected console is ${1^^} " So you have an overview of your systemchoice if you want to exchange the shader. Just a convenient addon...

... There is a small bug Please add a return between line 53 & 54. I update code.

What is the difference between overlay and overlay-real?

TMNTturtlguy

@cyperghost said in Scripting help for replacing retroarch.cfg:

Greetings fly out to @meleu (<--- this guy is responsible for my rising interest in bash) and @TMNTturtlguy (<--- this guy is responsible that I get in need to use bash ... EUREKA!)

Greetings back to you @cyperghost! Hope all is well. I just come up with crazy ideas, and pretty down and dirty ways to try to make them work, you and @meleu are the true artists that clean up the messes I start!

cyperghost

@TMNTturtlguy Yes I'm fine. Thank you - it's always a pleasure to do some coding stuff

meleu

Nice job guys!

@cyperghost I would like to recommend this reading when you have a time: Importance of code indentation. ;-)

FlashPC

@cyperghost said in Scripting help for replacing retroarch.cfg:

... There is a small bug Please add a return between line 53 & 54. I update code.

I found that out, does the additional return clear the $shaderchoice variable? I just added "shaderchoice=" at the start of the modemenu routine and that allowed me to select another mode after returning the main screen

@cyperghost said in Scripting help for replacing retroarch.cfg:

What is the difference between overlay and overlay-real?

overlay has side art and a bezel around the screen, overlay-real is the same with an additional semi transparent layer over the play area to mimic sun glare and smears on an old CRT TV

The current script is here

Thanks again

meleu

@tmntturtlguy said in Scripting help for replacing retroarch.cfg:

you and @meleu are the true artists that clean up the messes I start!

haha! I tried to be an artist. I even created a blog with some drawings I made: https://meleuartblog.wordpress.com/

Stopped updating that 2 years ago...

@FlashPC sorry for this off-topic post on your thread. :)

cyperghost

@meleu Hehe... I like artists ;)
Drawing is good for relaxing and maybe once a time you come back to design studies. Believe me I've married a talented artist - my wife do great drawings if there is muse. You can do also good scratches with Corel Painter and a small Bamboo Tableu.

Btw.: This one is epic ;)

@FlashPC

I found that out, does the additional return clear the $shaderchoice variable? I just added "shaderchoice=" at the start of the modemenu routine and that allowed me to select another mode after returning the main screen

Yes that also helps, maybe it helps to declare shaderchoice as local variable.
You did a great job ;) Even if the coding is a bit "uncommon" ;) You use tons of if-then clauses to set a variable. Please don't see this as an offence but now you have to maintain two locations of systemnames. It's better to use an array for this and to create the menu out of this, too. But as the script works for you it's okay ....

FlashPC

@cyperghost Thanks man, I'm an absolute noob when it comes to this type of coding, I know it's uncommon, but following the information and instructions from various sources I'm finding its a little bit disjointed and I've done some BASIC self training back in the 90s and recently Lua for creating (very simple) programs in windows.

I know of if / then clauses and so it meant I could a load a variable with text depending on the selection. But I want to take this further as a pet project and a learning exercise.

This is how I started with Lua (indigorose auto media studio), I started with a basic idea and then added to it, tidying the code, adding features, and streamlining by searching forums and checking YouTube.

Any learning directions would be definitely be appreciated, I will never be a master but if I could muddle through it would be great

Cheers guys
Paul

cyperghost

@flashpc Yes the coding style looks like BASIC... btw - you can install FreeBasic on the Pie and create binaries out of this. So if you want to share your programms the other users don't need an interpreter. So you can stick to any language you want... bash or BASIC.

But I did not know about whiptails --notags command. So your systemlist looks really tidy ;) If you don't like whiptail you can stick to dialog. Dialog is a more expanded box creater as whiptail with much more commands but I prefer whiptail because of it's simply usage.

FlashPC

@cyperghost said in Scripting help for replacing retroarch.cfg:

But I did not know about whiptails --notags command. So your systemlist looks really tidy ;)

Thanks dude, I found it because I was looking for a way to remove the OK button and read the info for the whiptail command and thought --notags would be interesting.

FlashPC

@cyperghost Hey mate, can you please have a look at this as and see if I am on the right track, I am having a go at arrays to tidy the code.

Click here

cyperghost

@FlashPC
I think it can be done much easier without in need for a settings variable.

I write some pseudo-code okay


# Your array
console=("3do" "Panasonic" "nes" "Nintendo Entertainment system"......)

# Now let whiptail work for you... as it expects two strings for creating text boxes, we give them...

# I try to use proper indentation for better code reading
# We start our loop for systemchoice here
while true
    do
        syschoice=$(whiptail -- back title " Systemmenu ...... \ 
            --nocancel --menu " Choose " \
            "<--" "... back to EmulationStation"
            "${console[@]}" \          # <--- This creates the full menu structure out of array!!
            .... )

            case "$syschoice" in
                "<--") break ;;
                *) shaderchoice $syschoice ;; # <--- Call to function as in v3 script
            esac
    done

~~Later you can get real systemname out of array with ${console[$1]} (I hope so) ;)~~

EDIT:

Add this to function modemenu

local idx=0
until [[ "${console[idx]}" == "$1" ][
    do
       idx=$(( $idx + 1 ))
    done

Then use whiptail and higher idx+1 and you've the console name... That's it and it was funny ;)

cyperghost

@FlashPC

Here is the full working script... Change as you like

#!/bin/bash

# Retroswap.sh
#############
# This script lets you switch retroarch config files.
# v4 09.09.2017 by cyperghost and FlashPC
#
# Place this script in /home/pi/RetroPie/retropiemenu/
# In order to run the script you must create the following folders:
#    1. /opt/retropie/configs/all/retroswap
#    2. within that folder create a folder for each system
#       2.1 create there folder "overlay"
#       2.2 create there folder "overlay-real"
#       2.3 create there folder "shader"
#       2.4 create there folder "standard"
#
# full example path for 3DO = /opt/retropie/configs/all/retroswap/3do/overlay
#
# Within each folder you have created place a retroarch.cfg file.
# The file needs to be named the same in each folder.
# Place your current retroarch_shader.cfg in the shader folder. 
# Name the file retroarch.cfg.
#
# Greetings fly out to meleu and TMNTturtleguy
#
# Please respect the work of others

# -- F U N C T I O N S --
modemenu ()
{

local idx=0
local shaderchoice
until [[ "${console[idx]}" == "$1" ]]
    do
       idx=$(( $idx + 1 ))
    done


until [[ "$shaderchoice" == "<--" ]]
    do
        shaderchoice=$(whiptail --backtitle " Console is ${console[idx+1]} " \
        --title " Choose " \
        --nocancel --menu "Choose an option" 0 0 0 \
            "<--" "... back to systemselection" \
            "overlay" "Use Basic Overlay & Scanlines" \
            "overlay-real" "Use Realistic Overlay & Scanlines" \
            "shaders" "Use Shaders...." \
            "standard" "Disable Overlays & Scanlines" \
            2>&1 > /dev/tty)


    case "$shaderchoice" in
        overlay) echo "$1 Overlay" && sleep 2 ;;
        overlay-real) echo "$1 RealOverlay" && sleep 2 ;;
        shaders) echo "$1 shaders" && sleep 2 ;;
        standard) echo "$1 standard" && sleep 2 ;;
    esac
done
}

#-- A R R A Y --
# Feel free to add and edit new console
# TAKE CARE OF UPPER AND LOWER CASE!

console=("3do" "Panasonic 3DO" "atari2600" "Atari 2600" "atari5200" "Atari 5200" "atarijaguar" "Atari Jaguar" "coleco" "ColecoVision" \
         "dreamcast" "Sega Dreamcast" "famicom" "Nintendo Famicom" "intellivision" "IntelliVision" "markiii" "Sega Mark III" \
         "mastersystem" "Sega Master System" "megadrive" "Sega MegaDrive" "segacd" "Sega MegaCD" "n64" "Nintendo 64" \
         "neogeo" "SNK Neo-Geo AES" "nes" "Nintendo Entertainment System" "nintendobsx" "Nintendo Satelliview" \
         "pcengine" "NEC PC-Engine" "pcenginecd" "NEC PC_EnigneCD" "psx" "Sony Playstation" "saturn" "Sega Saturn" \
         "sega32x" "Sega 32X" "sfc" "Nintendo Super Famicom" "snes" "Super Nintendo Entertainment System" "tg16" "TurboGrafx 16" \
         "tg16cd" "TurboGrafx 16CD")

# -- M A I N - L O O P --

while true
    do
        syschoice=$(whiptail --backtitle " Choose " --title " Choose " \
        --nocancel --menu "Select Console" 0 0 0 \
            "<--" "... back to ES" \
            "${console[@]}" \
            "<--" "... back to ES" \
            2>&1 > /dev/tty)

    case "$syschoice" in
        "<--") break ;;
        *) modemenu "$syschoice" ;;
    esac
done

@meleu ;) How about the indentations?