Skip to content
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

Sickbeard-mp4-automator merge #54

Open
wants to merge 10 commits into
base: master
Choose a base branch
from
Open

Conversation

mdhiggins
Copy link

Updated the modifications made in Sickbeard mp4 automator to the latest
version of python-video-converter in such a way that they are backwards
compatible with prior version unlike my last pull attempt.

Changes made here add the ability to output files with multiple audio
and subtitle tracks taking advantage of FFMPEGs -map function. Can pull
from multiple or single sources. to create complex files with different
tracks.

Options are presented as nested dictionaries now but previously used
style dictionaries are still compatible.

Tests will need to be updated

Updated the modifications made in Sickbeard mp4 automator to the latest
version of python-video-converter in such a way that they are backwards
compatible with prior version unlike my last pull attempt.

Changes made here add the ability to output files with multiple audio
and subtitle tracks taking advantage of FFMPEGs -map function. Can pull
from multiple or single sources. to create complex files with different
tracks.

Options are presented as nested dictionaries now but previously used
style dictionaries are still compatible.

Tests will need to be updated
@mdhiggins
Copy link
Author

Looks like a lot of my changes from the pull request I made a year ago have found their way into the project which is great. I wanted to update my script to use the latest version of your converter but I still need the greater level of functionality. This time however, I made some changes so that I don't break backwards compatibility. In my (albeit limited) testing my script is working as it previously did and I'm also able to use the readme example (using the original style options) with no problems either.

@cytec
Copy link
Contributor

cytec commented Mar 3, 2015

nice!
+1

Michael Higgins added 5 commits March 4, 2015 23:08
makes it more compliant with how the script functioned previously
(allows it to pass some of the tests)
for python 3 compatibility
@mdhiggins
Copy link
Author

Updated the script some more to pass the various tests. had to modify some of the tests due to the way command line parameters being generated were different but its passing now.

@mdhiggins
Copy link
Author

Any info on this? Any additional changes you require?

Michael Higgins added 4 commits March 20, 2015 09:08
-option to pass ffmpeg options before the inputfile and before the
output file when needed. Being used for things like `-fix_sub_duration`
and `-threads auto`. Defaults to none.
-unicodedecodeerror exception catching which can occur with certain
characters in file names breaking the script
-support for passing audio filters using the `filter` option
-bitrates violating the min or max are approximated to their closest
min/max value to avoid ffmpeg crashing due to absence of any bitrate
setting
-do not allow ac3 channels > 6. AC3 does not support any channel
settings greater than 5.1 and will cause ffmpeg to abort
includes the minimum bitrate
ffprobe read video level
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants