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

[burek]

[00:44:06 CEST] <cone-020> ffmpeg 03Muhammad Faiz 07master:a11757d7cba9: avfilter/showcqt: add csp option
[00:53:46 CEST] <cone-020> ffmpeg 03Muhammad Faiz 07master:7c5fca15bbc6: avfilter/firequalizer: change WFUNC_MIN/MAX to NB_WFUNC
[00:53:47 CEST] <cone-020> ffmpeg 03Muhammad Faiz 07master:01ab60201be9: avfilter/firequalizer: add tukey window
[00:58:37 CEST] <cone-020> ffmpeg 03Muhammad Faiz 07master:8e53c1567c7a: doc/filters: document tukey window on firequalizer
[01:53:53 CEST] <cone-020> ffmpeg 03James Almer 07master:bad8bbc26aa3: avformat/matroskadec: set aspect ratio only when DisplayWidth and DisplayHeight are in pixels
[01:53:54 CEST] <cone-020> ffmpeg 03James Almer 07master:90826031a205: avformat/matroskaenc: write a DisplayUnit element when aspect ratio is unknown
[17:20:11 CEST] <cone-168> ffmpeg 03James Almer 07master:4b0f37dadb74: avcodec/utils: print Chroma Location string in verbose log level
[18:59:03 CEST] <ubitux> is this avpacket fate test really useful for testing anything?
[19:22:07 CEST] <cone-168> ffmpeg 03James Almer 07master:8b59ce034268: Partially revert "avformat/matroskadec: set aspect ratio only when DisplayWidth and DisplayHeight are in pixels"
[19:22:55 CEST] <cone-168> ffmpeg 03Clément BSsch 07master:ad2d2ebd4ee9: ffmpeg: remove unused ist parameter in do_subtitle_out()
[19:23:47 CEST] <ubitux> matroska is getting standardized?
[19:30:48 CEST] <jamrial> yeah, alongside ffv1 and flac it seems
[19:38:35 CEST] <ubitux> nice
[19:54:21 CEST] <ubitux> damn the madness of do_video_out()... with half indented dup loop (to i suppose reduce conflict solving with libav)
[19:55:51 CEST] <wm4> having shit code is part of ffmpeg culture
[19:57:14 CEST] <wm4> (can we change this please)
[19:58:15 CEST] <ubitux> if we don't have to merge libav anymore i can reindent if you want
[19:58:18 CEST] <ubitux> ;)
[19:58:48 CEST] <ubitux> speaking of this, i hope to get done with subtitles asap, it really helps cleaning things up
[19:58:54 CEST] <ubitux> then i'll be available again for merges
[19:59:06 CEST] <ubitux> i hope we won't be 1000+ when i'm available again
[19:59:29 CEST] <ubitux> (423 currently)
[20:00:14 CEST] <ubitux> ah, we're at an avconv merge window
[20:00:20 CEST] <ubitux> annoying stuff
[20:12:23 CEST] <nevcairiel> i was going to mvoe forward but it would break qsv transcoding
[20:12:29 CEST] <nevcairiel> we've been discussing what to do about that
[20:13:32 CEST] <nevcairiel> tried contacting our qsv guy to port the required changes, but afaik jamrial and michaelni both didnt receive any response
[20:16:54 CEST] <ubitux> tell him we trash his work in 3days?
[20:20:30 CEST] <wm4> +1
[20:23:26 CEST] <nevcairiel> the positive thing is that after browsing the upcoming changes, after this avconv batch it should be quite a bit easier for a long w hile
[20:23:57 CEST] <ubitux> i mean, we're not going to put the project into jeopardy just because an external guy doesn't felt like maintaining his niche code
[20:24:15 CEST] <ubitux> so better stress him out and ditch the problem asap
[20:24:56 CEST] <nevcairiel> jamrial suggested reverting to libav state and just porting some of the improvements individually
[20:25:05 CEST] <nevcairiel> not sure if he plans to push that further
[20:28:30 CEST] <ubitux> unrelated question: we always have filters for a/v?
[20:28:37 CEST] <ubitux> (in a transcode process)
[20:31:11 CEST] <ubitux> like, the only place where do_{audio,video}_out is called is in reap_filters()
[20:31:26 CEST] <ubitux> and the only way these functions are called is if there is a filter
[20:34:17 CEST] <nevcairiel> probably yes then? =p
[20:34:24 CEST] <nevcairiel> i dont know the dataflow yet
[20:35:10 CEST] <ubitux> i wonder if i'll need to add a dummy lavfi subtitles support in order to keep things clean in ffmpeg :P
[20:35:11 CEST] <BtbN> there should allways be an auto-inserted scaler and stuff
[20:39:15 CEST] <jamrial> nevcairiel: yeah, since they didn't reply lets just revert things back to libav state so qsv merges become conflict free
[20:39:38 CEST] <jamrial> once they show up again, they can reimplement their custom changes
[20:40:34 CEST] <jamrial> this has blocked further merges long enough
[20:47:30 CEST] <wm4> their boss can pay them extra to redo the work
[20:48:19 CEST] <JEEB> esp. if it's just the MJPEG stuff :V
[20:54:39 CEST] <nevcairiel> mjpeg and some options in the encoder
[20:54:48 CEST] <nevcairiel> and maybe the CC stuff
[21:06:55 CEST] <JEEB> right
[21:07:08 CEST] <JEEB> still probably worth it to just override and re-apply
[22:22:10 CEST] <cone-168> ffmpeg 03Muhammad Faiz 07master:1a9513bfbc60: avfilter/firequalizer: add scale option
[23:25:18 CEST] <wbs> fyi, the patch recently about "support rtsps"; support for rtsps already exists since 2 years or so, this just tweaks it. kinda relevant for the commit naming
[00:00:00 CEST] --- Mon Oct 17 2016