[SCRIPT] RetroPie Convert Videos

mitu

Weird, it worked for me before with the arcade videos I downloaded from adb.arcadeitalia.net and your parameters are correct.

EDIT: can you upload somewhere the .mp4 file ?

toimas

Well... I don't think it's the videos but rather the script...
WIthout further investigation I'd suggest that the following case is rather 'counter-productive'

results+=("> \"$(basename "$video")\" --> Don't convert! Has the same Color Encoding System (C.E.S) as 'from_ces': '$from_ces'.")

I just can't think of a single case where I first check a file for matching conditions only to 'not convert' it then...

toimas

I've uploaded a yuv444p file (orginally from screenscsraper.fr)
https://www.dropbox.com/s/o0x30tzv0fpni7l/centiped-video.mp4?dl=0

Clyde

@toimas said in [SCRIPT] RetroPie Convert Videos:

I just can't think of a single case where I first check a file for matching conditions only to 'not convert' it then...

I may misunderstand you, but why check something only to ignore the result afterwards?

Conversion needs time and extra storage space, if only temporarily. Imagine that you have 5000 videos (not uncommon with MAME romsets), but only some of them are in 444p. You may not want to convert all 5000 just for a handful of them.

toimas

My goodness... so many misunderstandings...
Currently the script ignores conversion for videos THAT MATCH the FROM condition...

hiulit

Hi everyone! I've just released a new version 2.1.1 with the Skyscraper's video path fixed :)

hiulit

@memodellocos Can you try with the new version? I've merged your PR. Thanks! ;)

hiulit

@toimas I'll take a look at that when I have the time. The idea is that if from_ces and to_ces are equal, the script won't convert that video. But your settings should work.

hiulit

I've just released a new version 2.1.2.

Changelog

Fixed

The check_CES() function was checking incorrectly if the C.E.S from the video was equal to from_ces and thus not converting the videos.

@toimas Could you check if the script is working for you know? Thanks!

hiulit

BTW, the script is using libav-tools, should I change it for ffmpeg? What do you think?

mitu

@hiulit The new Debian Buster release doesn't have libav-tools anymore, they've switched back to ffmpeg (https://packages.debian.org/stretch/libav-tools). I think that you should transition to ffmpeg to have the script keep working in newer releases of Debian/Raspbian/Ubuntu.

hiulit

@mitu Thanks. I'll do it then.

Rion

@hiulit Any plans to release a standalone version for linux? Would be great to use this on my stationary desktop.

hiulit

@Rion What do you mean by standalone? With a GUI? I never though about it actually, and I don't think I will ever make such an app, mainly because I wouldn't know how, hehe :) But I could make a version of the script (or adapt it) to work without the RetroPie settings. If that's what you meant, I could do that ;)

mitu

Maybe @Rion meant 'standalone' as in 'no RetroPie installation required' - just a video conversions script. Something like Handbrake maybe ? It has a convenient GUI and can also be run from command line.

hiulit

@mitu @Rion Yes, that 'standalone' script I could do. It would just be a matter of removing all chechkings about RetroPie being installed. I will look into that once I've done the change from libav-tools to ffmpeg .

Rion

@hiulit said in [SCRIPT] RetroPie Convert Videos:

@mitu @Rion Yes, that 'standalone' script I could do. It would just be a matter of removing all chechkings about RetroPie being installed. I will look into that once I've done the change from libav-tools to ffmpeg .

Yes that's exactly what i meant. Sorry for being so vague :)

hiulit

@davidonabus It should be fixed in the new version. Could you try it?

hiulit

@Rion No worries! I'll have a look and see what I can do :)

toimas

@hiulit Thank for the update... 'sadly' I can't test it right now, I've trancoded my few videos 'by hand' meanwhile... and since I've also switched to Debian Buster I also had to use ffmpeg-tools anyway...