Opened 11 years ago
Closed 11 years ago
#3184 closed defect (fixed)
sfd: de/muxing does not properly work for cri adx and ac3
Reported by: | Soukyuu | Owned by: | |
---|---|---|---|
Priority: | normal | Component: | avformat |
Version: | git-master | Keywords: | mpegps sofdec ac3 |
Cc: | Blocked By: | ||
Blocking: | Reproduced by developer: | yes | |
Analyzed by developer: | no |
Description
Summary of the bug:
SFD (SofDec) is essentially mpeg1 video + adx audio (in later versions, also ac3 audio).
ffplay plays the file fine - adx is being played in file as well as out of file. The files come directly from the Ar tonelico 2 game, so they are official and not corrupted.
Demuxing:
- video works
- adx works
- ac3 is being detected as adx, 0 channels, cannot be demuxed.
Muxing:
- m1v + adx = mpeg with m1v+broken mp3
- m1b + adx = mpeg with m1v+broken mp3+working ac3
How to reproduce:
- Demuxing AC3
"D:\\Program Files\\ffmpeg-20131129-git-7ea9c4a-win64-static\\bin\\ffmpeg.exe" -i ".\\AT2_OP1D.SFD" -acodec copy -vn -map 0:1 -report -f ac3 "R:\\test.ac3" ffmpeg version N-58570-g7ea9c4a Copyright (c) 2000-2013 the FFmpeg developers built on Nov 28 2013 22:06:58 with gcc 4.8.2 (GCC)
- Muxing m1v + adx
"D:\\Program Files\\ffmpeg-20131129-git-7ea9c4a-win64-static\\bin\\ffmpeg.exe" -i "R:\\AT2_OP01.m1v" -i "R:\\AT2_OP01.adx" -report -c copy -map 0:v:0 -map 1:a:0 -f mpeg "R:\\out.sfd" ffmpeg version N-58570-g7ea9c4a built on Nov 28 2013 22:06:58 with gcc 4.8.2 (GCC)
- Muxing m1v + adx + ac3
"D:\\Program Files\\ffmpeg-20131129-git-7ea9c4a-win64-static\\bin\\ffmpeg.exe" -i "R:\\AT2_OP1D.m1v" -i "R:\\AT2_OP1D.adx" -i "R:\\AT2_OP1D.ac3" -map 0:v:0 -map 1:a:0 -map 2:a:0 -acodec copy -vcodec copy -report -f mpeg out.sfd ffmpeg version N-58570-g7ea9c4a Copyright (c) 2000-2013 the FFmpeg developers built on Nov 28 2013 22:06:58 with gcc 4.8.2 (GCC)
The demuxing error probably comes from the fact sfd isn't really following the format and doing something along the lines of using an adx header for ac3 stream. The muxing error is beyond me though. ffmpeg -formats lists ADX as being muxable, I am also telling it to copy the codec, yet it still botches it to mp2. I mentioned this on the irc channel some time ago and was told that the actual muxing code is not there.
Can provide samples if needed. Attaching logs only in the meantime
Attachments (2)
Change History (11)
by , 11 years ago
Attachment: | sfd_mux_demux_logs.7z added |
---|
comment:1 by , 11 years ago
Component: | undetermined → avformat |
---|---|
Keywords: | mpegps added |
Version: | unspecified → git-master |
Replying to Soukyuu:
- ac3 is being detected as adx, 0 channels, cannot be demuxed.
Please provide a sample.
Muxing sdf is not supported, if it is mentioned as supported, this would be a (separate) bug.
Please consider providing the failing command line for ac3-in-sdf together with the complete, uncut console output, please do not compress the console output.
by , 11 years ago
Attachment: | ffmpeg_mux_m1v+adx+ac3_console.txt added |
---|
muxing m1v+adx+ac3: console output
comment:3 by , 11 years ago
I am not sure what you mean by not compressing the console output. I'm using the -report switch as mentioned on the "submitting a bug report page", even adding -v 9 -loglevel 99 doesn't seem to produce the same thing I see on the console.
Attaching the console output for muxing m1v+adx+ac3
sfd_w_adx_ac3_m1v_small_bugnr3184.sfd uploaded to the ftp/incoming folder.
Note: only the first 10Mb as said in upload instructions, whole file is around 95mb, can upload this too if requested.
edit: I'm quite new to this, so please bear with me
comment:4 by , 11 years ago
Please provide the whole sample AT2_OP1D.SFD or the first xMB, as you prefer.
comment:5 by , 11 years ago
uploading AT2_OP1D.SFD in full length. Accidentally started uploading AT2_OP01.SFD but cancelled that, since it doesn't have an ac3 track. ETA is 20 minutes
comment:7 by , 11 years ago
Thanks for the fast response. The sample is now fully uploaded, but I guess you already managed to reproduce it.
By the way, sfd does not even seem to be defined as a separate format, so what I meant by "mentioned as supported" is muxing/demuxing adx (not sfd). The patch will only affect being unable to demux ac3, right? Should I make a separate issue/enhancement request to support proper sfd muxing?
comment:8 by , 11 years ago
Which application would read such samples / why do you think it would be a good idea if FFmpeg supported writing such files?
comment:9 by , 11 years ago
Keywords: | sofdec ac3 added |
---|---|
Resolution: | → fixed |
Status: | open → closed |
Decoding AC-3 should be fixed in 4ba90392 thank you for the sample!
Regarding encoding: Consider opening an enhancement request, but please explain the use-case.
7zipped logs of all commands run