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

mpv doesn't support Meridian Lossless Packing? #5923

Closed
filler56789 opened this issue Jun 14, 2018 · 8 comments

Comments

Projects
None yet
4 participants
@filler56789
Copy link

commented Jun 14, 2018

mpv version and platform

mpv 0.28.0-583-gfae9d7744c — Windows 7, x64

Reproduction steps

mpv filename.mka, where filename.mka contains MLP audio.

Expected behavior

Actual behavior

Error message below.

[mkv] Unknown/unsupported audio codec ID 'A_MLP' for track 1 or missing/faulty
[mkv] private codec data.
(+) Audio --aid=1 --alang=spa (*) ( 2ch 44100Hz)
Failed to initialize a decoder for codec ''.
Audio: no audio
No video or audio streams selected.

Exiting... (Errors when loading file)

Log file

[demux] Trying demuxers for level=normal.
[mkv] Found the head...
[mkv] + a segment...
[mkv] Parsing seek head...
[mkv] |+ segment information...
[mkv] | + muxing app: libebml v1.3.0 + libmatroska v1.4.0
[mkv] | + writing app: mkvmerge v5.8.0 ('No Sleep / Pillow') built on Sep 30 2012 10:57:14
[mkv] | + timecode scale: 22674
[mkv] | + duration: 216.112s
[mkv] | + segment uid b3 6f 3e 64 59 11 11 de af 64 ce a0 8f 7a 0a 71
[mkv] |+ segment tracks...
[mkv] | + a track...
[mkv] | + Track number: 1
[mkv] | + Track type: Audio
[mkv] | + Audio track
[mkv] | + Sampling frequency: 44100.000000
[mkv] | + Channels: 2
[mkv] | + Codec ID: A_MLP
[mkv] | + Language: spa
[mkv] |+ found cluster
[mkv] Deferring reading cues.
[mkv] All headers are parsed!
[mkv] Unknown/unsupported audio codec ID 'A_MLP' for track 1 or missing/faulty
[mkv] private codec data.
[demux] Detected file format: Matroska
[cplayer] Opening done: filename.mka
[find_files] Loading external files in .
[cplayer] Running hook: ytdl_hook/on_preloaded
[mkv] select track 0
[cplayer] (+) Audio --aid=1 --alang=spa (*) ( 2ch 44100Hz)
[ad] Codec list:
[ad] (no decoders)
[ad] Failed to initialize a decoder for codec ''.
[mkv] deselect track 0
[cplayer] Audio: no audio
[cplayer] No video or audio streams selected.
[cplayer] finished playback, no audio or video data played (reason 4)
[cplayer]
[cplayer]
[cplayer] Exiting... (Errors when loading file)
[ytdl_hook] Exiting...
[stats] Exiting...
[osc] Exiting...

@qyot27

This comment has been minimized.

Copy link
Contributor

commented Jun 14, 2018

--demuxer=lavf

Matroska is one of a few (or the only?) container format that still has its own dedicated demuxer in mpv that's separate from libavformat's, and in the case of Matroska, it is that non-lavf mkv demuxer that's the default (likely because it supports ordered chapters and libavformat's Matroska demuxer doesn't). The error about A_MLP not being recognized is coming from the mkv demuxer, so forcing lavf should fix it, provided FFmpeg/FFplay can open or play it correctly.

@CounterPillow

This comment has been minimized.

Copy link
Contributor

commented Jun 14, 2018

Do you have a sample file? Maybe it's as simple as the codec ID missing from mpv's list of mkv codec IDs.

@CounterPillow

This comment has been minimized.

Copy link
Contributor

commented Jun 14, 2018

Turns out ffmpeg even has an encoder for this that ~~~works~~~ produces output, so I could create my own sample file, and confirmed it really just was a case of adding a tag to a list.

Thanks for reporting this!

@filler56789

This comment has been minimized.

Copy link
Author

commented Jun 18, 2018

Too bad the source-code still hasn't been updated accordingly.

@CounterPillow

This comment has been minimized.

Copy link
Contributor

commented Jun 18, 2018

What? Literally just pull in from my branch if you can't wait for it to be merged upstream.

@filler56789

This comment has been minimized.

Copy link
Author

commented Jun 18, 2018

https://github.com/CounterPillow/mpv has no binaries for Windows released,
thank you very-much for nothing.

@CounterPillow

This comment has been minimized.

Copy link
Contributor

commented Jun 18, 2018

Yes, why would I build binaries for Windows for a 1-liner development branch? You fucking piece of shit ingrate, maybe learn a thing or two about software development before you demand free work from people already doing free labour for you.

@mpv-player mpv-player locked as too heated and limited conversation to collaborators Jun 18, 2018

@Akemi

This comment has been minimized.

Copy link
Member

commented Jun 18, 2018

i am not going to defend @CounterPillow's lets call it 'straightforwardness', but @filler56789 you should really work on your attitude. anyway, comments closed since the issue has been nearly resolved and i am not willing to read any more of this.

@jeeb jeeb closed this in #5924 Jun 22, 2018

jeeb added a commit that referenced this issue Jun 22, 2018

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
You can’t perform that action at this time.