Skip to content

Added _read_mp4box_stem_titles function#46

Open
naba89 wants to merge 4 commits intofaroit:masterfrom
naba89:mp4box-stem-title-reader
Open

Added _read_mp4box_stem_titles function#46
naba89 wants to merge 4 commits intofaroit:masterfrom
naba89:mp4box-stem-title-reader

Conversation

@naba89
Copy link
Copy Markdown

@naba89 naba89 commented Jan 15, 2023

Basically, for stems which have been created by the NI creator have the titles of the streams in the udta metadata.

I added the functionality to try to read the stream titles if MP4Box is available, else fall back to original default numbered streams.

This is useful if you want to extract wav files using the stem2files, and instead of getting filenames like "stem_0.wav", you can get "Drums.wav" or "Vox.wav" if this information is available in the stem files.

The code is taken almost verbatim from the StemMetadataViewer class in https://github.com/axeldelafosse/stemgen/blob/master/ni-stem/_internal.py

Basically, for stems which have been created by the NI creator have the titles of the streams in the `udta` metadata.

I added the functionality to try to read the stream titles if MP4Box is available, else fall back to original default numbered streams.

This is useful if you want to extract wav files using the stem2files, and instead of getting filenames like "stem_0.wav", you can get "Drums.wav" or "Vox.wav" if this information is available in the stem files.

The code is taken almost verbatim from the `StemMetadataViewer` class in https://github.com/axeldelafosse/stemgen/blob/master/ni-stem/_internal.py
@axeldelafosse
Copy link
Copy Markdown

Nice!

@naba89
Copy link
Copy Markdown
Author

naba89 commented Feb 14, 2023

I added sudo apt-get update to fix the previous issue of 404 errors while installing apt-get packages.

But still, the CI failed with some errors in FFmpeg unable to find x264 libraries.
ffprobe: error while loading shared libraries: libx264.so.138: cannot open shared object file: No such file or directory

@axeldelafosse
Copy link
Copy Markdown

@faroit sorry for the ping -- can you have a look when you have some free time please? 🙏

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.

3 participants