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

[burek]

[00:01:51 CEST] <JEEB> jamrial: < thresh> I'm going to find some courage and upgrade Albiero (which hosts git, mail, dns) to jessie tonight.
[00:01:56 CEST] <JEEB> from #videolan
[00:02:20 CEST] <JEEB> and then just a few moments ago < thresh> rebooting
[00:02:32 CEST] <JEEB> so I would expect there to be a sitrep soon on how it either succeeded or failed
[09:03:13 CEST] <bp0> michaelni, I have a patch to fix #5727 in af_hdcd, but it is kindof big as the functions had to be remade to process both channels together.
[09:06:53 CEST] <bp0> I submitted an unrelated set yesterday, but they are not yet applied. Should I submit the new patch against master or after those patches?
[10:33:01 CEST] <ubitux> OT but vlc doesn't support ffmpeg>2.8?
[10:33:40 CEST] <funman> ubitux: why not?
[10:33:57 CEST] <ubitux> dunno, it seems the only package that requires ffmpeg 2.8 on my machine is vlc
[10:34:08 CEST] <ubitux> maybe some packager madness but that's curious
[10:34:46 CEST] <funman> maybe vlc stable wasn't updated to some new API then
[10:34:49 CEST] <funman> git it from git :)
[10:34:51 CEST] <funman> get*
[10:35:10 CEST] <ubitux> (http://sprunge.us/BBJA)
[10:35:44 CEST] <ubitux> funman: i don't actually use vlc, it was pulled when i tried some phonon qt4 stuff
[10:50:40 CEST] <ubitux> server is down?
[10:50:45 CEST] <ubitux> ssh: connect to host source.ffmpeg.org port 22: Network is unreachable
[10:51:21 CEST] <ubitux> (git pull)
[10:52:45 CEST] <funman> yes
[10:53:54 CEST] <ubitux> :(
[10:54:13 CEST] <ubitux> Raz-: ping?
[10:54:51 CEST] <nevcairiel> the videolan server is down where git is hosted, not one of ours
[10:55:11 CEST] <nevcairiel> unless git moved and noone told me
[10:56:38 CEST] <ubitux> ah, my bad then
[10:56:58 CEST] <nevcairiel> apparently it went through upgrades yesterday, presumably follow-up maintenance
[10:57:01 CEST] <nevcairiel> or everything died
[10:57:03 CEST] <nevcairiel> who knows!
[10:57:12 CEST] <andrey_turkin> who cares, it's git )
[11:26:26 CEST] <JEEB> nevcairiel: yeah it failed :/
[11:26:33 CEST] <JEEB> they're trying to fix it as we speak
[14:26:28 CEST] <funman> kierank: what do you think of extending swresample to take fractional samplerate, then adding speexdsp to libswresample (like is done for soxr)
[14:26:46 CEST] <kierank> don't see the point
[14:26:57 CEST] <kierank> I'd rather use speexdsp directly
[14:27:23 CEST] <funman> less upipe code to write ^_^
[14:27:34 CEST] <kierank> well you'd have to write the drift code anyway
[14:27:39 CEST] <funman> true
[16:22:21 CEST] <Raz-> ubitux, pong 
[16:22:32 CEST] <Raz-> big delays today :-)
[16:22:43 CEST] <ubitux> Raz-: i thought it was an ffmpeg.org issue, but it was actually videolan
[16:22:47 CEST] <ubitux> sorry for the hilight :p
[16:50:46 CEST] <ubitux> videolan seems up again
[16:59:01 CEST] <jamrial> yeah
[17:06:27 CEST] <BBB> hey guys, is there still a reason that allocating a new AVFrame zeroes the contents?
[17:06:30 CEST] <BBB> can we get rid of that?
[17:20:13 CEST] <durandal_1707> BBB: nope
[17:20:30 CEST] <BBB> nope, no reason? or nope, we cannot get rid of it?
[17:25:14 CEST] <jamrial> BBB: av_frame_alloc() sets default values after zeroing the whole thing. isn't that the correct thing to do?
[17:26:59 CEST] <BBB> isnt it the job of the decoder to fill frame contents instead of av_frame_alloc?
[17:27:56 CEST] <nevcairiel> it gives you one with every field properly initialized, which isnt necessarily zero for all fields, and not all decoders write every single field
[17:28:02 CEST] <nevcairiel> not sure whats wrong with that
[17:32:06 CEST] <cone-568> ffmpeg 03James Almer 07master:26cb7232c343: Revert "lavc: always build dnxhddata"
[17:32:06 CEST] <cone-568> ffmpeg 03James Almer 07master:0c75bd8e3cd7: avcodec/dnxhddata: move avpriv_dnxhd_parse_header_prefix to a header
[17:33:10 CEST] <jamrial> oh nice, gitweb was updated
[17:33:35 CEST] <jamrial> this version hightlights changes to each line
[17:38:46 CEST] <nevcairiel> it was probably running on the version that shipped with debian for ever
[18:09:13 CEST] <michaelni> jamrial, speaking of gitweb versions, is the version on our box (https://git.ffmpeg.org/ffmpeg.git) new enough ? iam asking as mplayer&rtmpdump and any other stuff have probably only that
[18:10:11 CEST] <ubitux> we have word highlight apparently so i guess that's fine
[18:10:42 CEST] <ubitux> i wonder if they handle word highlights with splits (multiple matches per line)
[18:44:11 CEST] <jamrial> michaelni: as ubitux it has word highlight so it's new enough
[18:44:30 CEST] <jamrial> michaelni: you may want to disable the snapshot functionality if bandwidth is a concern, though
[18:44:50 CEST] <jamrial> it's disabled in git.videloan.org
[18:48:36 CEST] <michaelni> i dont think its a problem, it wasnt one till now
[18:49:58 CEST] <jamrial> alright
[18:52:41 CEST] <cone-568> ffmpeg 03James Almer 07release/2.5:5141a0ceafd3: avformat/oggparsevp8: fix pts calculation on pages ending with an invisible frame
[18:52:42 CEST] <cone-568> ffmpeg 03James Almer 07release/2.6:ebd4d7189c2a: avformat/oggparsevp8: fix pts calculation on pages ending with an invisible frame
[18:52:43 CEST] <cone-568> ffmpeg 03James Almer 07release/2.7:470bdd397f2a: avformat/oggparsevp8: fix pts calculation on pages ending with an invisible frame
[18:52:44 CEST] <cone-568> ffmpeg 03James Almer 07release/2.8:e9b7ae08d621: avformat/oggparsevp8: fix pts calculation on pages ending with an invisible frame
[18:52:45 CEST] <cone-568> ffmpeg 03James Almer 07release/3.0:21c36d83f850: avformat/oggparsevp8: fix pts calculation on pages ending with an invisible frame
[18:52:46 CEST] <cone-568> ffmpeg 03James Almer 07release/3.1:7cab4142c541: avformat/oggparsevp8: fix pts calculation on pages ending with an invisible frame
[18:53:09 CEST] <jamrial> mmh, just got a bunch of "remote: fatal: unknown change class '|' in --diff-filter=A|C|M|R" error when pushing that
[18:53:38 CEST] <jamrial> four per branch
[18:57:42 CEST] <jamrial> something may have gone wrong when updating git and/or gitweb in videolan's box it seems
[18:58:09 CEST] <jamrial> look at the history of almost every file in the tree. it will show every single merge commit, even if they didn't touch the file in question
[18:58:42 CEST] <JEEB> jamrial: I recommend poking thresh @ #videolan
[19:00:21 CEST] <jamrial> although the same seems to be happening in https://git.ffmpeg.org/ffmpeg.git, so not sure
[19:23:12 CEST] <BBB> nevcairiel: Id like the data to be uninitialized (the buffer contents, as in, the pixel data)
[19:23:23 CEST] <BBB> nevcairiel: since the decoder will decoder pixels
[19:23:34 CEST] <BBB> nevcairiel: its about 0.2% of runtime so its a slight speedup
[19:23:43 CEST] <BBB> nevcairiel: (for all decoders)
[19:24:30 CEST] <nevcairiel> av_frame_alloc creates an empty frame, it doesnt allocate buffers or init them
[19:25:03 CEST] <nevcairiel> it just fills AVFrame, the struct
[19:26:27 CEST] <nevcairiel> if that takes 0.2% of all time then i would be surprised, but its also not something we should change, a freshly allocated struct should not contain uninitialized data hoping someone fills it
[19:27:47 CEST] <BBB> something in vp9.c is calling av_buffer_allocz
[19:27:53 CEST] <BBB> maybe its the segmentation buffer?
[19:28:30 CEST] <nevcairiel> well buffer allocation is something else
[19:28:52 CEST] <nevcairiel> normal frame buffers are not zeroed from what i can tell
[19:42:39 CEST] <cone-568> ffmpeg 03Kacper MichajBow 07master:e947b75b1c76: libavformat/rtpdec_asf: zero initialize the AVIOContext struct
[19:42:40 CEST] <cone-568> ffmpeg 03Josh de Kock 07master:384251daffb9: lavd/libdc1394: distinguish between enumeration errors and no cameras found
[19:45:21 CEST] <jamrial> michaelni: did you get the errors i mentioned above when pushing that?
[19:45:41 CEST] <michaelni> jamrial, yes
[19:45:50 CEST] <jamrial> ok, thanks
[19:45:53 CEST] <michaelni> "remote: fatal: unknown change class '|' in --diff-filter=A|C|M|R"
[19:46:08 CEST] <jamrial> already mentioned it in #videolan, but thresh seems to be away
[19:49:08 CEST] <omerjerk> I'm adding a new muxer
[19:49:46 CEST] <omerjerk> I added a new entry in allformats.c
[19:49:54 CEST] <omerjerk> but getting error - error: use of undeclared identifier 'CONFIG_ALS_MUXER'
[19:50:05 CEST] <omerjerk> where do I need to define it?
[19:50:39 CEST] <jamrial> you need to rerun configure after adding the entry in allformats.c
[19:51:38 CEST] <omerjerk> oh okay
[20:07:11 CEST] <bp0> michaelni, if I have a new patch to submit to the ML, but already have some un-applied patches waiting there, should I make the new patch against master or after the other ones?
[20:08:24 CEST] <Chloe> bp0: if the new patch doesn't require the other patches then do it against master imo
[20:08:36 CEST] <michaelni> bp0 is there a overlap between the patches ? if not then it wouldnt matter
[20:09:01 CEST] <bp0> there is an overlap
[20:09:32 CEST] <bp0> they are all for the same file, af_hdcd.c
[20:09:54 CEST] <nevcairiel> well not everything touches the same things in those files
[20:15:23 CEST] <bp0> ok, hmm
[20:15:50 CEST] <bp0> well, I will submit and see, I can always resubmit
[21:27:47 CEST] <BBB> michaelni: weve had this discussion about ME for encoding and true motion before, you know Im not a dummy and I fundamentally disagree
[21:28:03 CEST] <BBB> michaelni: I dont think we (ffmpeg, as a project) should have multiple ME stacks in our tree
[21:28:24 CEST] <BBB> its one thing to play around for academics but that code should never be merged as-is
[21:28:40 CEST] <BBB> michaelni: Im just trying to set my expectations so that you dont go down a rabbit hole and end up wandering why I objected
[21:29:21 CEST] <ubitux> (down a Big Buck Bunny hole)
[21:29:23 CEST] <ubitux> (sorry)
[21:30:55 CEST] <BBB> :)
[21:31:08 CEST] <michaelni> BBB, ill resign as mentor and leave this to you 
[21:31:17 CEST] <michaelni> in fact iam not mentor of this one
[21:31:22 CEST] <BBB> who is mentor?
[21:36:28 CEST] <michaelni> paul is
[21:37:01 CEST] <michaelni> BBB, also i think you should not fight this fight on the stuents back
[21:37:28 CEST] <BBB> possibly
[21:37:56 CEST] <BBB> can we at least agree to not make ffmpeg.git a dumping ground for all code ever written by people competent or not?
[21:42:35 CEST] <durandal_1707> what's issue?
[21:47:22 CEST] <iive> clean design
[21:48:07 CEST] <durandal_1707> everything is local to lavfi
[21:53:05 CEST] <durandal_1707> so I don't get it
[22:40:13 CEST] <cone-568> ffmpeg 03James Almer 07master:fb91850fe11f: avutil/frame: access avframe fields directly in get_frame_defaults()
[00:00:00 CEST] --- Tue Jul 26 2016