michaelni changed the topic of #ffmpeg-devel to: Welcome to the FFmpeg development channel | Questions about using FFmpeg or developing with libav* libs should be asked in #ffmpeg | This channel is publicly logged | FFmpeg 7.1.1 has been released! | Please read ffmpeg.org/developer.html#Code-of-conduct
mkver has quit [Ping timeout: 260 seconds]
TheVibeCoder has quit [Ping timeout: 260 seconds]
TheVibeCoder has joined #ffmpeg-devel
<fjlogger> [FFmpeg/FFmpeg] New comment on pull request #20168 avcodec/apv_decode: Check that tiles are within the frame (https://code.ffmpeg.org/FFmpeg/FFmpeg/pulls/20168#issuecomment-2323) by j⁠amrial
<fjlogger> [FFmpeg/FFmpeg] New comment on pull request #20168 avcodec/apv_decode: Check that tiles are within the frame (https://code.ffmpeg.org/FFmpeg/FFmpeg/pulls/20168#issuecomment-2324) by m⁠ichaelni
<fjlogger> [FFmpeg/FFmpeg] New comment on pull request #20168 avcodec/apv_decode: Check that tiles are within the frame (https://code.ffmpeg.org/FFmpeg/FFmpeg/pulls/20168#issuecomment-2325) by j⁠amrial
<philipl> BtbN: https://github.com/elFarto/nvidia-vaapi-driver/issues/74#issuecomment-3165300189 brute-force work-around for the nvdec cuda p2 power usage behaviour
jamrial has quit []
<fjlogger> [FFmpeg/FFmpeg] Pull request #20170 opened: vulkan_encode: Remove unneeded VkImageUsageFlags (https://code.ffmpeg.org/FFmpeg/FFmpeg/pulls/20170) by b⁠cheng
Guest65 has joined #ffmpeg-devel
Guest65 has quit [Client Quit]
<fjlogger> [FFmpeg/FFmpeg] New comment on pull request #20170 vulkan_encode: Remove unneeded VkImageUsageFlags (https://code.ffmpeg.org/FFmpeg/FFmpeg/pulls/20170#issuecomment-2329) by L⁠ynne
<fjlogger> [FFmpeg/FFmpeg] New comment on pull request #20170 vulkan_encode: Remove unneeded VkImageUsageFlags (https://code.ffmpeg.org/FFmpeg/FFmpeg/pulls/20170#issuecomment-2331) by b⁠cheng
<fjlogger> [FFmpeg/FFmpeg] New comment on pull request #20028 swscale: Implement neon assembly for yuv2nv12cx and yuv2planeX_10 (https://code.ffmpeg.org/FFmpeg/FFmpeg/pulls/20028#issuecomment-2332) by d⁠ashsantosh-mcw
grillo_0 has quit [Server closed connection]
grillo_0 has joined #ffmpeg-devel
<fjlogger> [FFmpeg/FFmpeg] New comment on issue #20034 Feature Request in FLV (Flash Video) Format (https://code.ffmpeg.org/FFmpeg/FFmpeg/issues/20034#issuecomment-2333) by e⁠xekutive
khrbtxyz has quit [Server closed connection]
khrbtxyz has joined #ffmpeg-devel
<JEEB> /33
<JEEB> whoops
GewoonLeon has quit [Ping timeout: 272 seconds]
HarshK23 has quit [Quit: Connection closed for inactivity]
_whitelogger has joined #ffmpeg-devel
<TheVibeCoder> how to clone AVIOContext * so if it is used in such a way that when seeiking within such context one get always fixed +offset from original context ?
<fjlogger> [FFmpeg/FFmpeg] New comment on issue #20171 Rebasing Security, Supply chain attacks (https://code.ffmpeg.org/FFmpeg/FFmpeg/issues/20171#issuecomment-2335) by p⁠rogrammerjake
<kierank> TheVibeCoder: yes
<TheVibeCoder> kierank: the format container have inside multiple other formats, concat-ed together
<fjlogger> [FFmpeg/FFmpeg] New comment on pull request #20164 avfilter/vf_libplacebo: various frame blending improvements (https://code.ffmpeg.org/FFmpeg/FFmpeg/pulls/20164#issuecomment-2337) by h⁠aasn
Kei_N_ has joined #ffmpeg-devel
Kei_N has quit [Read error: Connection reset by peer]
sudden has quit [Server closed connection]
<fjlogger> [FFmpeg/FFmpeg] New comment on pull request #20117 Add a Vulkan VP9 hwaccel (https://code.ffmpeg.org/FFmpeg/FFmpeg/pulls/20117#issuecomment-2340) by L⁠ynne
sudden has joined #ffmpeg-devel
<fjlogger> [FFmpeg/FFmpeg] New comment on issue #20171 Rebasing Security, Supply chain attacks (https://code.ffmpeg.org/FFmpeg/FFmpeg/issues/20171#issuecomment-2341) by k⁠asper93
GewoonLeon has joined #ffmpeg-devel
<fjlogger> [FFmpeg/FFmpeg] Pull request #20117 merged: Add a Vulkan VP9 hwaccel (https://code.ffmpeg.org/FFmpeg/FFmpeg/pulls/20117) by L⁠ynne
<JEEB> Lynne: btw have you seen AMD's vulkan driver randomly deciding to allocate "shared" memory instead of actual VRAM, even if looking at the stats it seems like things would fit?
<Lynne> this is solved by the mechanism to increase the visible VRAM addressing, right?
<JEEB> no idea, I may look into the code of this thing later
<JEEB> just wondered if you had seen something like that, esp. with compute
<Lynne> not really? hard to notice on 24GiB of VRAM
<JEEB> I'm at 10.9 or 11 gig VRAM usage, out of 16 (80MiB reserved), dedicated VRAM usage goes from 10.9 to 9.5. shared VRAM usage goes from 0.4 to 2.2. previous runs of that app had allocated dedicated memory fine, but then it went pröööt
<JEEB> yea, 9070 XT on my side :)
<Lynne> oh, nice
<Lynne> that one's got jkqxz's work running on it
<fjlogger> [FFmpeg/FFmpeg] New comment on pull request #20116 avformat/bdmvdec: add BDMV demuxer, powered by libbluray (https://code.ffmpeg.org/FFmpeg/FFmpeg/pulls/20116#issuecomment-2344) by L⁠ynne
mkver has joined #ffmpeg-devel
<fjlogger> [FFmpeg/FFmpeg] Pull request #20170 merged: vulkan_encode: Remove unneeded VkImageUsageFlags (https://code.ffmpeg.org/FFmpeg/FFmpeg/pulls/20170) by b⁠cheng
<fjlogger> [FFmpeg/FFmpeg] New comment on pull request #20024 rewrite/add mcc [de]muxer, add AV_CODEC_ID_SMPTE_436M_ANC with mxf support, and add conversions <-> eia-608/708 (https://code.ffmpeg.org/FFmpeg/FFmpeg/pulls/20024#issuecomment-2347) by p⁠rogrammerjake
j45 has quit [Ping timeout: 248 seconds]
j45 has joined #ffmpeg-devel
j45 has quit [Changing host]
j45 has joined #ffmpeg-devel
<fjlogger> [FFmpeg/FFmpeg] New comment on pull request #20115 forgejo/pre-commit/config: ensure .forgejo/pre-commit/ignored-words.txt is sorted (https://code.ffmpeg.org/FFmpeg/FFmpeg/pulls/20115#issuecomment-2354) by p⁠rogrammerjake
<fjlogger> [FFmpeg/FFmpeg] New comment on pull request #20168 avcodec/apv_decode: Check that tiles are within the frame (https://code.ffmpeg.org/FFmpeg/FFmpeg/pulls/20168#issuecomment-2356) by m⁠ichaelni
<fjlogger> [FFmpeg/FFmpeg] New comment on pull request #20024 rewrite/add mcc [de]muxer, add AV_CODEC_ID_SMPTE_436M_ANC with mxf support, and add conversions <-> eia-608/708 (https://code.ffmpeg.org/FFmpeg/FFmpeg/pulls/20024#issuecomment-2359) by p⁠rogrammerjake
<fjlogger> [FFmpeg/FFmpeg] New comment on pull request #20024 rewrite/add mcc [de]muxer, add AV_CODEC_ID_SMPTE_436M_ANC with mxf support, and add conversions <-> eia-608/708 (https://code.ffmpeg.org/FFmpeg/FFmpeg/pulls/20024#issuecomment-2364) by p⁠rogrammerjake
<fjlogger> [FFmpeg/FFmpeg] Pull request #20115 merged: forgejo/pre-commit/config: ensure .forgejo/pre-commit/ignored-words.txt is sorted (https://code.ffmpeg.org/FFmpeg/FFmpeg/pulls/20115) by L⁠ynne
<fjlogger> [FFmpeg/FFmpeg] Pull request #20115 merged: forgejo/pre-commit/config: ensure .forgejo/pre-commit/ignored-words.txt is sorted (https://code.ffmpeg.org/FFmpeg/FFmpeg/pulls/20115) by L⁠ynne
<fjlogger> [FFmpeg/FFmpeg] Pull request #20024 merged: rewrite/add mcc [de]muxer, add AV_CODEC_ID_SMPTE_436M_ANC with mxf support, and add conversions <-> eia-608/708 (https://code.ffmpeg.org/FFmpeg/FFmpeg/pulls/20024) by L⁠ynne
j45_ has joined #ffmpeg-devel
j45 has quit [Ping timeout: 260 seconds]
j45_ is now known as j45
j45 has quit [Changing host]
j45 has joined #ffmpeg-devel
<fjlogger> [FFmpeg/FFmpeg] New comment on pull request #20155 avutil/hwcontext_vulkan: add include for boolean literals (https://code.ffmpeg.org/FFmpeg/FFmpeg/pulls/20155#issuecomment-2367) by L⁠ynne
<Lynne> moronix didn't even mention prores raw
<TheVibeCoder> you are obsolete
<Lynne> you believe anthropomorphic reptilian alien crocodiles live on antarctica
<Lynne> and I just discovered a brand new sentence, how's that for a friday night
Kei_N has joined #ffmpeg-devel
Kei_N_ has quit [Ping timeout: 240 seconds]
Kei_N has quit [Read error: Connection reset by peer]
Kei_N has joined #ffmpeg-devel
<fjlogger> [FFmpeg/FFmpeg] Pull request #20172 opened: vulkan_encode_av1: Fix uninitialized return value in write_extra_headers (https://code.ffmpeg.org/FFmpeg/FFmpeg/pulls/20172) by n⁠owrep
<fjlogger> [FFmpeg/FFmpeg] Pull request #20173 opened: checkasm: ac3dsp: Fix typos, increase tolerance to fix stray checkasm failures (https://code.ffmpeg.org/FFmpeg/FFmpeg/pulls/20173) by m⁠storsjo
<Lynne> BtbN: I think that scheduling for merging, then approving, causes jobs to remain stuck forever
<TheVibeCoder> Lynne: so the steps in proresraw fixed ?
<fjlogger> [FFmpeg/FFmpeg] Pull request #20174 opened: vulkan_encode_av1: Fix setting QP (https://code.ffmpeg.org/FFmpeg/FFmpeg/pulls/20174) by n⁠owrep
<Lynne> not entirely, I know the code I need to port to fix it, but I needed a few more hours without worrying about whether it was going to land in 8.0 or not, so I'll fix it tomorrow or so
<fjlogger> [FFmpeg/FFmpeg] Pull request #20164 merged: avfilter/vf_libplacebo: various frame blending improvements (https://code.ffmpeg.org/FFmpeg/FFmpeg/pulls/20164) by h⁠aasn
<Lynne> the non-opencl decoder .dll is better to RE, since it has almost no optimizations
<Lynne> so you can clearly see where dequant happens
<TheVibeCoder> ok
<Lynne> the stairsteps come from the top bits of the coeffs
<Lynne> basically the code does (coeff * scale1) * ((coeff >> bits) * scale1) * scale2 >> 13
<Lynne> so you need to boost according to the top bits
<TheVibeCoder> very messy
<Lynne> quod est, est
<BtbN> Lynne: no, there is just an incredibly backlog of jobs
<BtbN> see the number on the Actions tab
<Sean_McG> the new Vulkan stuff broke my 32-bit powerpc FATE node: https://fate.ffmpeg.org/history.cgi?slot=ppc-linux-gcc-14.2-altivec
<Sean_McG> In file included from src/libavcodec/vulkan_av1.c:21:
<Sean_McG> src/libavcodec/vulkan_decode.h:41:5: error: unknown type name ‘VkVideoDecodeVP9ProfileInfoKHR’; did you mean ‘VkVideoDecodeAV1ProfileInfoKHR’?
<BtbN> I've manually launched 20 runners, so the backlog should clear quickly now, will need to check why the automatic mechanism didn't work
<Lynne> Sean_McG: is CONFIG_VP9_VULKAN_HWACCEL defined in config.h?
<Sean_McG> this was from a FATE run so that config.h doesn't exist -- I'll have to run it manually
<fjlogger> [FFmpeg/FFmpeg] Pull request #20174 merged: vulkan_encode_av1: Fix setting QP (https://code.ffmpeg.org/FFmpeg/FFmpeg/pulls/20174) by L⁠ynne
<Sean_McG> looks like no, not defined.
<fjlogger> [FFmpeg/FFmpeg] Pull request #20172 merged: vulkan_encode_av1: Fix uninitialized return value in write_extra_headers (https://code.ffmpeg.org/FFmpeg/FFmpeg/pulls/20172) by L⁠ynne
<Lynne> Sean_McG: could you apply https://0x0.st/8FrB.patch and report back if it worked?
<Sean_McG> sure
<Lynne> tbh I don't see any non-messy way other than bumping the minimum required headers version
<fjlogger> [FFmpeg/FFmpeg] New comment on pull request #20138 avfilter/vf_colordetect: optimize C functions a bit (https://code.ffmpeg.org/FFmpeg/FFmpeg/pulls/20138#issuecomment-2398) by h⁠aasn
<fjlogger> [FFmpeg/FFmpeg] New comment on pull request #20138 avfilter/vf_colordetect: optimize C functions a bit (https://code.ffmpeg.org/FFmpeg/FFmpeg/pulls/20138#issuecomment-2399) by h⁠aasn
<fjlogger> [FFmpeg/FFmpeg] Pull request #20175 opened: hw_base_encode: Support refs option to limit number of references (https://code.ffmpeg.org/FFmpeg/FFmpeg/pulls/20175) by n⁠owrep
<fjlogger> [FFmpeg/FFmpeg] New comment on pull request #20175 hw_base_encode: Support refs option to limit number of references (https://code.ffmpeg.org/FFmpeg/FFmpeg/pulls/20175#issuecomment-2403) by L⁠ynne
<Sean_McG> also, separately, it seems Address Sanitizer does not like the MCC demux
<fjlogger> [FFmpeg/FFmpeg] New comment on pull request #20175 hw_base_encode: Support refs option to limit number of references (https://code.ffmpeg.org/FFmpeg/FFmpeg/pulls/20175#issuecomment-2404) by n⁠owrep
<Sean_McG> Lynne: that patch does not fix it here: error: unknown type name ‘VkVideoDecodeVP9ProfileInfoKHR’
<Lynne> yeah, I didn't think it would
jamrial has joined #ffmpeg-devel
<fjlogger> [FFmpeg/FFmpeg] New comment on pull request #20024 rewrite/add mcc [de]muxer, add AV_CODEC_ID_SMPTE_436M_ANC with mxf support, and add conversions <-> eia-608/708 (https://code.ffmpeg.org/FFmpeg/FFmpeg/pulls/20024#issuecomment-2405) by s⁠ean_mcg
<fjlogger> [FFmpeg/FFmpeg] New comment on pull request #20175 hw_base_encode: Support refs option to limit number of references (https://code.ffmpeg.org/FFmpeg/FFmpeg/pulls/20175#issuecomment-2406) by L⁠ynne
<fjlogger> [FFmpeg/FFmpeg] Pull request #20176 opened: vulkan_vp9: Fix frame threading (https://code.ffmpeg.org/FFmpeg/FFmpeg/pulls/20176) by b⁠cheng
<fjlogger> [FFmpeg/FFmpeg] New comment on pull request #20176 vulkan_vp9: Fix frame threading (https://code.ffmpeg.org/FFmpeg/FFmpeg/pulls/20176#issuecomment-2412) by L⁠ynne
<fjlogger> [FFmpeg/FFmpeg] Pull request #20158 merged: lavu/hwcontext_vaapi: fix a race mapping to allow CSC to YUV420P (https://code.ffmpeg.org/FFmpeg/FFmpeg/pulls/20158) by L⁠ynne
<fjlogger> [FFmpeg/FFmpeg] Pull request #20157 merged: lavc/vaapi_encode_av1: Fix ref_order_hint value for second slot (https://code.ffmpeg.org/FFmpeg/FFmpeg/pulls/20157) by L⁠ynne
<fjlogger> [FFmpeg/FFmpeg] Pull request #20177 opened: Keep compatibility with old Vulkan headers (https://code.ffmpeg.org/FFmpeg/FFmpeg/pulls/20177) by L⁠ynne
<fjlogger> [FFmpeg/FFmpeg] New comment on pull request #20177 Keep compatibility with old Vulkan headers (https://code.ffmpeg.org/FFmpeg/FFmpeg/pulls/20177#issuecomment-2422) by L⁠ynne
<Sean_McG> OK, give me a few minutes, in the middle of something
<fjlogger> [FFmpeg/FFmpeg] New comment on pull request #20129 avcodec/libmpeghdec: Add MPEG-H 3DA Fraunhofer IIS mpeghdec decoder (https://code.ffmpeg.org/FFmpeg/FFmpeg/pulls/20129#issuecomment-2423) by d⁠stadelmann-iis
MisterMinister has quit [Ping timeout: 272 seconds]
zane has joined #ffmpeg-devel
MisterMinister has joined #ffmpeg-devel
<Sean_McG> Lynne: the configure portion is broken
<Lynne> fixed it 1 minute ago
<Sean_McG> ok
psykose has quit [Remote host closed the connection]
psykose has joined #ffmpeg-devel
<Sean_McG> sorry, still no joy: src/libavcodec/vulkan_decode.c:106:40: error: ‘VK_STRUCTURE_TYPE_VIDEO_DECODE_VP9_PROFILE_INFO_KHR’ undeclared
<Lynne> Sean_McG: try again
jianhuaw has joined #ffmpeg-devel
<fjlogger> [FFmpeg/FFmpeg] New comment on pull request #20134 FFV1 / utvideo / magicyuv uninitilaized memory use (https://code.ffmpeg.org/FFmpeg/FFmpeg/pulls/20134#issuecomment-2426) by m⁠ichaelni
<fjlogger> [FFmpeg/FFmpeg] Pull request #20178 opened: avcodec/ilbcdec: Clear cbvec when used with create_augmented_vector() (https://code.ffmpeg.org/FFmpeg/FFmpeg/pulls/20178) by m⁠ichaelni
<Sean_McG> lavc/vulkan_{de,en}code.o built now, trying the rest of lavc
<Sean_McG> (it'll be a bit -- dual CPU G5 from 2005 is not that fast, even with an SSD
mkver has quit [Ping timeout: 276 seconds]
microlappy has joined #ffmpeg-devel
microlappy has quit [Client Quit]
microlappy has joined #ffmpeg-devel
microlappy has quit [Client Quit]
<fjlogger> [FFmpeg/FFmpeg] New comment on pull request #20176 vulkan_{av1,vp9}: Fix frame threading (https://code.ffmpeg.org/FFmpeg/FFmpeg/pulls/20176#issuecomment-2435) by b⁠cheng
DauntlessOne498 has quit [Remote host closed the connection]
<fjlogger> [FFmpeg/FFmpeg] New comment on issue #20171 Rebasing Security, Supply chain attacks (https://code.ffmpeg.org/FFmpeg/FFmpeg/issues/20171#issuecomment-2436) by m⁠ichaelni
<fjlogger> [FFmpeg/FFmpeg] New comment on pull request #20176 vulkan_{av1,vp9}: Fix frame threading (https://code.ffmpeg.org/FFmpeg/FFmpeg/pulls/20176#issuecomment-2440) by L⁠ynne
<jamrial> BtbN: i see g.v.o is up to date. did they remove the hooks or just disabled them for the one offending commit?
<BtbN> thresh came back from vacation and disabled the hooks
<BtbN> cone is also disabled
<jamrial> ok
<Sean_McG> so just for reference, Debian unstable currently only has libvulkan 1.4.309.0
blb has quit [Quit: brb]
<fjlogger> [FFmpeg/FFmpeg] Pull request #20176 merged: vulkan_{av1,vp9}: Fix frame threading (https://code.ffmpeg.org/FFmpeg/FFmpeg/pulls/20176) by L⁠ynne
<fjlogger> [FFmpeg/FFmpeg] Pull request #20179 opened: Assorted Vulkan fixes (https://code.ffmpeg.org/FFmpeg/FFmpeg/pulls/20179) by j⁠amrial
<jamrial> BtbN: guess codeowners is not working? the above pr should have added Lynne as reviewer
blb has joined #ffmpeg-devel
<fjlogger> [FFmpeg/FFmpeg] Pull request #20177 merged: Keep compatibility with old Vulkan headers (https://code.ffmpeg.org/FFmpeg/FFmpeg/pulls/20177) by j⁠amrial
<BtbN> Did you manually add it now, or was that just delayed?
<jamrial> added it manually
<BtbN> I don't think *vulkan* works the way you think
<BtbN> I think it should be **/*vulkan*
<BtbN> this way it only matches files in the top level
<BtbN> Oh
<BtbN> "Lines in this file represent rules. Each rule consists of a Go-formatted regular expression matching paths to changed files"
<BtbN> It's regexp, not globbing
<BtbN> so none of our rules should work
<BtbN> That's confusing, and not how it works elsewhere, let me double check that
<BtbN> Yeah, it's regexp on Forgejo.
DauntlessOne498 has joined #ffmpeg-devel
<fjlogger> [FFmpeg/FFmpeg] Pull request #20180 opened: forgejo/CODEOWNER: forgejo uses regex to match paths (https://code.ffmpeg.org/FFmpeg/FFmpeg/pulls/20180) by B⁠tbN
<Lynne> Sean_McG: did it work?
<Sean_McG> still waiting for the compile
<Sean_McG> almost done, it's a lavc/vvc/ now
<Sean_McG> *at
<Lynne> well, it was already merged, so hopefully it works :)
<fjlogger> [FFmpeg/FFmpeg] Pull request #20179 merged: Assorted Vulkan fixes (https://code.ffmpeg.org/FFmpeg/FFmpeg/pulls/20179) by j⁠amrial
<Sean_McG> it probably does
<Sean_McG> I wonder if we should poke Debian packagers to see if they can at least move to 1.4.317+
<Sean_McG> OK yeah, lavc builds fine for me now -- I'll fire off a new FATE run and hopefully that's good. I think I am heading out fod lunch shortly.
<Sean_McG> *for
<JEEB> debian backports often has newer versions of things
<Sean_McG> ohhh I didn't think to check there
<fjlogger> [FFmpeg/FFmpeg] New comment on pull request #20134 FFV1 / utvideo / magicyuv uninitilaized memory use (https://code.ffmpeg.org/FFmpeg/FFmpeg/pulls/20134#issuecomment-2462) by m⁠ichaelni
zane has quit [Quit: WeeChat 4.7.0]
mkver has joined #ffmpeg-devel
jianhuaw has quit [Ping timeout: 252 seconds]
jianhuaw has joined #ffmpeg-devel
<fjlogger> [FFmpeg/FFmpeg] Pull request #20181 opened: CODEOWNERS: Add VVC Code Owners (https://code.ffmpeg.org/FFmpeg/FFmpeg/pulls/20181) by j⁠ianhuaw
<fjlogger> [FFmpeg/FFmpeg] New comment on pull request #20168 avcodec/apv_decode: Check that tiles are within the frame (https://code.ffmpeg.org/FFmpeg/FFmpeg/pulls/20168#issuecomment-2469) by j⁠amrial
<fjlogger> [FFmpeg/FFmpeg] New comment on pull request #20138 avfilter/vf_colordetect: optimize C functions a bit (https://code.ffmpeg.org/FFmpeg/FFmpeg/pulls/20138#issuecomment-2470) by k⁠asper93
<fjlogger> [FFmpeg/FFmpeg] New comment on pull request #20181 CODEOWNERS: Add VVC Code Owners (https://code.ffmpeg.org/FFmpeg/FFmpeg/pulls/20181#issuecomment-2471) by B⁠tbN
<fjlogger> [FFmpeg/FFmpeg] New comment on pull request #20181 CODEOWNERS: Add VVC Code Owners (https://code.ffmpeg.org/FFmpeg/FFmpeg/pulls/20181#issuecomment-2472) by j⁠ianhuaw
<fjlogger> [FFmpeg/FFmpeg] Pull request #20181 closed: CODEOWNERS: Add VVC Code Owners (https://code.ffmpeg.org/FFmpeg/FFmpeg/pulls/20181) by j⁠ianhuaw
<fjlogger> [FFmpeg/FFmpeg] New comment on pull request #20181 CODEOWNERS: Add VVC Code Owners (https://code.ffmpeg.org/FFmpeg/FFmpeg/pulls/20181#issuecomment-2474) by B⁠tbN
<fjlogger> [FFmpeg/FFmpeg] New comment on pull request #20181 CODEOWNERS: Add VVC Code Owners (https://code.ffmpeg.org/FFmpeg/FFmpeg/pulls/20181#issuecomment-2475) by j⁠ianhuaw
<kasper93> don't we want snprinf ub fix merged before 8.0?
<TheVibeCoder> no
<kasper93> ok
<BtbN> could someone quickly re-approve the CODEOWNERS fix?
<kasper93> it's really weird that thoes are not globs
<fjlogger> [FFmpeg/FFmpeg] Pull request #20180 merged: forgejo/CODEOWNER: forgejo uses regex to match paths (https://code.ffmpeg.org/FFmpeg/FFmpeg/pulls/20180) by B⁠tbN
novaphoenix has quit [Server closed connection]
novaphoenix has joined #ffmpeg-devel
<jianhuaw> Hi  there. Could someone with admin access to the Fate server help review this PR (https://code.ffmpeg.org/FFmpeg/FFmpeg/pulls/20161)? The small VVC fix has been reviewed by Nuomi and Frank, which is needed for 8.0.
mkver has quit [Ping timeout: 245 seconds]
<frankplow> jianhuaw: Have you sent an email to samples-request@ffmpeg.org?
mkver has joined #ffmpeg-devel
<jianhuaw> No. I didn't know about it. Should I send the links to the samples to the samples-request@ffmpeg.org?
<frankplow> jianhuaw: Yeah exactly, plus the paths you want them to be stored at in the FATE samples dir, and it might also be useful to send a link to the relevant PR
<jianhuaw> Okay. I'll try. Thanks!
<BtbN> I've just added them. I guess I as supposed to unzip the files, and add the *.bit files, but rename them to remove the manufacturer part?
<JEEB> reference suite files are usually kept as-is? at least looking at the ISO/ITU formats
<jianhuaw> Yes. The manufacturer is removed as convention.
<jianhuaw> Most of them in fate-suite/vvc-conformance have remove the manufacturer.
<jamrial> BtbN: could disabling cone also made the cvslog ml not get any new email?
<BtbN> I don't think so? iirc the cvslog ml is on our server
<BtbN> as in, the thing that sends to it
<BtbN> But I have never come into contact with that
<jamrial> well, something happened because as nicolas reported it hasn't gotten new mails in three days :/
<BtbN> Hm, that does coincide with the videolan hooks being removed
<wbs> jamrial: it didn't get mails for pushes to master before that either; it was silent for many days, but then there was some pushes to release branches, which did generate mails there
<wbs> (possibly that commit syncs for release branches worked, while syncs for master didn't)
<BtbN> yeah, must have come from videolan then
<BtbN> but there is ffmpeg-web commits on there, which isn't on videolan
<BtbN> so we must have it set up as well, and I just need to turn it on. But I have no clue where the mails come from
<wbs> the mails for ffmpeg-web look slightly different than the ones from ffmpeg as well, so they're probably different scripts on different hosts
<BtbN> I've just run git config hooks.mailinglist ffmpeg-cvslog@ffmpeg.org on the ffmpeg.git repo
<BtbN> that's the only difference I see compared to ffmpeg-web. So maybe that's all it takes
<jianhuaw> Thanks BtbN The fate works. Feel free to merge the PR if you have permission as Michael is going to make the 8.0 branch. frankplow Thanks.
<fjlogger> [FFmpeg/FFmpeg] Pull request #20161 merged: avcodec/vvc/ctu: should use the width and height of the start component (https://code.ffmpeg.org/FFmpeg/FFmpeg/pulls/20161) by B⁠tbN
mkver has quit [Ping timeout: 245 seconds]
mkver has joined #ffmpeg-devel
<Yalda> kasper93: the good news is, based on some initial tests, this demuxer does not seem to have the issue. the bad news is, I don't know why the issue does not manifest. so I'm running with some more tests/analysis first to be truly sure.
<fjlogger> [FFmpeg/FFmpeg] New comment on pull request #20181 CODEOWNERS: Add VVC Code Owners (https://code.ffmpeg.org/FFmpeg/FFmpeg/pulls/20181#issuecomment-2485) by j⁠ianhuaw
<fjlogger> [FFmpeg/FFmpeg] Pull request #20181 reopened: CODEOWNERS: Add VVC Code Owners (https://code.ffmpeg.org/FFmpeg/FFmpeg/pulls/20181) by j⁠ianhuaw
<kasper93> Yalda: have you tried with some of those most problematic examples?
<Yalda> kasper93: not yet but I will test thoroughly
<Yalda> I agree it should happen based on what I am reading
<Yalda> (as in, the issue should happen)
<kasper93> I think the issue was manifested mostly on discs with 1000+ segments, where audio tracks had overlapping parts and if you don't correct them, the audio went out of sync because the small offset adds up
<Yalda> appreciate you flagging this, I would have missed it completely
<Yalda> beside accounting for this issue and seeking still being janky the demuxer is basically done
<kasper93> Yalda: thanks, I don't have blu-ray drive in my pc anymore, can't really test much. without connecting it
<kasper93> my current pc case, don't even have bay for 3.5" xd
<Yalda> no worries. I have an army of drives
<Yalda> mostly DVD but have some BD in there
<kasper93> 5.25" I should say, bad memory to numbers
sepro has quit [Quit: Bye!]
<Yalda> it was sad building my last machine. local store only had one case with 5.25" bay, off in the corner, unwanted by anyone (except for me lol)
<JEEB> I have a buffalo USB3 blu-ray drive for laptop or new stuff usage. and then I have a "UHD friendly" Asus SATA drive for UHD BD usage
<Yalda> the ASUS/LG drives are good. whatever you do, do not trust them for DVD
<Yalda> they will return junk data on scratched or bad sectors and simply say "block read ok"
<Yalda> this is called "Jamless Play" feature. good for playback, bad for archival
<fjlogger> [FFmpeg/FFmpeg] New comment on issue #20171 Rebasing Security, Supply chain attacks (https://code.ffmpeg.org/FFmpeg/FFmpeg/issues/20171#issuecomment-2490) by k⁠asper93
sepro has joined #ffmpeg-devel
<fjlogger> [FFmpeg/FFmpeg] Pull request #20181 merged: CODEOWNERS: Add VVC Code Owners (https://code.ffmpeg.org/FFmpeg/FFmpeg/pulls/20181) by B⁠tbN
^Neo has joined #ffmpeg-devel
<fjlogger> [FFmpeg/FFmpeg] Pull request #20182 opened: smpte436m_to_eia608 and eia608_to_smpte436m: fix memory leaks -- forgot to free input av_packet (https://code.ffmpeg.org/FFmpeg/FFmpeg/pulls/20182) by p⁠rogrammerjake
<fjlogger> [FFmpeg/FFmpeg] New comment on pull request #20024 rewrite/add mcc [de]muxer, add AV_CODEC_ID_SMPTE_436M_ANC with mxf support, and add conversions <-> eia-608/708 (https://code.ffmpeg.org/FFmpeg/FFmpeg/pulls/20024#issuecomment-2496) by p⁠rogrammerjake
<fjlogger> [FFmpeg/FFmpeg] New comment on pull request #20182 smpte436m_to_eia608 and eia608_to_smpte436m: fix memory leaks -- forgot to free input av_packet (https://code.ffmpeg.org/FFmpeg/FFmpeg/pulls/20182#issuecomment-2498) by p⁠rogrammerjake
<kasper93> jamrial: do we want to fix memory leak in mov before 8.0?
<kasper93> fate is missing some samples looks like
<fjlogger> [FFmpeg/FFmpeg] New comment on pull request #20182 smpte436m_to_eia608 and eia608_to_smpte436m: fix memory leaks -- forgot to free input av_packet (https://code.ffmpeg.org/FFmpeg/FFmpeg/pulls/20182#issuecomment-2501) by s⁠ean_mcg
minimal has joined #ffmpeg-devel
Guest57 has joined #ffmpeg-devel
Guest57 has quit [Client Quit]
MisterMinister has quit [Ping timeout: 260 seconds]
MisterMinister has joined #ffmpeg-devel
jianhuaw has quit [Ping timeout: 252 seconds]
<fjlogger> [FFmpeg/FFmpeg] New comment on issue #20183 MCC FATE tests request the wrong input file (https://code.ffmpeg.org/FFmpeg/FFmpeg/issues/20183#issuecomment-2504) by p⁠rogrammerjake
<fjlogger> [FFmpeg/FFmpeg] New comment on issue #20184 Consider the addition of a "needs backport" label for PRs (https://code.ffmpeg.org/FFmpeg/FFmpeg/issues/20184#issuecomment-2505) by Y⁠alda
<kasper93> Yalda: you should have powers to add a label, I think for those things you can just add ad ask for forgiveness later if someone goes off that it cannot be like that :)
<beastd> true, but we should eventually come to strong rules on how to label what. otherwise labels are just free form tags and don't provide much universal utility. or the opposite if developers get confused.
Guest has joined #ffmpeg-devel
<kasper93> https://x.com/colin_fraser/status/1953668411029909892 it should generate python to solve this... lmao
<Yalda> I know michaelni does a bulk of backport duty so was gonna let the ticket sit for feedback
<Yalda> also lol at math fail
<Yalda> I am also cautious of label overload but I think this one is wise
<TheVibeCoder> ffmpeg is dead and full of regressions and bugs
<fjlogger> [FFmpeg/FFmpeg] New comment on pull request #20182 smpte436m_to_eia608 and eia608_to_smpte436m: fix memory leaks -- forgot to free input av_packet (https://code.ffmpeg.org/FFmpeg/FFmpeg/pulls/20182#issuecomment-2508) by p⁠rogrammerjake
<fjlogger> [FFmpeg/FFmpeg] New comment on issue #20183 MCC FATE tests request the wrong input file (https://code.ffmpeg.org/FFmpeg/FFmpeg/issues/20183#issuecomment-2510) by s⁠ean_mcg
<fjlogger> [FFmpeg/FFmpeg] New comment on pull request #20022 libavfilter: Whisper audio filter (https://code.ffmpeg.org/FFmpeg/FFmpeg/pulls/20022#issuecomment-2512) by m⁠ichaelni
<kasper93> yes
<beastd> Yalda,kasper93: regarding math and LLMs ... i had some strange thoughts lately
<beastd> 1. you have a problem you want to solve
<beastd> 2. you have a computer that can potentially solve problems
<beastd> 3. you need to programs that solve your problem at hand
<beastd> 4. you do not know which program to use and how
<beastd> 5. you use one program to solve all of your problems. say that is a chatbot that is powered by an llm
<fjlogger> [FFmpeg/FFmpeg] New comment on pull request #20022 libavfilter: Whisper audio filter (https://code.ffmpeg.org/FFmpeg/FFmpeg/pulls/20022#issuecomment-2513) by k⁠asper93
<beastd> so now suppose you have that simple math problem or say you want to decode this BASE64 string dGhpcyBpcyBzbyBmcmVha2luZyB3cm9uZwo=
<beastd> the llm needs somehow come to a solution to what you want. in many cases it should use non-llm tooling to get the answer right
<fjlogger> [FFmpeg/FFmpeg] New comment on issue #20183 MCC FATE tests request the wrong input file (https://code.ffmpeg.org/FFmpeg/FFmpeg/issues/20183#issuecomment-2514) by p⁠rogrammerjake
<beastd> so to just decode that base64 string you implicitly lots of things which in the end boild down to e.g. just to a commond like `base64 -d`
<beastd> especially weird, if that chatbot is not running completely local
<beastd> i have the feeling the irony is lost on a lot of people that actually do things like that...
<Yalda> i feel like even for simple things like rearrange this array to a different format, i have to check its work. have caught mistakes for sure
<fjlogger> [FFmpeg/FFmpeg] New comment on pull request #20022 libavfilter: Whisper audio filter (https://code.ffmpeg.org/FFmpeg/FFmpeg/pulls/20022#issuecomment-2515) by v⁠palmisano
<Yalda> i have been cloning some ancient IR remotes and using it to help with pronto codes
<beastd> Yalda: yes, the checking the work is what makes the whole thing not really helpful at all. and that makes the use case to use LLMs for coding / generating programs inefficient.
<beastd> unless you are yolo and reckless, which seems to be almost our complete industrie as it seems :-/
<Yalda> beastd: agreed
<Yalda> i now ask it more along, "make me the regex" or "generate the command to do it myself"
<beastd> you can get sth out of LLMs, but to me it looks like it is no where near as advertised and portrayed in the press. and i am convinced you have to be very careful.
<Yalda> they are a tool in the box, like anything else, cannot depend on for everything
<fjlogger> [FFmpeg/FFmpeg] New comment on pull request #20022 libavfilter: Whisper audio filter (https://code.ffmpeg.org/FFmpeg/FFmpeg/pulls/20022#issuecomment-2516) by m⁠ichaelni
sepro has quit [Ping timeout: 276 seconds]
sepro has joined #ffmpeg-devel
TheVibeCoder has quit [Ping timeout: 265 seconds]
<fjlogger> [FFmpeg/FFmpeg] New comment on issue #20171 Rebasing Security, Supply chain attacks (https://code.ffmpeg.org/FFmpeg/FFmpeg/issues/20171#issuecomment-2520) by b⁠eastd
<fjlogger> [FFmpeg/FFmpeg] New comment on pull request #20022 libavfilter: Whisper audio filter (https://code.ffmpeg.org/FFmpeg/FFmpeg/pulls/20022#issuecomment-2521) by m⁠ichaelni
<fjlogger> [FFmpeg/FFmpeg] Pull request #20022 closed: libavfilter: Whisper audio filter (https://code.ffmpeg.org/FFmpeg/FFmpeg/pulls/20022) by m⁠ichaelni
<fjlogger> [FFmpeg/FFmpeg] New comment on pull request #20022 libavfilter: Whisper audio filter (https://code.ffmpeg.org/FFmpeg/FFmpeg/pulls/20022#issuecomment-2523) by m⁠ichaelni
<kierank> frankplow: just booked arsenal tottenham
ccawley2011 has joined #ffmpeg-devel
<fjlogger> [FFmpeg/FFmpeg] New comment on issue #20171 Rebasing Security, Supply chain attacks (https://code.ffmpeg.org/FFmpeg/FFmpeg/issues/20171#issuecomment-2527) by p⁠rogrammerjake
secondcreek has quit [Quit: secondcreek]
<fjlogger> [FFmpeg/FFmpeg] New comment on pull request #20168 avcodec/apv_decode: Check that tiles are within the frame (https://code.ffmpeg.org/FFmpeg/FFmpeg/pulls/20168#issuecomment-2533) by m⁠ichaelni
mkver has quit [Ping timeout: 252 seconds]
mkver has joined #ffmpeg-devel
Sean_McG has quit [Remote host closed the connection]
Guest has quit [Quit: Client closed]
<jkqxz> jamrial: Please don't put non-spec fields in the spec structures if avoidable.
<jkqxz> Maybe it would be better to move the _starts fields into the decoder? CBS doesn't care about them, all it wants is the total number of tiles.
<jkqxz> That would be faster for other CBS APV users as well since they don't care either.
<jamrial> jkqxz: ok, will chekc
<fjlogger> [FFmpeg/FFmpeg] Pull request #20186 opened: forgejo/autolabeler: add some logic to also auto-remove the "New" label (https://code.ffmpeg.org/FFmpeg/FFmpeg/pulls/20186) by B⁠tbN
<jkqxz> The encoder also wants them before CBS write can run. I think I moved the derive function out of CBS for that, but splitting it would probably be better.
<jkqxz> I'll have a look tomorrow. (And also maybe finish the encoder at some point.)
<fjlogger> [FFmpeg/FFmpeg] Pull request #20187 opened: avcodec/apv_decode: make apv_format_table consistent with the code and check it (https://code.ffmpeg.org/FFmpeg/FFmpeg/pulls/20187) by m⁠ichaelni
<michaelni> jkqxz, ^
<michaelni> fix for 2nd testsample
<fjlogger> [FFmpeg/FFmpeg] New comment on issue #20171 Rebasing Security, Supply chain attacks (https://code.ffmpeg.org/FFmpeg/FFmpeg/issues/20171#issuecomment-2549) by k⁠asper93
minimal has quit [Quit: Leaving]
<fjlogger> [FFmpeg/FFmpeg] Pull request #20188 opened: avcodec/scpr3: Clear clr (https://code.ffmpeg.org/FFmpeg/FFmpeg/pulls/20188) by m⁠ichaelni
Flat_ has joined #ffmpeg-devel
Flat has quit [Ping timeout: 248 seconds]
mkver has quit [Ping timeout: 244 seconds]
<kierank> BtbN: what's with all the spam?
<BtbN> ?
<kierank> I'm getting a ton of ffmpeg/ffmpeg-citest spam
<kierank> on an email not subscribed to that
<BtbN> ah, I thought making private would prevent that
<BtbN> well, the name should make it obvious for what it is.
<kierank> yeah but it's my personal email, it's annoying
<BtbN> Can only test CI inside of the FFmpeg org.
<kierank> so why does it need to spam me?
<kierank> for something I haven't subscribed to
<BtbN> No idea
<BtbN> I explicitly made it private
<BtbN> I guess all org members just get auto-subscribed to every org repo
<kierank> great...
<BtbN> The good news is, testing is long done, so no more spam
<BtbN> Though I did plan to point everyone at that repo who wants to test CI stuff
<kierank> yes but any random repos need warning so people can disable the log spam
<kierank> I just disabled "watch" for nv-codec-headers
<BtbN> Might need to figure out how to turn off notifications globally for that repo first
<kierank> I certainly didn't watch that explicitly
<BtbN> Yeah, it must watch every member to everything by default
<kierank> pfft
<BtbN> nv-codec-headers is decently low traffic at least
<fjlogger> [FFmpeg/FFmpeg] Pull request #20141 merged: MAINTAINERS: add myself as hwcontext_d3d12va maintainer (https://code.ffmpeg.org/FFmpeg/FFmpeg/pulls/20141) by m⁠ichaelni
iive has joined #ffmpeg-devel
<fjlogger> [FFmpeg/FFmpeg] Pull request #20189 opened: libavfilter: add af_whisper codeowner (https://code.ffmpeg.org/FFmpeg/FFmpeg/pulls/20189) by v⁠palmisano
kimapr has joined #ffmpeg-devel
Kimapr_ has quit [Remote host closed the connection]
jianhuaw has joined #ffmpeg-devel
<fjlogger> [FFmpeg/FFmpeg] New comment on pull request #20097 Fix more warnings (https://code.ffmpeg.org/FFmpeg/FFmpeg/pulls/20097#issuecomment-2562) by k⁠asper93
<kasper93> there are also cbs_vp9_syntax_template.c:374:9: warning: result of comparison of constant -1 with expression of type 'uint8_t' (aka 'unsigned char') is always true
<kasper93> not sure what you wanted to check there tho
uau has quit [Server closed connection]
ccawley2011 has quit [Read error: Connection reset by peer]
uau has joined #ffmpeg-devel
_whitelogger has joined #ffmpeg-devel
jianhuaw has quit [Quit: Client closed]
<desmond-netint> I want to add hwaccel video codecs to libavcodec. Is it better to add them in one go or add them piece-meal in separate PRs?