[SCRIPT] RetroPie Convert Videos

Clyde

@Menion said in [SCRIPT] RetroPie Convert Videos:

I am no expert of bash so there are few limitation, one is that the home directory is set in the script itself

Just curious, why didn't you use the original home detection or the config variant you still have as a comment in your script?

Menion

@Clyde because the modified script shall run on a host pc without requiring to have Retropie

Clyde

@Menion Perfectly understandable, but reading the rom path from the cfg file or as an command line argument wouldn't be Retropie-dependant either, and yet more consistant with the other options.

Just some friendly feedback. And thanks for mentioning me in the credits. 😊

Menion

The script requires improvement, but I am very bad in bash script

Clyde

@Menion I would be very much inclined to help you, but my bash skills are extremely limited, too. My video conversion script is an example of the maximum level of my bash knowledge, and even for that I had to do some web research. Maybe @hiulit can give you some clues.

Menion

No problem. Meanwhile I need to figure out why the bitrate set seems to produce odd results. Without specifying anything the encoding goes up to double the bitrate of the original file, while setting it the result is 40% less. I know that with single pass enconding the bitrate cannot be precise, but this is a little bit too much

hiulit

@Menion Hey, thanks!

A couple of questions:

Does your script work with a Raspberry Pi? Or does it only work on a "host" machine?
Does your script only work with VAAPI compatible GPUs? And if so, how could we detect it with the script?

About merging your script with mine, I would be more inclined in creating a new branch and having two separate scripts, the "default" one and the "host version" one.

Also, right now i don't have that much time to take a deep look at your script to check it but I'll try in some time soon(ish).

PS. I'm with @Clyde about using home="$(get_config "home")" instead of hardcoding it into the script ;)

Menion

The script should work with any vaapi host, but there is no detection of it, it is just hard coded

furnace

@hiulit
I've started to use your script and works like a charm. I've even modified it to support video scrapes attained by using Skyscraper on the retropie. I was just wondering though the following.

1- When the conversion is done, the converted videos are put into the converted directory, correct? can these then be copied back to the original directory to override the bad videos to reduce space usage?

2- Also if I use the --convert-system flag and let say I already converted a set of videos but then scrape new ones. Will re-running the conversion tool reconvert all the videos or just those that had not been done? If it's all the videos is there a why to just convert specific videos.

hiulit

@furnace Hey thanks! I'm glad you find it useful :D

It would be great if you can show me what modifications you've done, so maybe I can merge it with my version. Do you have a GitHub account to create a Pull request to my repository with you changes?

Now, I'm talking without looking at the code:

The script creates this converted folder so you can check if the videos are converted correctly or not. If not, they will a -failed suffix. The ones that are converted successfully are to be copied manually to the proper folder. I didn't code it in the script just in case anything goes wrong with the conversion.
I think it won't reconvert all the videos. Well, it depends, if you pass a from_color argument, the script will only search for the videos that have that particular "color encoding system". If you don't pass any from_color then yes, it will reconvert all the videos. I think.

In that latter case, I think I will try and add another option to the script to detect the "color encoding system" of the videos you want to encode.

furnace

@hiulit

Thanks for the feedback.

Sorry I don't have a github account. To modify the scripts though it was fairly simple. Steven Selph's Scraper will put the videos in a folder call video within the ROM folder. Skyscraper can be set to do the same, except the path will be $home/Retropie/roms/media/videos. So all I did was change the VIDEOS_DIR to the proper path.

Also your script looks for video file names that end in "-video.mp4", I replaced that to *.mp4 because the video file names are different between the two.

I had not set the from_color parameter before, so I'll test that out. Though I had tried to take a video that was not functional with the OMX enabled and pass it to AVprobe as per you script to see the encoding and it returned yuv420p. But once I convert them they are functional. I'm thinking the conversion might change something else which enables the functionality with OMX enabled. I'll keep investigating this and let you know.

hiulit

@furnace I can try to do something to "detect" the folders. And yeah, I think I can change the -video.mp4 for just .mp4. Thanks!

furnace

@hiulit
Or just as you force the customer to set the -to-color, you could just make the user to set a variable like
-source_scraper in the config to shelp or skyscraper and based on this set the video path. Cause I doubt someone would use various scrapers for different systems.

hiulit

@furnace Yeah, something like that could work. Thanks!

hiulit

@furnace Does this answer your question?

# From color (optional)
# Target only videos with this particular C.E.S.
# If left blank, it will target all videos, regardless of the C.E.S.
# (e.g. "yuv444p")

I haven't tried it yet, but looking at the code I found this.

hiulit

@furnace

~~This folder you mentioned $home/Retropie/roms/media/videos when using SkyScraper, does it contain folders for all the systems inside?~~

$home/Retropie/roms/media/videos/megadrive
$home/Retropie/roms/media/videos/nes

~~etc?~~

Forget that! I've found this table explaining where all the metadata and media are stored using different scrapers :)

https://retropie.org.uk/docs/Scraper/#where-are-my-scraped-media-and-metadata-saved

Is this information correct? (I'm guessing it is)

mitu

@hiulit said in [SCRIPT] RetroPie Convert Videos:

Is this information correct? (I'm guessing it is)

If you're using the scrapers through the RetroPie-Setup script, then it's correct.
Note that fi you're using the CLI parameters for folders or otherwise customize your configuration file, then the folders can be anywhere.

hiulit

@mitu Ok, thanks! I'll state that ROM folder for gamelists & media must be enabled. As well as I state the same thing when using Steven Selph's Scraper (use ROM folder for gamelists & images)

hiulit

@mitu Just out of curiosity, does the built-in EmulationStation scraper have the ability to download videos? It's been ages since I used it and maybe now it does. Thanks.

mitu

@hiulit No, it doesn't.