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

burek burek021 at gmail.com
Sat Apr 13 02:05:02 CEST 2013


[00:03] <ubitux> no one wants to deal with scale filter? :(
[00:06] <michaelni> ubitux, you could just drop the single unnamed parameter case
[00:07] <ubitux> well it's not that it's hard, it just takes me quite some time to do all these filters
[00:07] <ubitux> i'll go for pan now
[00:08] <ubitux> which is tricky as well
[00:08] <saste> michaelni, i have a few patches to push
[00:09] <saste> do you want me to wait, or you're already done
[00:13] <ubitux> pan is actually gonna get extremely tricky
[00:13] <ubitux> unless i run some escaping just like mp
[00:14] <michaelni> saste, i wrote a script to rebase a merge so its less annoying now, at least for simple case
[00:14] <michaelni> cases
[00:21] <cone-607> ffmpeg.git 03Stefano Sabatini 07master:a77454ec145b: lavfi/overlay: reindent constants in option list
[00:21] <cone-607> ffmpeg.git 03Stefano Sabatini 07master:ed2c827575a6: lavfi/overlay: add repeatlast option
[00:21] <cone-607> ffmpeg.git 03Stefano Sabatini 07master:aff6cebb4166: lavfi/overlay: fix crash in case of invalid expression
[00:21] <cone-607> ffmpeg.git 03Stefano Sabatini 07master:b2e22731e4ee: doc/filters: apply minor fixes to hue Commands section
[00:21] <cone-607> ffmpeg.git 03Stefano Sabatini 07master:fdedfc029b35: doc/filters: rework Commands section for overlay filter
[00:21] <cone-607> ffmpeg.git 03Stefano Sabatini 07master:9da369604ecf: lavfi/overlay: improve feedback in case of invalid expression
[00:22] <ubitux> saste: thx for a77454ec :)
[00:34] <ubitux> well i'm really too tired to deal with the remaining filters
[00:39] <cone-607> ffmpeg.git 03Anton Khirnov 07master:bc1a985ba030: lavfi: replace avfilter_open() with avfilter_graph_alloc_filter().
[00:39] <cone-607> ffmpeg.git 03Michael Niedermayer 07master:86070b8e5a0e: Merge commit 'bc1a985ba030e9861d24965d42792850b43a43ea'
[00:44] <cone-607> ffmpeg.git 03Anton Khirnov 07master:c2c9801bc9bc: lavfi: deprecate avfilter_graph_add_filter().
[00:44] <cone-607> ffmpeg.git 03Michael Niedermayer 07master:5c68e14e42c1: Merge commit 'c2c9801bc9bce688d51d1a96f5f3ea93933e2dee'
[00:48] <durandal11707> ubitux: did you tested ivtc with our telecine?
[00:48] <ubitux> no, not yet
[00:51] <cone-607> ffmpeg.git 03Anton Khirnov 07master:111367263af4: lavfi: add AVFilterContext.graph.
[00:51] <cone-607> ffmpeg.git 03Michael Niedermayer 07master:4fde70539675: Merge commit '111367263af41c88a44bd763ceefc11d53a7f655'
[00:59] <cone-607> ffmpeg.git 03Anton Khirnov 07master:1565cbc65cbb: lavfi: make avfilter_free() remove the filter from its graph.
[00:59] <cone-607> ffmpeg.git 03Michael Niedermayer 07master:eb0f774d4bfd: Merge commit '1565cbc65cbb9f95c11367314a080068895e0cf0'
[01:02] <ubitux> any idea what anton is planing with the filter removing from a graph?
[01:05] <durandal11707> what that silly thing that guy removed in tinterlace filter?
[01:05] <ubitux> ?
[01:08] <saste> ubitux, dynamic reconfiguration?
[01:09] <ubitux> saste: that's what i thought, but why is it necessary?
[01:10] <ubitux> i mean, why removing the filter from the graph actually helps
[01:10] <saste> only anton knows, and they don't discuss design on list, they just say "ok"
[01:10] <ubitux> :D
[01:38] <cone-607> ffmpeg.git 03Anton Khirnov 07master:48a5adab62bd: lavfi: add avfilter_init_str() to replace avfilter_init_filter().
[01:38] <cone-607> ffmpeg.git 03Michael Niedermayer 07master:710b0aa8012a: Merge commit '48a5adab62bd2a553f5069d41fa632a0701835e5'
[02:40] <cone-607> ffmpeg.git 03Michael Niedermayer 07master:9ab796127a38: avfilter: Update scale to dictionary based options
[09:40] <cone-607> ffmpeg.git 03Pavel Koshevoy 07master:8f3c440a39e8: lavfi/atempo: switch to an AVOptions-based system.
[09:54] <cone-607> ffmpeg.git 03Michael Niedermayer 07master:16e5e13c7e46: buffersrc: switch to an AVOptions-based shorthand system.
[10:00] <cone-607> ffmpeg.git 03Michael Niedermayer 07master:c378ba19b5c0: abuffer: switch to an AVOptions-based shorthand system.
[10:20] <cone-607> ffmpeg.git 03!5 at 6 8A:>@>4>2 07master:9470b541e5c4: af_pan: fix parsing arguments from the example in the documentation
[10:33] <durandal_1707> anybody working on remaining filters?
[10:34] <durandal_1707> this is getting ridiculous, how long it passed why I uploaded gif disposal background to fate and still same machines have not done rsync
[10:34] <durandal_1707> s/why/when
[10:36] <ubitux> durandal_1707: i'm not on them
[10:37] <michaelni> durandal_1707, which machines miss it ?
[10:39] <durandal_1707> michaelni: only one, cygwin something, http://fate.ffmpeg.org/history.cgi?slot=x86_64-win7-cygwin-gcc-4.5.3
[10:41] Action: michaelni is updating af_pan
[10:42] <michaelni> michaelni, sent mail to beastd about cygwin (i think he is admin of it)
[10:43] <cone-607> ffmpeg.git 03Michael Niedermayer 07master:3c821e755086: af_pan: switch to an AVOptions-based shorthand system.
[10:45] <durandal_1707> michaelni: that commit added some funny history in log output
[10:47] <durandal_1707> ugh, ignore, i did not know about addititonal hacks added there
[10:48] <ubitux> michaelni: will that pan commit enough to deal with the '='?
[10:48] <ubitux> pan = 5 : c0=c0 : c1=c1 : ... ?
[10:49] Action: ubitux cleaning the doc
[10:49] <durandal_1707> but why we are updating filters if init_filter still have >1 args?
[10:50] <michaelni> ubitux, do you have a failing command line ?
[10:52] <ubitux> michaelni: if pan=stereo:c0=c1:c1=c0 works then it should be ok
[10:54] <cone-607> ffmpeg.git 03Michael Niedermayer 07master:80116b6e6755: af_pan: avoid spaces before % to avoid possible issues with MSVC
[10:55] <michaelni> ubitux, no errors just the | vs : spam
[10:56] <ubitux> ok well then i guess it's good
[10:59] <cone-607> ffmpeg.git 03Clément BSsch 07master:8876b0816d39: doc/filters: update blackframe options description.
[10:59] <cone-607> ffmpeg.git 03Clément BSsch 07master:fc15ad408c78: doc/filters: update boxblur options description.
[10:59] <cone-607> ffmpeg.git 03Clément BSsch 07master:9811b7942db3: doc/crop: update boxblur options description.
[10:59] <cone-607> ffmpeg.git 03Clément BSsch 07master:9c5f392cbcf0: doc/cropdetect: update boxblur options description.
[11:00] <ubitux> looks like it's gonna take me the day
[11:00] <durandal_1707> what?
[11:01] <cone-607> ffmpeg.git 03Michael Niedermayer 07master:04924bc923ef: avfilter: remove old shorthand system
[11:02] <ubitux> michaelni: i think aconvert will need some update as well actually (if you want to drop the args from the init)
[11:03] <ubitux> durandal_1707: most of the filters need an update of their doc; i need to check for aliases, order, non-merged doc changes, etc
[11:03] <durandal_1707> am i the only one that think vf_copy code can be simplified?
[11:04] <ubitux> what is copy used for?
[11:04] <durandal_1707> how to handle this destruct deprecated warning in lavf?
[11:04] <durandal_1707> ubitux: to copy frame unchanged
[11:05] <ubitux> sounds like an ineffective passthrough
[11:05] <durandal_1707> see b5670209438481f21addc8b7c4891627dd5ddac7
[11:05] <durandal_1707> avfilter_draw_slice is gone
[11:06] <durandal_1707> and it was much simpler in old days
[11:06] <ubitux> the filter looks completely useless nowadays
[11:06] <ubitux> unless i'm missing something
[11:07] <durandal_1707> leave it to l to deprecate it?
[11:07] <ubitux> i guess so yes
[11:07] <ubitux> if you want to play with copies, just use (a)perms
[11:07] <ubitux> otherwise i don't see much point in this
[11:08] <durandal_1707> i thought about replacing that code with clone_frame
[11:08] <durandal_1707> or clone != copy?
[11:09] <ubitux> it will copy if necessary
[11:09] <ubitux> but really it makes no sense to "copy" anyway afaik
[11:10] <ubitux> except eventually testing your ram
[11:10] <durandal_1707> :)
[11:11] <ubitux> michaelni: can we drop shorthand from AVFilter now?
[11:11] <ubitux> (it's not public, we can drop without any problem, right?)
[11:16] <durandal_1707> anybody fixing scale spam?
[11:16] <ubitux> and we'll likely have to deprecate av_opt_set_from_string now
[11:16] <ubitux> durandal_1707: 'not working on it
[11:16] <durandal_1707> [auto-inserted scaler 0 @ 0x29c50680] The <w>:<h>:flags=<flags> option syntax is deprecated. Use either <w>:<h>:<flags> or w=<w>:h=<h>:flags=<flags>
[11:19] <michaelni> ubitux, feel free to drop shorthand
[11:20] <ubitux> ok
[11:32] <durandal_1707> where is all scale filter auto inserted?
[11:40] <ubitux> libavfilter/avfiltergraph.c:                    if (!(filter = avfilter_get_by_name("scale"))) {
[11:41] <ubitux> anyone heard of vaapi crashes?
[11:42] <durandal_1707> michaelni: patch to silence warnings sent
[11:43] <durandal_1707> *scale warnings
[11:47] <cone-607> ffmpeg.git 03Paul B Mahol 07master:ae8d435ba8bc: doc/filters: add quick emboss example
[11:50] <durandal_1707> ubitux: iirc you were mentioned twitter?
[11:55] <cone-607> ffmpeg.git 03Clément BSsch 07master:dee2db523d40: doc/filters: update volume options description.
[11:55] <ubitux> durandal_1707: huh?
[11:56] <cone-607> ffmpeg.git 03Michael Niedermayer 07master:491d261adece: avfilter/asrc_aevalsrc: check correct variable
[11:56] <cone-607> ffmpeg.git 03Anton Khirnov 07master:d69a4177b988: lavfi: remove now unused args parameter from AVFilter.init
[11:56] <cone-607> ffmpeg.git 03Michael Niedermayer 07master:fd6228e65711: lavfi: remove now unused args parameter from AVFilter.init and init_opaque
[11:58] <durandal_1707> ubitux: oh it was saste
[11:59] <ubitux> 491d261ad is wrong; args1 can not be set in that scope; you can just ret = AVERROR(ENOMEM) here
[11:59] <ubitux> michaelni ^
[12:02] <michaelni> ubitux, hmm
[12:03] <michaelni> ill fix
[12:03] <ubitux> michaelni: or well, you should check for the string in the context to be set instead
[12:05] <ubitux> -        ret = args1 ? AVERROR(ENOMEM) : AVERROR(EINVAL);
[12:05] <ubitux> +        ret = eval->exprs ? AVERROR(EINVAL) : AVERROR(ENOMEM);
[12:05] <ubitux> this should do the trick
[12:05] <ubitux> ...
[12:05] <ubitux> without my stupid switch
[12:05] <durandal_1707> ubitux: what's about that s=%s comment?
[12:06] <ubitux> ?
[12:06] <ubitux> durandal_1707: i just prefer the s= syntax, it's simpler
[12:06] <ubitux> and i just hope it's still supported
[12:07] <ubitux> possibly "faster" btw, since it's not parsed as an expression
[12:07] <durandal_1707> ubitux: if i use scale=s=800x600 i get again that retarded spam (which is incorrect btw)
[12:08] <cone-607> ffmpeg.git 03Michael Niedermayer 07master:3d7d819aad52: avfilter/asrc_aevalsrc: fix return code, 2nd try
[12:08] <ubitux> durandal_1707: huh?
[12:08] <durandal_1707> this is big mess, switch all filter users to opt based thing, snprintfs hacks is wasting by brain cells
[12:08] <ubitux> there is no reason to warn in the scale=s=800x600 case
[12:08] <ubitux> if it warns there is a bug
[12:09] <ubitux> and i believe that bug is because of the limited-libav-based wrapper in avfilter.c
[12:10] <ubitux> the reason is that libav doesn't support s/size
[12:13] <ubitux>             strchr(args, ':') < strchr(args, '=')) {
[12:13] <ubitux> this check is wrong
[12:14] <ubitux> scale=s=800x600 ’ NULL < 0x...
[12:14] <ubitux> it needs something similar to what's done for aevalsrc
[12:15] <ubitux> anyway, the compat wrapper need to be reworked
[12:15] <michaelni> ubitux, why does scale need this hack in avfilter.c at all ?
[12:16] <ubitux> it might not be needed, except to warn
[12:16] <ubitux> maybe it's not needed actually
[12:17] <michaelni> ok, ill try to remove it
[12:18] <ubitux> yeah right it's not needed at all
[12:18] <ubitux> it's just a wrapper because libav doesn't support mixed/long
[12:18] <ubitux> so they can't have w:h:flags=...
[12:18] <cone-607> ffmpeg.git 03Nicolas George 07master:af0d270aac86: lavu: add helper functions for integer lists.
[12:18] <cone-607> ffmpeg.git 03Nicolas George 07master:e48ded855117: lavfi/buffersink: accept parameters as options.
[12:18] <cone-607> ffmpeg.git 03Nicolas George 07master:62d888ef0e3b: ffmpeg: use options for abuffersink.
[12:20] <ubitux> shorthand?
[12:20] <ubitux> again :(
[12:22] <ubitux> durandal_1707: after the avfilter scale hack is dropped, your patch should not be necessary
[12:25] <michaelni> ill remove the shorthand in a moment
[12:25] <cone-607> ffmpeg.git 03Michael Niedermayer 07master:f1e62af0e03f: avfilter: remove mixed args hack for scale.
[12:26] <ubitux> yay :)
[12:29] <ubitux> michaelni: libavfilter/vf_libopencv.c:100:9: error: args undeclared (first use in this function)
[12:29] <cone-607> ffmpeg.git 03Michael Niedermayer 07master:4a8898925460: buffersink: remove shorthand
[12:31] <michaelni> ubitux, ERROR: opencv not found
[12:32] <durandal_1707> opencv need switching to opt
[12:32] <durandal_1707> (i said that yesterday i think)
[12:32] <ubitux> durandal_1707: it has a compat layer
[12:32] <ubitux> michael dropped too much "args" in a recent patch
[12:33] <durandal_1707> ahh
[12:33] <ubitux> the sub functions takes an args grabbed from the local context
[12:33] <durandal_1707> could yust remove compat layer?
[12:33] <ubitux> noo
[12:33] <ubitux> just give me a minute i'll fix it
[12:33] <durandal_1707> i mean in ocv
[12:34] <ubitux> ocv already has opt anyway
[12:34] <durandal_1707> ahh it insternal thing ...... confusing everyone
[12:35] <durandal_1707> diracdec still touch deprecated stuff....
[12:35] <durandal_1707> is it actually still working?
[12:38] <cone-607> ffmpeg.git 03Clément BSsch 07master:51e71d6e1127: lavfi/ocv: fix build after fd6228e65.
[12:39] <durandal_1707> what will be done with l setpts filter?
[12:42] <ubitux> wow ocv is nicely broken
[12:44] <cone-607> ffmpeg.git 03Clément BSsch 07master:490786c02e85: lavfi/ocv: fix crash with no filter name specified.
[12:45] <ubitux> ok, now another crash..
[12:45] <ubitux> seems that was well tested
[12:45] <ubitux> there is even a warning about the problem
[12:45] <ubitux> :p
[12:46] <durandal_1707> ffmpeg -f lavfi -i smptebars=hd980
[12:48] <ubitux> didn't you remove some smptebars opt thing? :p
[12:48] <durandal_1707> no this crash if you set invalid video size string
[12:49] <durandal_1707> happens with testsrc too
[12:50] <durandal_1707> for some reason life swithced to monob as default format
[12:54] <ubitux> wasn't it already the case?
[12:54] Action: ubitux fixing ocv
[12:57] <durandal_1707> michaelni: know reason of that regression?
[12:58] <durandal_1707> something broke in opt handling recently
[13:00] <xlinkz0> are there any examples for 0.5 version?
[13:00] <durandal_1707> that is extremly old
[13:01] <durandal_1707> and unmaintained
[13:01] <xlinkz0> well.. it's the one on debian :(
[13:01] <xlinkz0> squeeze
[13:01] <cone-607> ffmpeg.git 03Clément BSsch 07master:f16a6f667c99: lavfi/ocv: fix crash with no dilate arguments.
[13:01] <durandal_1707> are you sure that is really ffmpeg and not libav?
[13:01] <ubitux> xlinkz0: 0.5 please...
[13:02] <xlinkz0> it's both
[13:02] <michaelni> libav did not exist at 0.5 times
[13:02] <xlinkz0> i just need to get some info about the file.. nothing fancy yet
[13:02] <ubitux> xlinkz0: 0.5 was release beginning 2009...
[13:03] <xlinkz0> that's just 4 years ago !
[13:03] <ubitux> just 30k commits ago
[13:03] <xlinkz0> so no examples yet?
[13:03] <xlinkz0> ok i'll try to compile latest
[13:05] <xlinkz0> is yasm 0.8 old?
[13:06] <ubitux> ...
[13:06] <durandal_1707> michaelni: actually bug is in lavfi
[13:06] <durandal_1707> *device
[13:06] <ubitux> xlinkz0: you should seriously think about upgrading your system
[13:06] <xlinkz0> "upgrading" a stable distro makes in unstable
[13:07] <ubitux> yes but maybe usable
[13:09] <ubitux> michaelni: you switched the way of deleting filters from a graph, and it seems to be what's causing the -f lavfi -i smptebars=hd980 crasg
[13:09] <ubitux> crash*
[13:17] <J_Darnley> <xlinkz0> "upgrading" a stable distro makes in unstable <-- That must be why I'm still using XP
[13:17] Action: Compnn still on win2k.
[13:17] <Compnn> ehe
[13:17] <J_Darnley> :)
[13:17] <xlinkz0> you really don't afford more than 4gb ram?
[13:17] <J_Darnley> Of DDR?  Of Course I can't afford it!
[13:18] <Compn> i chose my software via how not bloated it is
[13:18] <Compn> so my web browser only uses 300mb ram for 100 tabs :P
[13:18] <Compn> javascript disabled 
[13:18] <kierank> oh
[13:18] <kierank> which browser?
[13:18] <kierank> can't be firefox
[13:18] <kierank> ie is pretty light on memory
[13:18] <Compn> old opera
[13:18] <kierank> ah
[13:19] <Compn> new opera sucks, chrome wont run on win2k, firefox wont even run on win2k anymore :P
[13:19] <kierank> upgrade to xp
[13:19] <kierank> xp is the best OS
[13:19] <kierank> I'll be sad to see updates go
[13:19] <Compn> everyone links against xp dlls when building
[13:20] <JEEB> well, there are some actual new APIs with XP
[13:20] <JEEB> which I can see useful
[13:20] <Compn> all of mplayers hard fought windows video support probably wont work too well in new windows :\
[13:20] <Compn> i remember it shuts down aero if using the wrong vo
[13:21] <JEEB> atomics and unaligned alloc etc.
[13:21] <Compn> and people love that aero crap
[13:21] <JEEB> desktop composition is nice
[13:21] <JEEB> overall vsync for the whole screen
[13:21] <kierank> not that useful
[13:23] <JEEB> also at least for me the newer opengl-based renderer by wm4 works Just Fine on 7+, and I guess if someone /really/ wanted a well-supported NT6+ renderer they'd plug in EVR to mplayer/mplayer2/mpv in one way or another
[13:36] <Compn> i thought someone was working on icod 
[13:38] <ubitux> durandal_1707: http://pastie.org/7463412 fixes your crash
[13:40] <durandal_1707> ubitux: so you want approval or what?
[13:41] <ubitux> no i'm just wondering about the code aroung
[13:41] <ubitux> around
[13:45] <cone-607> ffmpeg.git 03Michael Niedermayer 07master:a1e7e02eaffd: abuffersink: switch from opaque to AVOptions for params
[13:52] <durandal_1707> ubitux: so you push it or?
[13:52] <ubitux> in a few minutes eventually
[13:53] <cone-607> ffmpeg.git 03Michael Niedermayer 07master:3fc7b47145da: buffersink: switch from opaque to AVOptions for params
[14:00] <cone-607> ffmpeg.git 03Michael Niedermayer 07master:0acf7e268b2f: avfilter: remove opaque passing hacks
[14:13] <cone-607> ffmpeg.git 03Clément BSsch 07master:6b5ec762830d: lavfi: fix forgotten chunk in eb0f774d.
[14:22] <durandal_1707> hmm lavfi does not support creating/filtering frames only on demand? 
[14:30] <durandal_1707> there is no filter to pass N inputs to one output?
[14:30] <cone-607> ffmpeg.git 03Anton Khirnov 07master:1ba95a9cca57: lavfi: add avfilter_init_dict() for initializing a filter with a dict.
[14:30] <cone-607> ffmpeg.git 03Michael Niedermayer 07master:46de9ba59815: Merge commit '1ba95a9cca57b023b9b9de071a5671fc05b05e58'
[14:38] <durandal_1707> huh buffer example mentions size but that is not in code
[14:50] <cone-607> ffmpeg.git 03Anton Khirnov 07master:7cdd737ba81b: lavfi: mark filters with dynamic number of inputs or outputs with special flags
[14:50] <cone-607> ffmpeg.git 03Michael Niedermayer 07master:16fc24b240ca: Merge commit '7cdd737ba81b5c2c9521c4509edf0ac315fabc65'
[14:50] <durandal_1707> ubitux: see http://www.vapoursynth.com/doc/functions/splice.html#std.Splice
[14:54] <cone-607> ffmpeg.git 03Anton Khirnov 07master:7e8fe4be5fb4: lavfi: add a function for counting elements in AVFilterPad arrays.
[14:54] <cone-607> ffmpeg.git 03Michael Niedermayer 07master:f4db6bfeb86f: Merge commit '7e8fe4be5fb4c98aa3c6a4ed3cec999f4e3cc3aa'
[15:14] <cone-607> ffmpeg.git 03Anton Khirnov 07master:fa2a34cd40d1: lavfi: change the filter registering system to match the other libraries
[15:14] <cone-607> ffmpeg.git 03Michael Niedermayer 07master:835cc0f2e709: Merge commit 'fa2a34cd40d124161c748bb0f430dc63c94dd0da'
[15:16] <ubitux> durandal_1707: ok.. ? :p
[15:16] <ubitux> michaelni: "This removes the ability to put AVFilters in read only memory and having them shareable."
[15:16] <ubitux> what does that mean?
[15:18] <durandal_1707> ubitux: vsynth:splice = lavfi:concat, lavfi:splice == vsynth:interleave
[15:18] <cone-607> ffmpeg.git 03Anton Khirnov 07master:f160c6a18a57: lavfi: do not segfault on NULL passed to avfilter_get_by_name()
[15:18] <cone-607> ffmpeg.git 03Michael Niedermayer 07master:9a91c67478cd: Merge commit 'f160c6a18a574b4c7d063e0a748037cbbd9912ee'
[15:20] <ubitux> ah, ok
[15:22] <durandal_1707> ubitux: note that l added trim filter which is afaik exact in behaviour to vsynth one
[15:22] <durandal_1707> perhaps they gonna clone vsynth?
[15:23] <durandal_1707> or it is just coincidence
[15:23] <ubitux> as long as it's not introducing compat break idc
[15:24] <burek> would it be possible to create a simple voip-like thing with ffmpeg, using ffmpeg/ffplay/ffserver, using for example a simple wrapper (let's call it ffvoip or ffspeak or so) which could use all the features already present in ffmpeg/ffplay/ffserver
[15:24] <burek> it would only call all those tools with proper params
[15:25] <burek> it might boost popularity of ffmpeg even more and the best thing is that all the parts needed, are already implemented in various parts of ff tools
[15:26] <durandal_1707> the tile example does not work any more
[15:27] <burek> i'll try to draw an example and create a ticket, it will be easier that way
[15:30] <durandal_1707> ubitux: do you get assertions now when using testsrc with ffplay?
[15:30] <ubitux> indeed.
[15:31] <cone-607> ffmpeg.git 03Anton Khirnov 07master:bee2d75b66e2: avconv: update to the new options syntax.
[15:31] <cone-607> ffmpeg.git 03Michael Niedermayer 07master:7b6c0bda9e94: Merge commit 'bee2d75b66e2f02e0877796c01202299a4b56cfb'
[15:31] <durandal_1707> perhaps gmake clean is needed
[15:38] <durandal_1707> seems to be ffplay only issue
[15:52] <cone-607> ffmpeg.git 03Anton Khirnov 07master:dc574658d15a: avconv: print filter options in -h full output.
[15:52] <cone-607> ffmpeg.git 03Michael Niedermayer 07master:f81a8e89a1c2: Merge commit 'dc574658d15ad8f171dbdecbdce7197acf9424a0'
[15:52] <cone-607> ffmpeg.git 03Michael Niedermayer 07master:292f68154b66: avfilter: Filter options that are used for both video and audio should have both flags
[15:52] <cone-607> ffmpeg.git 03Michael Niedermayer 07master:1007de703305: asetpts: add AVOptions
[15:52] <cone-607> ffmpeg.git 03Michael Niedermayer 07master:3dedcef8b889: avfilter: add missing AV_OPT_FLAG_FILTERING_PARAM
[16:01] <cone-607> ffmpeg.git 03Anton Khirnov 07master:e19e8aeeaaff: avtools: move media_type_string() from avprobe to cmdutils.
[16:01] <cone-607> ffmpeg.git 03Michael Niedermayer 07master:608d86013a86: Merge commit 'e19e8aeeaaff3d4bc8f6065764c93f0298782ac6'
[16:03] <ubitux> durandal_1707: i think the example will break if you don't use the vsync opt
[16:05] <durandal_1707> ubitux: does not break here, perhaps i need some special file
[16:06] <ubitux> durandal_1707: look at #1644
[16:06] <durandal_1707> ubitux: the older example works with avi and mpeg4 , but not with mkv and avc
[16:11] <durandal_1707> anybody found why ffplay crashes?
[16:13] <durandal_1707> ubitux: using -skip_frame nokey is much faster because it decodes less...
[16:13] <durandal_1707> but i don't think that can work with avc
[16:18] <ubitux> http://starlogs.net/#ffmpeg/ffmpeg
[16:21] <cone-607> ffmpeg.git 03Anton Khirnov 07master:921046008f53: cmdutils: allow -h filter=<name> to print information about a filter.
[16:21] <cone-607> ffmpeg.git 03Michael Niedermayer 07master:7e68491367ce: Merge commit '921046008f5335601f9d76dc3a768eb5cd91e1b9'
[16:24] <durandal_1707> ubitux: what is that?
[16:24] <av500> click on it
[16:25] <durandal_1707> i only heard loud music
[16:26] <av500> open your eyes then
[16:27] <durandal_1707> av500: i see only one big image
[16:28] <av500> it shows commit logs
[16:29] <durandal_1707> i don't see them it opera
[16:32] <ubitux> too bad
[16:37] <av500> people in Belarus will suffer most
[16:37] <cone-607> ffmpeg.git 03Vittorio Giovara 07master:3fce13679837: lavfi: new interlace filter
[16:37] <cone-607> ffmpeg.git 03Vittorio Giovara 07master:25882a7ff642: FATE: add a test for the interlace filter
[16:37] <cone-607> ffmpeg.git 03Michael Niedermayer 07master:ebbd4fd5f06a: Merge remote-tracking branch 'qatar/master'
[16:40] <durandal_1707> now we have 3 useless interlacers
[16:42] Action: michaelni thought there where more
[16:42] <av500> interlacers or deinterlacers?
[16:44] <durandal_1707> number of useless deinterlacers is even higher
[16:44] <av500> is there more than one way to interlace?
[16:45] <av500> maybe vertically too?
[16:45] <cone-607> ffmpeg.git 03Michael Niedermayer 07master:227b4458fb43: vf_interlace: fix frame rate
[16:46] <michaelni> av500, yes you can interlace more than 2 fields
[16:46] <michaelni> for example one can interlace 6 fields
[16:51] <kierank> av500: interlace the luma but not the chroma :)
[16:51] <kierank> michaelni: how?
[16:51] <av500> kierank: maybe for a video wall :)
[16:52] <kierank> iphone on its side
[16:52] <kierank> as a line
[16:52] <michaelni> kierank, i dont know but i think ive a sample IIRC so there must have been a way
[17:16] <j-b> michaelni: btw, what source did you take for your downmixing matrices?
[17:18] <michaelni> j-b, thats a good question, i think google and some audio decoder specs maybe (e)ac3 or aac i dont remember exactly
[17:19] <michaelni> note, i assume you talk about swr
[17:19] <michaelni> i also tested it 
[17:19] <michaelni> j-b, why do you ask ?
[17:20] <j-b> I do talk about swr
[17:25] <durandal_1707> my ffplay broke
[17:31] Action: Snaggle is trying to write a patch for the opencl detection I mentioned yesterday.  But I'm having problems with getting a HAVE_CL_CL_H token declared.  I changed configure to use check_lib to find the right heard/lib combo because the current require2 didn't work when checking different combinations.
[17:33] <j-b> michaelni: ITU-R BS.775-2
[17:36] <michaelni> durandal_1707, which commit broke what ?
[17:37] <durandal_1707> ffplay crashes when displaying any video
[17:37] <durandal_1707> audio seems to work here
[17:43] <michaelni> durandal_1707, no crash here
[17:43] <michaelni> j-b, is this a feature request ?
[17:44] <j-b> michaelni: no.
[17:44] <j-b> michaelni: just wondering about doc
[17:49] <michaelni> Snaggle, HAVE_CL_CL_H would be defined if theres a CL/cl.h only id guess
[17:49] <Snaggle> michaelni: same thought here, but if I flip the logic around (look for OpenCL/cl.h), HAVE_OPENCL_CL_H is not being defined either
[17:50] <Snaggle> I'm following the example of the libopenjpeg check since that seems closest
[17:53] <michaelni> Snaggle, you did look in config.log ?
[18:00] <Snaggle> ahhh..  I probably have to add the token to the HAVE_LIST variable in configure
[18:05] <Snaggle> oh good
[18:15] <j-b> michaelni: I was looking for 7.1 to 5.1 matrices, to be exact
[18:18] <kurosu> michaelni: re apply noise not applying correctly, strange, as I have just pull-rebased against ffmpeg's master without issue
[18:20] <kurosu> I'll regenerate them anyway
[18:21] <michaelni> kurosu, thx, i think some define NREGS was lost
[18:21] <kurosu> right, probably when rebasing from libav to ffmpeg
[18:21] <kurosu> and indeed
[18:23] <cone-607> ffmpeg.git 03Stefano Sabatini 07master:b59aa007077f: doc/libavutil: remove include of opencl.texi
[18:23] <cone-607> ffmpeg.git 03Stefano Sabatini 07master:949c8627b639: doc: merge syntax.texi, eval.texi and opencl.texi into utils.texi
[18:23] <cone-607> ffmpeg.git 03Stefano Sabatini 07master:9ae9ae10f9fe: doc/utils: apply misc fixes to OpenCL docs
[18:23] <kurosu> I have probably ran the fate-aac test twice in the same build dir...
[18:23] <saste> burek, this ffspeak thing could be everything
[18:24] <saste> also i suppose this is what ffserver is supposed to be
[18:24] <saste> but as you know we lack a maintainer for it
[18:28] <kurosu> oh, %elifdef works
[18:38] <durandal_1707> i will do convolution filter next
[18:43] <saste> durandal_1707, good
[18:43] <saste> we have already relevant code in edgedetect and transform
[18:44] <durandal_1707> saste: transform?
[18:45] <saste> durandal_1707, libavfilter/transform.h
[18:45] <saste> used by deshake
[18:46] <ubitux> durandal_1707: did you look at the ffplay -f lavfi testsrc problem?
[18:47] <durandal_1707> ubitux: what problem? crash for lavfi have been fixed, but ffplay now crashes when displaying any video (expcept internal spectrum)
[18:47] <durandal_1707> michaelni could not reproduce it
[18:48] <durandal_1707> actually its av_assert crash
[18:48] <durandal_1707> saste: its 3x3 only matrix
[18:48] <kurosu> michaelni: about the ac3 FP implementation by the mips guy, I feel bad reviewing it while it's not up to me to decide what to do of it
[18:49] <durandal_1707> thought that one header looks to not be insttalled, but have av* prefix ,. .....
[18:49] <ubitux> durandal_1707: i have a problem playing videos with ffplay, only with -f lavfi testsrc triggering a lavu/imgutils assert
[18:50] <ubitux> life is broken
[18:50] <ubitux> well something in lavfi looks broken
[18:51] <saste> ubitux, same here
[18:51] <durandal_1707> Assertion abs(dst_linesize) >= bytewidth failed at libavutil/imgutils.c:249
[18:51] <ubitux> yes, that one
[18:51] <ubitux> with anything involving lavfi
[18:51] <saste> something related to the recent refactoring wave
[18:51] <durandal_1707> but i have that always ....
[18:52] <ubitux> durandal_1707: most likely a filter is inserted all the time
[18:52] <durandal_1707> i think its probably because filters are differently created/allocated 
[18:53] <michaelni> kurosu, if you want to make the final decission on ac3 FP, i dont mind
[18:53] <michaelni> mips ac3 FP
[18:53] <michaelni> just note that the mips code itself is maintained by the mips guys so
[18:53] <MrNice06> Hi, I am a newbie don't know exactly how to start my request regarding a bug. Can I now and here?
[18:53] <michaelni> that part is their decission
[18:53] <kurosu> I'm a bit uncomfortable about it, I'm not experienced, and was just looking how they did it
[18:53] <kurosu> yeah of course
[18:55] <MrNice06> OK. I opened a ticket in the track-bug #2367. Can you read it, mainly the second part?
[18:55] <durandal_1707> saste: that transform thing have nothing of interest for me
[18:56] <saste> durandal_1707, how do you plan to provide the convolution matrix?
[18:57] <durandal_1707> saste: binary and/or string
[18:57] <saste> durandal_1707, ok
[18:57] <MrNice06> It seems that this bug appears with other users
[18:57] <saste> i wanted to add a matrix parser, but never found the time
[18:57] <durandal_1707> now is there some code for matrix multiplication that optimized?
[18:57] Action: saste trying to fix the crop bug
[18:58] Action: saste found the bug
[18:58] <ubitux> that was fast
[18:59] <michaelni> MrNice06, for ffmpeg/v4l2 bugs it might make sense to mail Giorgio Vazzana unless he already said something in the bug report (didnt read the ticket as ive no time ATM so dunno)
[19:01] <MrNice06> I have no answer in the ticket so far. Where can I find Giorgio mail?
[19:02] <durandal_1707> lol, this shows how auto insterted filters are evil
[19:03] <michaelni> MrNice06, git log for sure possibly elsewhere too
[19:03] <durandal_1707> but why is crop always inserted in my case, that is just retarded
[19:04] <ubitux> a366beaf6a7cbaa43fef09cabc2248225290208f ?
[19:05] <durandal_1707> my resolutions of files are not odd
[19:05] <kurosu> michaelni, anyway, they want to create a fixedpoint dsp context, but using a predetermined FP format, which may not suit FP implementation for other codecs
[19:05] <ubitux> michaelni: can you reproduce the ffplay -f lavfi testsrc bug? (and almost anything involving filters?)
[19:06] <kurosu> I don't think that's good but I have no idea what to make of this
[19:07] <michaelni> kurosu, i share your feeling on this, it doesnt feel "right"
[19:08] <michaelni> ubitux, "Assertion abs(dst_linesize) >= bytewidth failed at libavutil/imgutils.c:249"
[19:08] <michaelni> but ive to go afk now for 30-60min
[19:08] <kurosu> michaelni, anyway I'd prefer if someone would comment too, because I don't feel like making decisions
[19:09] <michaelni> kurosu, ok ill comment too when iam back, need to leave now
[19:09] <kurosu> no problem
[19:20] <cone-607> ffmpeg.git 03Stefano Sabatini 07master:1d86fe6970fa: lavfi/crop: restore pos constant, and fix "t" variable misplacement in variable array
[19:20] <cone-607> ffmpeg.git 03Stefano Sabatini 07master:9fa3b5b8a899: lavfi/crop: log pos in debug message
[19:20] <saste> ubitux, what's the status with regards to filtering mergemess?
[19:21] <saste> what's still missing?
[19:23] <MrNice06> michaelni: Thanks, I found the mail and will send him my questions.
[19:23] <ubitux> michaelni: regression since 0acf7e268b2f873379cd854b4d5aaba6f9c1f0b5
[19:23] <ubitux> saste: mostly done afaik
[19:23] <ubitux> ...documentation needs some review
[19:50] <ubitux> nice the crop bug :)
[19:51] <durandal_1707> lol, what was this?: http://ffmpeg.org/pipermail/ffmpeg-devel/2009-October/078962.html
[19:54] <saste> durandal_1707, slicify
[19:54] <saste> it was removed less than one year ago
[19:58] <saste> uhm hue regression?
[19:58] <saste> related to copy permission, meat for ubitux
[20:01] <ubitux> isn't hue tested with perms?
[20:03] <saste> ubitux: do we have a hue test?
[20:03] <saste> we should
[20:03] <ubitux> tests/lavfi-regression.sh:do_lavfi "hue"                "perms=random,hue=s=sin(2*PI*t)+1"
[20:03] <ubitux> we should add the propers deps to the lavfi filters btw
[20:04] <ubitux> filters tests*
[20:06] <saste> ubitux, ok still need to check the report
[20:09] <durandal_1707> it assumes linesize[1] == linesize[2]
[20:10] <ubitux> indeed
[20:10] <durandal_1707> but always wonder why would that not be same
[20:11] <cone-607> ffmpeg.git 03Clément BSsch 07master:72ad0d56a135: lavfi/hue: make do while (0) form meaningful.
[20:17] <durandal_1707> hah now it starts adding other filters...
[20:21] <ubitux> something fishy in the second bug he's reporting
[20:21] <durandal_1707> alpha is left uninitialized, how added alpha format?
[20:21] <ubitux> sounds like another auto-inserted filter hell
[20:25] <durandal_1707> i think i fixed one bug
[20:30] <cehoyos> Is anybody able to compile vlc? https://trac.videolan.org/vlc/ticket/8320
[20:34] <cone-607> ffmpeg.git 03Paul B Mahol 07master:047a027d0b55: lavfi/hue: copy alpha plane too
[20:35] <durandal_1707> cehoyos: is that really ffmpeg bug? (iirc vlc use l)
[20:35] <cehoyos> durandal_1707: Can you compile vlc to find out?
[20:36] <durandal_1707> i dont have vlc here
[20:36] <cehoyos> Did your last commit fix ticket 2465?
[20:37] <durandal_1707> only hue part
[20:37] <cehoyos> The other part is not relevant for ticket afaict
[20:37] <durandal_1707> i did not download his samples, (it may be yet another bug)
[20:37] <cehoyos> I will test later.
[20:41] <cone-607> ffmpeg.git 03Michael Niedermayer 07master:e87c1cdbb548: ffplay: replace opaque/params for buffersinks
[20:41] <cone-607> ffmpeg.git 03Michael Niedermayer 07master:9b595e86e342: avdevice/lavfi: add error checking for av_opt_set_int_list()
[20:48] <durandal_1707> ubitux: fixing bug(s) or?
[21:13] <ubitux> durandal_1707: not really
[21:20] <durandal_1707> looks like if filters are combined, worst pix fmt is picked
[21:29] <durandal_1707> funny if i remove scale filter, colors are preserved
[21:31] <langoliers> somebody has a link to the serpent standard? i can not locate it ;/
[21:46] <cone-607> ffmpeg.git 03Paul B Mahol 07master:a47288f6dc4b: lavfi/hue: add support for more alpha formats
[22:27] <cone-607> ffmpeg.git 03Diego Biurrun 07master:a89c05500f68: x86: h264qpel: int --> ptrdiff_t for some line_size parameters
[22:27] <cone-607> ffmpeg.git 03Michael Niedermayer 07master:52bda1d90395: Merge commit 'a89c05500f68d94a0269e68bc522abfd420c5497'
[22:36] <cone-607> ffmpeg.git 03Diego Biurrun 07master:e027032fc6a4: x86: dsputil: ff_h263_*_loop_filter declarations to a more suitable place
[22:36] <cone-607> ffmpeg.git 03Michael Niedermayer 07master:43bf4ee9a974: Merge commit 'e027032fc6a49db5a4ce12fc3e09ffb86ff20522'
[22:41] <ubitux> #include "libavfilter/libmpcodecs/img_format.h"
[22:41] <ubitux> wat
[22:42] <durandal11707> where?
[22:42] <ubitux> x11 patch
[22:42] <durandal11707> yust ignore it
[22:42] <durandal11707> i will just take patch and makt it usefull
[23:11] <cone-607> ffmpeg.git 03Clément BSsch 07master:0bc34ef02e07: doc/utils: use @code markup for opencl function names.
[23:18] <ubitux> btw, what's the reason to prefer a configurable buffersink over inserting a format constraint filter like l. does?
[23:22] <saste> (The effect that the bug produced may be worth a separate filter imo.)
[23:22] <saste> i agree :)
[23:23] <durandal11707> reallz_
[23:28] <michaelni> ubitux, iam in favor or loosing the sink config stuff if its not neccesary and simpler without
[23:31] <ubitux> i'm not comfortable with that stuff, so i let the maintainers (saste & nicolas?) deal with it :p
[23:31] <ubitux> it was just curiosity
[23:31] <saste> michaelni, I agree
[23:31] <ubitux> it seems we have to deal with various related troubles everytime we have a lavfi merge
[23:32] <saste> ubitux, why, don't you like fixing bugs?
[23:33] <cehoyos> Is it known why fate fails some tests with msvc?
[23:34] <ubitux> cehoyos: the ebur128 is a msvc bug; i think nevcairiel reported it
[23:34] <ubitux> for the rest, no idea
[23:34] <ubitux> saste :)
[23:35] <nevcairiel> in mxf the code must be faulty, it casues a crash
[23:35] <nevcairiel> i didnt bother to look into it yet
[23:35] <cehoyos> "msvc bug" as in: A bug in the Microsoft compiler? If yes, is no work-around possible?
[23:36] <cehoyos> Is the change known that causes the mxf crash?
[23:37] <cone-607> ffmpeg.git 03Diego Biurrun 07master:2004c7c8f763: x86: dsputil: cosmetics: Remove two pointless variable indirections
[23:37] <cone-607> ffmpeg.git 03Michael Niedermayer 07master:ed3680bc9b8f: Merge commit '2004c7c8f763280ff3ba675ea21cf25396528fd3'
[23:39] <cehoyos> Assuming it is 25b143a - in which function (line) does it crash?
[23:41] <nevcairiel> if i knew, i would fix it :P
[23:41] <cehoyos> If you knew if the ebur regression is a Microsoft bug, or if you knew in which line mxf crashes?
[23:42] <cehoyos> (Wasn't the reason for msvc that debugging is so much easier now?)
[23:42] <nevcairiel> it is, still means someone has to do it
[23:44] <nevcairiel> i know that its a msvc bug because i debugged it, and then created a small test case and reported the issue to MS, but i didnt have motivation to look into mxf
[23:45] <cehoyos> Iirc, the usual response time from MS is >months, so perhaps a work-around should be committed?
[23:48] <cone-607> ffmpeg.git 03Diego Biurrun 07master:a3cb865310cc: x86: dsputil: Move some ifdefs to avoid unused variable warnings
[23:48] <cone-607> ffmpeg.git 03Michael Niedermayer 07master:34b78ad04fe3: Merge remote-tracking branch 'qatar/master'
[23:53] <saste> michaelni, there is a patch from highgod that you can apply (if you are fine with it)
[00:00] --- Sat Apr 13 2013


More information about the Ffmpeg-devel-irc mailing list