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

faad for aac support? #39

Open
ecifer opened this issue Nov 24, 2010 · 5 comments
Open

faad for aac support? #39

ecifer opened this issue Nov 24, 2010 · 5 comments
Milestone

Comments

@ecifer
Copy link

ecifer commented Nov 24, 2010

Hey there-

Great script, made the dts->ac3 conversion a breeze on my Snow Leopard machine. I was just wondering if you would consider adding additional input types, other than DTS? (Obviously the name of the script would become a little wonky then ;)

Using faad2, you can accomplish pretty much the same exact work as you did w/ dcadec, but for aac encodings instead of DTS.

If I'm not mistaken:

dcadec -o wavall "/mnt/media/tmp/Some.Random.Movie.dts" == faad -w "/mnt/media/tmp/Some.Random.Movie.aac"

@JakeWharton
Copy link
Owner

I had someone request AAC encoding as well (provided by faac). I'll try to take a look at a possible implementation over this long weekend.

@ecifer
Copy link
Author

ecifer commented Nov 29, 2010

Sweet... it might make more sense in your new python'd version, but with so many codecs, it might just be worth it to make a "codec module"... eg. a simple extendable object which would store codec name, decoder location, decoder flags, encoder location, encoder flags... that way the end user could select i/o formats.

Then again, it might just be easier to support if you mandate the encoders and flags :)

@choekstr
Copy link
Collaborator

I did confirm this seems to work as straightforward as we would expect. The challenge is that right now the tools, mkvinfo and mkvextract only work on mkv files. So the AAC audio stream in my MKV file came out no problem.

However, of my 200 movies the majority of them with AAC audio tracks are mp4 or m4v files and I confirmed neither of the mkvtools listed above work with mp4 files.

This would be a fairly straight forward addition to the 1.5.2 branch as long as we limit it to MKV containers. Would this be of benefit or too limited?

@JakeWharton
Copy link
Owner

I like the idea of moving to a X-to-Y in a Z container in the distant future. Any container with appropriate and similarly-designed tools can be supported. Any audio codec can be decoded and encoded to and from each other as long as the tools exist.

The script operates in three distinct steps:

  1. Extract audio from the container
  2. Transcode the audio
  3. Remux the audio back into the container

There's no reason these steps can't be abstracted to support multiple tools.

This is obviously a very long-term goal.

@Haefler1985
Copy link

Really interested in this feature (change aac 6 chanel to ac3 6 channel within mkv). I looked into the script but do not have a clou how and where to adopt.

I guess you changed from dcadec to ffmpeg. I also think ffmpeg also can do aac to ac3 but I need to change the script on many parts...

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants