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

[burek]

[00:00] <ubitux> wm4: if you're not looking into it, please open a ticket, i don't have time to look into it currently
[00:01] <wm4> ubitux: yeah, I want to take a closer look at it, it'll end with a ticket or a patch
[00:01] <ubitux> thanks
[00:02] <BBB> nevcairiel: done, see #x264dev for x264 patch
[00:02] <BBB> oops maybe wrong channel, dunno
[00:21] <BBB> ubitux: Daemon404: I'll start doing some loopfilter work meanwhile
[00:21] <ubitux> ok
[00:21] <ubitux> sorry for taking some time :p
[00:24] <BBB> that's fine
[00:24] <wm4> also how can I convince Nicolas George NOT to spam the log with errors when a subtitle is not utf-8
[00:24] <wm4> because it fucking spams the terminal and you can't prevent it
[00:25] <wm4> grep on the log callback output?
[00:25] <ubitux> wm4: come to vdd at the end of the month with a cutter or another random letal weapon
[00:25] <wm4> I think that would get me into prison
[00:26] <wm4> ubitux: found the issue... sending patch
[00:32] <mateo`> is there a list of topics that will be discuted during the vdd ?
[00:32] <ubitux> "Schedule
[00:32] <ubitux> To be defined&"
[00:33] <mateo`> :(
[00:35] <wm4> hm: return FFERRTAG('N', 'I', 'G', 'E');
[00:35] <wm4> I think this creates wrong associations
[00:36] <wm4> then return FFERRTAG('N', 'I', 'C', 'O'); it is
[00:40] <mateo`> I'll try to come to the VDD this year if I can a flight for a resonable price
[00:40] <ubitux> don't forget you need to register
[00:46] <wm4> is it ok to post unrelated patches as one series?
[00:47] <wm4> hm or maybe git-send-email has an option to do that
[01:47] <Daemon404> michaelni, dont merge 869b04e89154cd92d2bcfdabcecbe3217864c099 and pals until my fix goes into libav
[01:47] <Daemon404> they broke all windows builds.
[01:57] <michaelni> Daemon404, if you point me to the fix ill cherry pick it immedeatly after the problematic commits or if possible before them. That way i dont have to wait for some external thing to happen and fewer checkouts are broken
[02:01] <wm4> ubitux: I'm starting to believe what you said is the only thing that would work
[02:03] <Daemon404> michaelni, http://lists.libav.org/pipermail/libav-devel/2013-August/049638.html
[02:03] <Daemon404> it has no been OK'd yet
[02:03] <Daemon404> and may change
[02:03] <Daemon404> not*
[02:03] <Daemon404> it may not even be correct
[02:05] <michaelni> thanks, will cherry pick it when i merge the stuff
[02:33] <Compnn> Daemon404 : who cares about windows ?
[02:33] <Compnn> :P
[02:34] <wm4> most people who use computers
[02:38] <drv> doesn't Compnn run windows 2000 or something? :P
[02:39] <Daemon404> yes he hates life
[02:45] <kierank> xp4eva
[02:49] <cone-488> ffmpeg.git 03Michael Niedermayer 07master:0b5d1b88e09b: avutil/opt: fix types passed to the format string "%s"
[03:02] <cone-488> ffmpeg.git 03Michael Niedermayer 07master:bc721ac9f775: swscale/utils: Fix potential overflow of dstPos*xInc before converting to 64bit
[03:02] <cone-488> ffmpeg.git 03Michael Niedermayer 07master:8d745281a489: swscale/utils: Fix potential overflow of srcPos*C  before converting to 64bit
[03:09] <michaelni> ubitux, did you read / do you have any comments on "76729     0626  1:20 wm4             (1.5K) [FFmpeg-devel] [PATCH] lavc: make invalid UTF-8 in subtitle output a non-fatal error" and
[03:10] <michaelni> 0808  1:04 wm4             (1.3K) [FFmpeg-devel] [PATCH 3/4] libavcodec: return specific error code on subtitles with invalid UTF-8
[03:10] <wm4> Nicolas expressed his dislike over the error code fourcc
[03:10] <wm4> so maybe that should be changed if it's accepted
[03:10] <Daemon404> nicolas is out of touch with reality
[03:17] <BBB> Daemon404: how's inter mode going?
[03:17] <Daemon404> i ended up helping ruggles with something today... started reading libvpx
[03:18] <Daemon404> perhaps a PR toomorrow
[03:20] <llogan> who here is in the US and wants some FFmpeg stickers?
[04:41] <llogan> -pass 512
[07:15] <ubitux> michaelni: i'll try to find some time for it today
[10:07] <zidanne> can't we convert AV_SAMPLE_FMT_S16 to AV_SAMPLE_FMT_S16P with swr_convert? I get BAD_ACCESS crash at "CONV_FUNC(AV_SAMPLE_FMT_S16, int16_t, AV_SAMPLE_FMT_S16,  *(const int16_t*)pi)"
[10:12] <zidanne> (I need this because I want to encode decoded audio (which is AV_SAMPLE_FMT_S16 by request_format) into mp3 with libmp3lame and libmp3lame needs S16P not S16)
[10:33] <michaelni> zidanne, S16->S16P should work fine like any other convertion
[10:34] <zidanne> I use this line which should be familiar.. I think I'm doing something wrong somewhere else& swr_convert(resampleCtx, &resampledOut, AVCODEC_MAX_AUDIO_FRAME_SIZE,(const uint8_t**)decoded_frame.extended_data,decoded_frame.nb_samples);
[10:48] <cone-434> ffmpeg.git 03Martin Storsjö 07master:f4d371b9737c: rtsp: Don't include the listen flag in the SDP demuxer flags
[10:48] <cone-434> ffmpeg.git 03Michael Niedermayer 07master:eb02f0c6e833: Merge commit 'f4d371b9737c0405b3bc46d7ca0c856c0a8616b1'
[10:51] <zidanne> michaelni: swr_init(resampleCtx) is returning error. The line I use is: SwrContext* resampleCtx = swr_alloc_set_opts(NULL,av_get_default_channel_layout(pCodecCtx->channels), AV_SAMPLE_FMT_S16P, pCodecCtx->sample_rate,av_get_default_channel_layout(pCodecCtx->channels), pCodecCtx->sample_fmt,pCodecCtx->sample_rate,0, 0);
[10:51] <zidanne> Maybe swr_convert really does not want S16 as the input?
[11:07] <cone-434> ffmpeg.git 03Vittorio Giovara 07master:22c879057ead: mpegvideo_enc: drop outdated copy_picture_attributes() in favour of a modern av_frame_copy_props()
[11:07] <cone-434> ffmpeg.git 03Michael Niedermayer 07master:1b2a5817fcd6: Merge commit '22c879057ead189c0f59241cb9eeb926381e3299'
[11:15] <cone-434> ffmpeg.git 03Rémi Denis-Courmont 07master:869b04e89154: libavutil: add avpriv_open() to open files with close-on-exec flag
[11:15] <cone-434> ffmpeg.git 03Michael Niedermayer 07master:5f38317e59bd: Merge commit '869b04e89154cd92d2bcfdabcecbe3217864c099'
[11:15] <cone-434> ffmpeg.git 03Derek Buitenhuis 07master:87e8cbf70931: libavutil: Don't use fcntl if the function does not exist
[11:24] <cone-434> ffmpeg.git 03Rémi Denis-Courmont 07master:880391ed2d2f: libavutil: use avpriv_open()
[11:24] <cone-434> ffmpeg.git 03Michael Niedermayer 07master:579a13789799: Merge commit '880391ed2d2faf796ca3a16f63cec69767546a21'
[11:47] <cone-434> ffmpeg.git 03Rémi Denis-Courmont 07master:fee9db1fdf92: libavcodec: use avpriv_open()
[11:47] <cone-434> ffmpeg.git 03Rémi Denis-Courmont 07master:71bf6b41d974: libavdevice: use avpriv_open()
[11:47] <cone-434> ffmpeg.git 03Michael Niedermayer 07master:eec9c7af3370: Merge commit 'fee9db1fdf921295233e94cbe2769f9cd722206d'
[11:47] <cone-434> ffmpeg.git 03Michael Niedermayer 07master:95fa1fe437c7: Merge commit '71bf6b41d974229a06921806c333ce98566a5d8a'
[11:53] <cone-434> ffmpeg.git 03Rémi Denis-Courmont 07master:51eb213d0015: libavformat: use avpriv_open()
[11:54] <cone-434> ffmpeg.git 03Michael Niedermayer 07master:756f865e3b66: Merge commit '51eb213d00154b8e7856c7667ea62db8b0f663d4'
[12:03] <cone-434> ffmpeg.git 03Diogo Franco 07master:e8edf4e1cf60: cmdutils: Only do the windows-specific commandline parsing on _WIN32
[12:03] <cone-434> ffmpeg.git 03Derek Buitenhuis 07master:0f1fb6c0194c: libavutil: Don't use fcntl if the function does not exist
[12:03] <cone-434> ffmpeg.git 03Michael Niedermayer 07master:f09b5fb4afed: Merge commit '0f1fb6c0194c85483dedb93b20a5b76f6fc9d520'
[12:09] <cone-434> ffmpeg.git 03Ben Avison 07master:5afe1d27912b: avio: Add const qualifiers to ffio_read_indirect
[12:09] <cone-434> ffmpeg.git 03Michael Niedermayer 07master:a0fb6083967a: Merge commit '5afe1d27912be9b643ffb4ddc21f6d920260dbb0'
[12:26] <cone-434> ffmpeg.git 03Kostya Shishkov 07master:3e5898782dce: Voxware MetaSound decoder
[12:27] <cone-434> ffmpeg.git 03Michael Niedermayer 07master:69ea65b46f1a: Merge commit '3e5898782dce60334ab294821ca00b19c648cf66'
[12:36] <cone-434> ffmpeg.git 03Ben Avison 07master:7a82022ee2f9: h264_parser: Initialize the h264dsp context in the parser as well
[12:36] <cone-434> ffmpeg.git 03Michael Niedermayer 07master:50b7ce12574d: Merge commit '7a82022ee2f9b1fad991ace0936901e7419444be'
[12:51] <cone-434> ffmpeg.git 03Ben Avison 07master:218d6844b37d: h264dsp: Factorize code into a new function, h264_find_start_code_candidate
[12:51] <cone-434> ffmpeg.git 03Michael Niedermayer 07master:c0f2ad3dbdea: Merge commit '218d6844b37d339ffbf2044ad07d8be7767e2734'
[12:57] <cone-434> ffmpeg.git 03Ben Avison 07master:45e10e5c8d3d: arm: Add assembly version of h264_find_start_code_candidate
[12:57] <cone-434> ffmpeg.git 03Michael Niedermayer 07master:e1a5ee25e048: Merge remote-tracking branch 'qatar/master'
[15:14] <cone-434> ffmpeg.git 03Michael Niedermayer 07master:9386f334af53: avcodec/bitstream: Dont try to free buffers for static VLCs
[15:57] Action: pippin updates his test gif of his position stable dither - now better adapted to RGB332 ; and with a less prominent pattern http://pippin.gimp.org/dither/
[16:38] <zidanne> yo
[16:44] <zidanne> michaelni: sorry I couldn't connect after my connection is gone, did you write anything about my issue that is related to swr_init() returning error?
[16:45] <michaelni> zidanne, did you look at swr_init() ? / where does it fail exactly ?
[16:46] <cone-434> ffmpeg.git 03Matthieu Bouron 07master:88a1ff22336e: MAINTAINERS: add myself as maintainer for lavf/aiff* and lavf/movenc.c
[16:46] <cone-434> ffmpeg.git 03Michael Niedermayer 07master:55a88daf6ff1: MAINTAINERS: remove myself from movenc, 2 maintainers should be enough
[18:04] <ubitux> BBB: totally untested (except compilation) and with personal comments: https://github.com/ubitux/FFmpeg/commit/7801a2ee4f3e9cc9cbe70a3cea570a331bbd9ad5
[18:06] <mateo`> ubitux: funny
[18:08] <nevcairiel> why does that look like disassembled code
[18:09] <ubitux> :)
[18:12] <ubitux> nevcairiel: to ease the port to assembly of course
[18:20] <ubitux> BBB: i hope there is no idct64+ though now :p
[19:01] <membrane> Hi. I've created an blend filter which allows me to remove transparent logos based on a mask. It is simple but it works very well. Is it ok to submit this as an blend filter or should it be in an own module? Is it too trivial?
[19:07] <ubitux> what's the difference with removelogo?
[19:11] <membrane> it keeps the remaining information
[19:11] <ubitux> can't you update that filter to add the missing bit instead?
[19:12] <ubitux> we already have 2 filters to remove logo
[19:12] <membrane> i'm not a real programmer but i can try
[19:13] <membrane> this is how it looks like: http://www.xbmcnerds.com/index.php?page=Attachment&attachmentID=2946
[19:13] <ubitux> "Der Zutritt zu dieser Seite ist Ihnen leider verwehrt. Sie besitzen nicht die notwendigen Zugriffsrechte, um diese Seite aufrufen zu können."
[19:14] <ubitux> i don't know what that means, but it's in red and there is nothing else
[19:14] <membrane> give me a minute, i will upload it at imageshack
[19:15] <membrane> you have to be registered to view uploads in this forum. i did'nt think of that - doh
[19:17] <ubitux> i didn't understand the "it keeps the remaining information" btw
[19:17] <membrane> this link works http://imageshack.us/photo/my-images/441/nzf9.png/
[19:18] <membrane> there is some colour information left in transparent logos
[19:23] <membrane> the colour range in this example spans from 127 to 255. all i have to to is to stretch that to the normal range
[19:25] <membrane> here is an image without those artifacts: http://imageshack.us/photo/my-images/197/8dgy.jpg/
[19:25] <mateo`> membrane: are the results better than the current filters ?
[19:30] <membrane> definitely. removelogo and delogo guess the colour information based on the sourrounding pixels. my attempt is to remove the logo without destroying the infomation underneath
[19:34] <membrane> the equation is: ((A - B) * 255) / (255 - B)
[19:35] <membrane> A is the original pixel, B is the mask
[19:37] <thegeek> looks like it works really well
[19:37] <thegeek> would be very nice to have as a filter
[19:37] <Daemon404> delogos never work "well"
[19:37] <Daemon404> as a viewer, the blurry patch is never gone, and it's always annoying
[19:38] <thegeek> this case seems to deal with transparent logo overlays
[19:38] <thegeek> in which case there is obviously a lot of information to recover
[19:38] <Daemon404> It's Never Perfect(TM)
[19:39] <thegeek> :)
[19:39] <hotwings> just grabbed the latest ffmpeg git and now i get no video (mpeg2 or h264) using vdpau. if i revert to ffmpeg.20130730.git.d4925272, video works again. audio works fine however
[19:40] <Daemon404> there were recent(ish) changes
[19:40] <Daemon404> to its API, iirc
[19:40] <Daemon404> what are you using to view?
[19:40] <hotwings> im using vdr with the softhddevice plugin to watch tv
[19:41] <Daemon404> well.. i have no knowledge of that, so im not of much help
[19:41] <hotwings> no prob. if there was an api change in the last 8 days, thats likely the problem i think
[19:42] <Daemon404> 3 days ago
[19:42] <Daemon404> http://git.videolan.org/?p=ffmpeg.git;a=search;s=R%C3%A9mi+Denis-Courmont;st=author
[19:45] <membrane> i think it is better than nothing. the loss of colour information is about 12% in my example. the logo is far less noticable than without the correction.
[19:45] <hotwings> thanks Daemon404
[19:45] <thegeek> indeed, I guess the color information loss would be highly dependent on the logo overlay though
[19:45] <Daemon404> might be 2852740e23f91d6775714d7cc29b9a73e1111ce0
[19:46] <thegeek> just 12% is excellent, most of the tv broadcasts I've seen used transparent logo overlays so it would be much better to use your kind of filter
[19:47] <thegeek> defining the logo/position etc should be fairly straightforward, would be a really nice option for e.g. xbmc + pvr
[19:48] <Daemon404> in theory it should be possible to build a mask after N frames
[19:48] <membrane> it is possible to create this masks with an other filter
[19:48] <Daemon404> just by examinng luma SAD
[19:48] <Daemon404> or something
[19:48] <thegeek> ah nice
[19:48] <membrane> just cache the lowest value and use the equation
[19:54] <membrane> there could be an filter which detects the present of  logos
[20:02] <Daemon404> those exist in e.g. MythTV
[20:02] <Daemon404> it uses logos and black screens for automatic commercial cutting
[20:04] <JEEB> Daemon404, I really stopped believing in that way when some shows started showing the logos a few seconds in :D
[20:06] <Daemon404> its not ok on its own
[20:06] <Daemon404> also it failed miserably for 24
[20:06] <Daemon404> because of its clock-on-black-screen
[20:06] <Daemon404> sc metrics + black screen + nologo = failed
[20:08] <JEEB> :D
[20:08] <JEEB> at least looking at Japanese TV there should be ~0.6 seconds of complete silence (!) between ads and the shows
[20:09] <JEEB> that could be used as a hint together with the logos
[20:12] <kierank> there are also scte 104 messages
[20:12] <kierank> or scte 35 i forget which spec hits the consumer
[20:12] <kierank> Daemon404: some of the delogos are superb
[20:12] <kierank> the ones that use motion compensation
[20:13] <Datalink-Studio> http://pastebin.com/cr4rDgSU OK, I'm attempting to use a Raspberry Pi to connect to one of the streams for the PEG station I volunteer with, I have been trying for a couple weeks, and I've been banging my head against this input/output error for a while without an idea what's causing it.
[20:13] <Daemon404> kierank, theyre never 100% perfect
[20:13] <kierank> Datalink-Studio: raspberry is far too slow
[20:13] <Compnn> can you connect to it using a regular puter Datalink ?
[20:13] <Compnn> er Datalink-Studio
[20:14] <Compnn> erm 
[20:14] <Compnn> have you tried that ffmpeg line on a regular puter, i mean :)
[20:17] <Datalink-Studio> I've gotta get a source, the Pi's handled it otherwise, just not liking going to rtmp
[20:17] <Datalink-Studio> yeah, I need a source
[20:27] <Compnn> what
[20:27] <Compnn> does it work on your pc?
[20:30] <Datalink-Studio> I don't have a video source at the moment, sorry, had other stuff come up related to a different project
[20:57] <Compnn> eh
[22:15] <cone-434> ffmpeg.git 03Reimar Döffinger 07master:a48979d71559: Reduce MAKE_ACCESSORS code duplication via a new header.
[23:34] <Daemon404> Kovensky, yours ^ ?
[23:43] <Kovensky> nope
[23:43] <Kovensky> that looks completely bananas -- cygwin64 requires gcc 4.8
[23:43] <Daemon404> orite
[00:00] --- Fri Aug  9 2013