k4yt3x / video2x

A machine learning-based lossless video super resolution framework. Est. Hack the Valley II, 2018.
https://video2x.org
GNU Affero General Public License v3.0
10.74k stars 1.01k forks source link

KeyError: 'ffmpeg_hwaccel' #25

Closed Azeemotron closed 5 years ago

Azeemotron commented 5 years ago

I'm in understanding that i've installed all Prerequisites. I've checked that they are correctly installed by typing 'ffmpeg' or 'python' in cmd.

I've also defined in the Config generator the location of the installed prerequisties, although I didn't know what to write for arguments passed ffmpeg or something along those lines.

When i try to use the program and type in what is required, I am met with the follow.

===================================================================


\ \ / / (_) | | | \ \ \ / / \ \ / / _ _| | ) | \ V / \ \/ / | | / ` | / \ / \ / / > < \ / | | | (_| | | _/ | () | / / / . \ \/ || _,| _| _/ |__| // \\

           Video2X Video Enlarger

               Version 2.1.5

Traceback (most recent call last): File "video2x.py", line 300, in ffmpeg_hwaccel = config['ffmpeg_hwaccel'] KeyError: 'ffmpeg_hwaccel'

=================================================================== I did 'ffmpeg -hwaccel_device 1' in cmd thinking it would do something but it didn't fix this. Truth be told I only half know what I'm doing, but I somehow got this far.

I've probably done something wrong, any help would be much appeciated.

Many Thanks

k4yt3x commented 5 years ago

I think there might be an error in the JSON file, which is an error on my part. I'll take a look as soon as I'm home.

k4yt3x commented 5 years ago

@Azeemotron can you please check that you're on the latest version of video2x? Just do a git pull to make sure that you have the latest version of the configuration file.

Azeemotron commented 5 years ago

I just did a pull, still getting the same issue.

Could it be that I left 'Extra arguments passed to ffmpeg:' blank on Config generator?

k4yt3x commented 5 years ago

@Azeemotron Yes it would be. I just checked the config generator and what you did indeed caused a problem. I will mark this as a bug, fix it and push an update.

Update

I don't think what you did actually caused the problem. It seems like tha the config generator is outdated.

k4yt3x commented 5 years ago

@Azeemotron I've just pushed a fix for this bug. Try it out.