-
Notifications
You must be signed in to change notification settings - Fork 40
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[encode] Doesn't seem to work but produces a valid command #39
Comments
The quoting should only be a problem if you copy-paste the command, the line is only being printed for informative purposes. Let me see |
Mh yes I think the problem is that the codec option is quite basic and the value is being split on whitespace, so your |
I don't actually know lua, and thus don't know why this doesn't work. It again produces a valid command- but nothing happens. I figured it's probably worthwhile to just explicitly have a filter complex setting. --- encode.lua 2020-09-10 13:50:55.429526500 -0400
+++ AppData/Roaming/mpv/scripts/encode.lua 2020-09-10 13:49:05.761400800 -0400
@@ -179,6 +179,12 @@
if #filters > 0 then
append_args({ "-filter:v", table.concat(filters, ",") })
end
+ -- attempt to add filter complex
+ local filterc = {}
+ if settings.append_filter_complex ~= "" then
+ filterc = settings.append_filter_complex
+ append_args({ "-filter_complex", filterc })
+ end
-- split the user-passed settings on whitespace
for token in string.gmatch(settings.codec, "[^%s]+") do
@@ -291,6 +297,7 @@
only_active_tracks = false,
preserve_filters = true,
append_filter = "",
+ append_filter_complex = "",
codec = "-an -sn -c:v libvpx -crf 10 -b:v 1000k",
output_format = "$f_$n.webm",
output_directory = "", |
I had to edit encode.lua to make it work at all (issue #38), but this issue looks like it shouldn't exist. I'm trying to use multiple complex filter graphs, and it fails, but the command that is (supposedly) used is valid and works if I copy-paste it to a regular console window.
My config file: https://pastebin.com/PV11Fpts
The text was updated successfully, but these errors were encountered: