[Ffmpeg-devel-irc] ffmpeg-devel.log.20140501

[burek]

[02:10] <lukaszmluki> Hi, I have question, AVStream.avg_frame_rate is it always set by demuxer or not necessarily? I have {0,0} and don't know if I do something wrong or its optional (doxy doesn't say anything about it)
[02:19] <michaelni> theres no gurantee that avg_frame_rate can be calcuated and is set but you could try setting fpsprobesize
[02:19] <michaelni> if that helps, iam interrested to see the file 
[02:20] <cone-315> ffmpeg.git 03Michael Niedermayer 07master:82ccefb904b0: avformat/nut: support storing RFC 3389 comfort noise
[02:30] <cone-315> ffmpeg.git 03Michael Niedermayer 07master:feaa31d35499: ffmpeg: use av_malloc_array()
[02:31] <lukaszmluki> source movie file?
[02:31] <lukaszmluki> or code?
[02:44] <Compn> lukaszmluki : both
[02:46] <lukaszmluki> code is ffmpeg.c usnell i messed something
[02:47] <lukaszmluki> movie i can share, but nieed to cat it before so will share it later
[02:50] <lukaszmluki> abyway, scenarios when it is not set are enough for me to redesign things
[02:56] <michaelni> i meant source movie
[03:28] <samidarko> hello guys
[03:29] <samidarko> I'm trying to make small program to encode a wav/pcm file to aac format but I'm very confused by the documentation
[03:30] <samidarko> I tried the example i found but it was even hard to compile (had to modify the code because compiler said some functions were deprecated)
[03:34] <samidarko> sorry I'm moving of channel just read above
[03:34] <samidarko> bye
[05:06] <cone-315> ffmpeg.git 03Anton Khirnov 07master:eb3b5501e8b8: matroskadec: read the CodecDelay element
[05:06] <cone-315> ffmpeg.git 03Michael Niedermayer 07master:8bf90562f6cb: Merge commit 'eb3b5501e8b85bfea09d533314cb6920efc42639'
[06:16] <cone-315> ffmpeg.git 03Michael Niedermayer 07master:91736025b280: avformat/avidec: set average frame rate from header
[09:18] <ValdikSS> Hello. I'd like to add an option to vf_subtitles to load subtitles and fonts from MKV. What is the best way to do it?
[09:19] <ValdikSS> If I understand correctly, we shouldn't use another input pad with AVMEDIA_TYPE_ATTACHMENT or AVMEDIA_TYPE_SUBTITLE
[09:20] <ValdikSS> Should the filter just reopen the file?
[09:20] <ubitux> wasn't the font support recently added?
[09:21] <ubitux> see 7e6b3ad6930c2f1daa5c4831bbeda3fb931921e6
[09:21] <ValdikSS> That patch is for fonts in ass files
[09:21] <ValdikSS> not in mkv attachements
[09:21] <ubitux> i tested with an mkv, so i'm pretty sure it worked
[09:22] <ValdikSS> Hrmm
[09:22] <ValdikSS> How do I test it then?
[09:22] <ubitux> -i bla.mkv -vf subtitles=bla.mkv :p
[09:22] <ValdikSS> oh my god
[09:23] <ValdikSS> I can't believe I always extracted ass from mkv and fonts
[09:23] <ValdikSS> This should be documented
[09:23] <ubitux> i remember seeing that in the wiki at least
[09:24] <ubitux> anyway, it would be much better if we could inject subtitles & friends in lavfi
[09:26] <ValdikSS> ubitux: I can find it neither in wiki nor in documentation.
[09:27] <ubitux> ah, might have been removed, or i'm dreaming
[09:30] <ValdikSS> ubitux: can i write about it in wiki?
[09:30] <ubitux> sure go ahead
[09:30] <ValdikSS> btw, is there any way to select subtitle if there are multiple in mkv?
[09:31] <ubitux> no, but should be easily doable
[09:31] <ValdikSS> I suppose filter selects the first one
[09:31] <ubitux> the "best" one to be more specific, but yeah
[09:33] <ubitux> if you want to do that, just copy the stream_index logic from libavfilter/src_movie.c
[09:36] <ValdikSS> http://trac.ffmpeg.org/wiki/How%20to%20burn%20subtitles%20into%20the%20video
[09:37] <ubitux> :)
[09:38] <ValdikSS> ubitux: should the stream index be global index or the subtitle index?
[09:39] <ValdikSS> I mean, I'd better use subtitle index, but somebody may find global stream index more friendly
[09:39] <ubitux> i'd say you should make it ocnsistent with -c:s N
[09:39] <ubitux> (which is subtitle index)
[09:39] <ValdikSS> -c:s:n?
[09:39] <ValdikSS> ok then
[09:39] <ubitux> -codec:subtitle 4
[09:40] <ubitux> meh sorry
[09:40] <ubitux> -map s:N
[09:40] <ubitux> hard wakeup.
[09:47] <ValdikSS> ubitux: ah, we shouldn't even write code
[09:48] <ValdikSS> av_find_best_stream has a parameter wanted_stream_nb	user-requested stream number, or -1 for automatic selection 
[09:49] <ubitux> yup
[09:49] <ubitux> you still need to add the option though (and documentation)
[09:49] <ValdikSS> What is better: introduce stream_index option or subtitles=file.mkv:1 ?
[09:50] <ubitux> stream_index option, with a si alias
[09:50] <ubitux> you can't do file.mkv:1, this will make filename to file.mkv and original_size to 1
[09:58] <ValdikSS> ubitux: It seems like wanted_stream_nb wants global stream id
[09:58] <ValdikSS> even with TYPE
[10:01] <ubitux> ValdikSS: too bad... do as you pleased
[11:54] <ValdikSS> Can anybody review this patch please? https://gist.github.com/ValdikSS/0068854fa8a8525922c3
[11:57] <ubitux> why do you call av_find_best_stream() if the index is specified?
[11:58] <ubitux> you should have 2 main code path, no stream index ’ find best stream, otherwise just loop
[11:59] <ubitux> don't forget to update doc/filters.texi btw
[12:00] <ValdikSS> ubitux: >why do you call av_find_best_stream() if the index is specified?
[12:00] <ValdikSS> To detect if there any subtitles at all
[12:00] <ValdikSS> before the loop
[12:00] <ubitux> well, don't set ret = j in your loop unless you have the exact id
[12:00] <ubitux> and keep ret = -1
[12:01] <ubitux> for (j = 0; j < fmt->nb_streams; j++) if (fmt->streams[j]->codec->codec_type == AVMEDIA_TYPE_SUBTITLE) if (ass->stream_index == k) { ret = j; break; }
[12:02] <ubitux> then you don't need all the extra check
[12:12] <ValdikSS> gist updated
[12:13] <ValdikSS> Does it look good now?
[12:15] <ubitux> if (ass->stream_index <= k) ?
[12:15] <ubitux> why not increment k post check, and check for equality?
[12:19] <ValdikSS> ubitux: I always thought it's better not to check for equality but to less/more or equal
[12:19] <ubitux> well, you want that exact stream id
[12:20] <ubitux> if it's not exactly equal something is wrong
[12:21] <ValdikSS> Gist updated
[12:21] <ubitux> LGTM :)
[12:22] <ubitux> add the doc, create a git-format-patch, and i'll apply
[12:22] <ValdikSS> ubitux: Awesome! Just a minute&
[12:59] <cone-259> ffmpeg.git 03ValdikSS 07master:7a0e689c4517: lavfi/subtitles: introduce stream_index
[12:59] <ubitux> ValdikSS: tested, works fine, applied, thx
[13:00] <ValdikSS> ubitux: thanks a lot!
[13:00] <ubitux> thx to you, that's a useful feature
[14:28] <cone-259> ffmpeg.git 03Alessandro Ghedini 07master:1c0210c7981b: lavfi: add Bauer stereo-to-binaural audio filter
[14:28] <cone-259> ffmpeg.git 03Michael Niedermayer 07master:154d80eae70b: Merge commit '1c0210c7981b6a61043d9171f506b435ff5a1f5e'
[15:58] <cone-259> ffmpeg.git 03Anton Khirnov 07master:a1aa37dd0b96: matroskaenc: write CodecDelay
[15:58] <cone-259> ffmpeg.git 03Michael Niedermayer 07master:82e4f3988393: Merge commit 'a1aa37dd0b96710d4a17718198a3f56aea2040c1'
[16:11] <cone-259> ffmpeg.git 03Anton Khirnov 07master:141fdc763c28: matroska: add the Opus mapping
[16:11] <cone-259> ffmpeg.git 03Michael Niedermayer 07master:97edb195f45c: Merge commit '141fdc763c2841b572d29a2ad78513e8d5325870'
[16:22] <cone-259> ffmpeg.git 03Clément BSsch 07master:6a3a7d7af1b6: avformat/vobsub: remove pointless FAIL macro.
[16:22] <cone-259> ffmpeg.git 03Clément BSsch 07master:b7dd250b7545: avformat/vobsub: use av_make_q().
[16:47] <cone-259> ffmpeg.git 03Anton Khirnov 07master:5fccedaa6739: lavc: deprecate CODEC_FLAG_INPUT_PRESERVED
[16:47] <cone-259> ffmpeg.git 03Michael Niedermayer 07master:cac07d0e1036: Merge commit '5fccedaa67390ccddd6347c8e1c71b7664558bcd'
[16:56] <cone-259> ffmpeg.git 03Anton Khirnov 07master:b2c31710c96f: lavc: move CODEC_FLAG_NORMALIZE_AQP to mpegvideo
[16:56] <cone-259> ffmpeg.git 03Michael Niedermayer 07master:185164398763: Merge commit 'b2c31710c96fa47d9dcd40b64d39663e8957f683'
[17:04] <cone-259> ffmpeg.git 03Anton Khirnov 07master:6484149158b6: lavc: make the xvid-specific "gmc" flag a private option of libxvid
[17:04] <cone-259> ffmpeg.git 03Michael Niedermayer 07master:e40ae8cc00ae: Merge commit '6484149158b6fc6d13d2b2ef84cb26a2d3275400'
[17:13] <cone-259> ffmpeg.git 03Anton Khirnov 07master:a2941c8cb216: lavc: move CODEC_FLAG_MV0 to mpegvideo
[17:13] <cone-259> ffmpeg.git 03Michael Niedermayer 07master:e77b9853ea66: Merge commit 'a2941c8cb216bdc144953cace64973f5600ffa2d'
[17:26] <cone-259> ffmpeg.git 03Anton Khirnov 07master:18f2514c4037: mov: export stsd Compressorname in metadata
[17:26] <cone-259> ffmpeg.git 03Michael Niedermayer 07master:7426acff1aed: Merge commit '18f2514c4037befd37b02e4d4f10c159edf3b26f'
[17:31] <cone-259> ffmpeg.git 03Anton Khirnov 07master:92e4b643dfda: oggparseskeleton: do not use AVCodecContext.codec_name
[17:31] <cone-259> ffmpeg.git 03Michael Niedermayer 07master:00c3ff26f230: Merge commit '92e4b643dfdafdb6528f1baffdbea2b9a028d7c0'
[17:50] <cone-259> ffmpeg.git 03Anton Khirnov 07master:6072184e702b: asfenc: use codec descriptors instead of AVCodecs to write codec info
[17:50] <cone-259> ffmpeg.git 03Michael Niedermayer 07master:bb5657fabfec: Merge commit '6072184e702b4b631ac72f1b66b75e5f21e0ce2d'
[18:00] <cone-259> ffmpeg.git 03Anton Khirnov 07master:ef9732162cd4: rmdec: do not export anything to AVCodecContext.codec_name
[18:00] <cone-259> ffmpeg.git 03Michael Niedermayer 07master:b917d7024e36: Merge commit 'ef9732162cd4b593c6db28fdd352ebef21b5c1ca'
[18:05] <cone-259> ffmpeg.git 03Anton Khirnov 07master:e7fc9796d82a: lavc: do not use AVCodecContext.codec_name in avcodec_string()
[18:05] <cone-259> ffmpeg.git 03Michael Niedermayer 07master:d1c359777f3e: Merge commit 'e7fc9796d82abc99ef0af71027fb9aaa5311d137'
[18:16] <cone-259> ffmpeg.git 03Anton Khirnov 07master:ba71c74017c2: lavc: deprecate AVCodecContext.codec_name
[18:16] <cone-259> ffmpeg.git 03Michael Niedermayer 07master:58f4d1266bce: Merge commit 'ba71c74017c287681153ec8f6f1cba650d797275'
[18:33] <cone-259> ffmpeg.git 03Michael Niedermayer 07release/1.2:2c71734d9ef1: avcodec/vorbisdec: try to workaround libvorbisenc bug
[18:33] <cone-259> ffmpeg.git 03Michael Niedermayer 07release/1.2:3d79041f8549: sws: dont use the optimized 410->420 unscaled conversion when height%4
[18:33] <cone-259> ffmpeg.git 03Michael Niedermayer 07release/2.1:172a5cab8716: avcodec/vorbisdec: try to workaround libvorbisenc bug
[18:33] <cone-259> ffmpeg.git 03Michael Niedermayer 07release/2.1:03c5ccb4eefe: sws: dont use the optimized 410->420 unscaled conversion when height%4
[18:33] <cone-259> ffmpeg.git 03Michael Niedermayer 07release/2.2:43d64829e621: avcodec/vorbisdec: try to workaround libvorbisenc bug
[18:34] <cone-259> ffmpeg.git 03Michael Niedermayer 07release/2.2:09abca68025f: sws: dont use the optimized 410->420 unscaled conversion when height%4
[18:42] <wm4> "ffplay someimage.jpg -vf yadif" returns no output
[18:42] <wm4> is that as designed?
[18:42] <wm4> not that this command line makes sense...
[18:51] <cone-259> ffmpeg.git 03Peter Ross 07master:be098f623798: avformat/wtvdec: ignore mpeg2 extradata
[20:27] <cone-259> ffmpeg.git 03Matt Oliver 07master:5b9bb4d9ec0e: opencl: add support for non-pthread locking
[20:30] <michaelni> wm4, sounds like a bug
[20:30] <ValdikSS> wm4: what do you expect on the output?
[20:31] <wm4> ValdikSS: I'm not sure...
[20:31] <wm4> but there should be some output
[20:36] <wm4> rather than signaling EOF
[22:07] <cone-259> ffmpeg.git 03Michael Niedermayer 07master:a3477fef2bd9: ffmpeg: put the sizeof() second in av_malloc_array()
[22:07] <cone-259> ffmpeg.git 03Michael Niedermayer 07master:77811482ab04: avcodec/utils: ff_decode_frame_props: set pkt_size
[22:07] <cone-259> ffmpeg.git 03Michael Niedermayer 07master:bbc604020fac: avcodec/utils: merge ff_decode_frame_props() and ff_init_buffer_info()
[23:24] <cone-259> ffmpeg.git 03Michael Niedermayer 07master:2ff81d091606: avcodec/utils/ff_init_buffer_info factor avctx->internal->pkt; out
[23:24] <cone-259> ffmpeg.git 03Michael Niedermayer 07master:e881a9663216: avcodec/libxvid: improve return codes
[00:00] --- Fri May  2 2014