Re: [FFmpeg-devel] [RFC] libpostproc splitout

2024-11-22 Thread Derek Buitenhuis
On 11/22/2024 12:59 PM, epira...@gmail.com wrote: > While I might not in principle disagree with the CCs action here, the (at > least perceived) inaction of the CC > in other cases makes this seem the CC has a huge double standard how to deal > with things… > (Or there is a big communication/tran

Re: [FFmpeg-devel] Fwd: [RFC] libpostproc splitout

2024-11-22 Thread Derek Buitenhuis
On 11/22/2024 12:41 PM, Nicolas George wrote: > I do not agree with you on many things, including what you said on the > criticised mail, but I fully agree with you that this intervention of a > CC member is entirely inappropriate and abusive. There are many things I've said lately that the CC cou

[FFmpeg-devel] Fwd: [RFC] libpostproc splitout

2024-11-22 Thread Derek Buitenhuis
, complete bullshit. The CC is a lame duck. - Derek Forwarded Message Subject:Re: [FFmpeg-devel] [RFC] libpostproc splitout Date: Thu, 21 Nov 2024 15:18:04 -0500 From: Ronald S. Bultje To: Derek Buitenhuis CC: c...@ffmpeg.org Hi Derek, On Thu, Nov 7, 2024

[FFmpeg-devel] [REQUEST] Remove me from GA

2024-11-19 Thread Derek Buitenhuis
Buckle in, this'll be a bit of a ride. Part of it is inflamatory, and part of it is paranoid (or not) rants. I accept this may be slander and will pre-emptively ban myself for the sake of getting it in the open. I do not wish to be involved in any voting or disucssions anymore, related to the dire

Re: [FFmpeg-devel] [RFC] dormant git accounts

2024-11-19 Thread Derek Buitenhuis
On 11/12/2024 7:55 PM, compn wrote: > On Tue, 12 Nov 2024 16:46:42 + > Derek Buitenhuis wrote: > >> On 11/11/2024 7:34 PM, compn wrote: >>> if your goal is to post old quotes, thats cool. >> >> Woosh. > > the quotes are from michael in 2015 s

Re: [FFmpeg-devel] [RFC] dormant git accounts

2024-11-19 Thread Derek Buitenhuis
On 11/12/2024 7:37 PM, compn wrote: > concern trolling? I am pointing out Michael's own logic isn't even consistent with itself. What is logic *actually* is is that the of course *he* is trustworthy, to him. > > you're concerned about one developer adding in a backdoor, so the > solution is to

Re: [FFmpeg-devel] [RFC] dormant git accounts

2024-11-19 Thread Derek Buitenhuis
On 11/12/2024 6:41 PM, Rémi Denis-Courmont wrote: > I don't think that Derek meant that literally. The GA is not a legal entity > so it can't hold a domain name or a trademark in the first place, or for that > matter physical servers or hosting service contracts. Just like the bank > account, th

Re: [FFmpeg-devel] [RFC] dormant git accounts

2024-11-12 Thread Derek Buitenhuis
On 11/12/2024 5:05 PM, James Almer wrote: > This is not true. I have write access to the website, for example, as do > others. And Michael cuts releases because he was given the task, not > because nobody else can or want. And nobody prevents anyone from just > fetching a git tag instead (Distro

Re: [FFmpeg-devel] [RFC] dormant git accounts

2024-11-12 Thread Derek Buitenhuis
On 11/12/2024 5:07 PM, James Almer wrote: > I personally don't agree with giving the domain/trademark to the general > assembly, as some have argued. It's just not safe at all. Sorry, I didn't necessarily mean giving it ot the GA. I mean having it in a better state than being held hostage by some

Re: [FFmpeg-devel] [RFC] dormant git accounts

2024-11-12 Thread Derek Buitenhuis
On 11/11/2024 7:34 PM, compn wrote: > if your goal is to post old quotes, thats cool. Woosh. > one of my goals is to make sure that certain developers, who made their > own project and then ran it into the ground, arent made as admins > again. they had a good run but couldnt even make an > announ

Re: [FFmpeg-devel] [RFC] dormant git accounts

2024-11-12 Thread Derek Buitenhuis
On 11/11/2024 7:33 PM, Michael Niedermayer wrote: >> This only convinces me further that it this whole setup ins't for for >> purpose, >> and is being run by people who have no concept of actual security. This is >> totally insane. Honestly, this is so exhausting and painful, I dread responding.

Re: [FFmpeg-devel] [RFC] dormant git accounts

2024-11-11 Thread Derek Buitenhuis
On 11/11/2024 4:42 PM, Michael Niedermayer wrote: > Publically listing which developer provides which part of the DNS infra > makes it easier to attack not harder. > That said, i suspect who provides what was mentioned in the past already It is already publically available info to anyone who can l

Re: [FFmpeg-devel] root access voting

2024-11-11 Thread Derek Buitenhuis
On 11/6/2024 2:48 PM, Michael Niedermayer wrote: > do i unilaterally decide who i trust as root, maybe. Q.E.D. That, along with Fabrice's control of ffmpeg.org and French trademark, despite not havign contributed in 20 years, purely for ego reasons, mean we can never move to a truly community run

Re: [FFmpeg-devel] [RFC] dormant git accounts

2024-11-11 Thread Derek Buitenhuis
On 11/10/2024 2:59 PM, Michael Niedermayer wrote: > Its there since a long time: > https://git.ffmpeg.org/gitweb/ffmpeg.git/blob/HEAD:/doc/infra.txt [...] > If something is missing, its not going to improve on its own. > Someone will have to say _what_ is missing and work toward filling it in. P

Re: [FFmpeg-devel] [RFC] libpostproc splitout

2024-11-10 Thread Derek Buitenhuis
On 11/7/2024 11:33 PM, Michael Niedermayer wrote: > And as a sidenote I and Derek where much friendlier towards each > other from what i remember, than in the discussion today. I hope we will be > friendlier towards each other in the future again. On the contrary, I remember being a much, much big

Re: [FFmpeg-devel] [RFC] dormant git accounts

2024-11-10 Thread Derek Buitenhuis
On 11/9/2024 6:04 PM, Rémi Denis-Courmont wrote: > What most people are concerned about right now is the incomplete > documentation > of any and all credentials - not just git write access - and more generally > the lack of transparency. Once that is sorted out, we can start arguing about > wha

Re: [FFmpeg-devel] [RFC] libpostproc splitout

2024-11-07 Thread Derek Buitenhuis
On 11/7/2024 7:52 PM, Michael Niedermayer wrote: > It also did not achieve the intended result. libpostproc did continue to > live in FFmpeg since that 2012 till today. Interesting way you've defined 'intended result' for yourself here. > The goal of this project is to have a seperate libpostproc

Re: [FFmpeg-devel] [RFC] libpostproc splitout

2024-11-07 Thread Derek Buitenhuis
On 11/7/2024 8:04 PM, Michael Niedermayer wrote: > This is slander > > libpostproc was simply deleted from libav. > > The messed up repository you created in 2012, was never maintained by anyone > > of course copying something in a messed up form and deleting support for it > is not worth 5k eu

Re: [FFmpeg-devel] [RFC] libpostproc splitout

2024-11-07 Thread Derek Buitenhuis
On 11/6/2024 11:11 PM, Michael Niedermayer wrote: > 1. split libpostproc out so it builds and links fine (already done) (send > to SPI/STF/Invoice in future) I did this in 2012, the git repo is still there: http://git.videolan.org/?p=libpostproc.git;a=summary This work was very easy and not

Re: [FFmpeg-devel] root access voting

2024-11-07 Thread Derek Buitenhuis
On 11/3/2024 4:37 PM, Michael Niedermayer wrote: > I very strongly object to you calling me "misleading and disparagin" I suspect this now deleted tweet meets the definition of "disparaging". See: https://imgur.com/a/RdPW15a The link was https://x.com/michael__ni/status/1853033116199219207 (I t

Re: [FFmpeg-devel] [RFC] libpostproc splitout

2024-11-07 Thread Derek Buitenhuis
On 11/6/2024 11:11 PM, Michael Niedermayer wrote: > 3. actually remove libpostproc from master repository (2025 future) (send > to SPI/STF/Invoice in future) I also did this exact work for Libav in 2012. It was very little work. Not 5k. - Derek ___

Re: [FFmpeg-devel] [RFC PATCH 0/2] Make fate tests succeed with zlib-ng

2024-10-18 Thread Derek Buitenhuis
On 10/18/2024 1:02 PM, Alexander Strasser via ffmpeg-devel wrote: > Alexander Strasser (2): > Reapply "tests/fate: disable compression for zlib-based codecs" > fate: Make it possible to have alternative reference files I want to add this here, since I brought it up on IRC. I think this entire

Re: [FFmpeg-devel] Uncompressed MP4

2024-09-24 Thread Derek Buitenhuis
Hi, On 9/24/2024 3:22 PM, Devon Sookhoo wrote: > Thanks for responding, I have been struggling to get any responses. I have > tried to use the IRC on Libera, however the server seems to be down. I suspect not too many people resonded on this list since it is more for code review than anything. W

Re: [FFmpeg-devel] A change in r_frame_rate values after upgrade to FFmpeg 6.1

2024-09-24 Thread Derek Buitenhuis
On 9/23/2024 4:34 PM, Vittorio Giovara wrote: > Should we schedule deprecation for one of the two fields? I agree it's > confusing for end users to check in two fields. Warning: Hot take. IMO r_frame_rate does more harm than good and should be nuked. Possibly avg_frame_rate should also be nuked,

Re: [FFmpeg-devel] A change in r_frame_rate values after upgrade to FFmpeg 6.1

2024-09-24 Thread Derek Buitenhuis
On 9/24/2024 12:27 AM, Michael Niedermayer wrote: > An "average" is precissely one number (once the type of average is defined) > Maybe you are thinkin of "approximate" instead of "average" Which doesn't actually represent how it is set usually. Take one grep around libavformat/*.c and you'll see

Re: [FFmpeg-devel] Uncompressed MP4

2024-09-24 Thread Derek Buitenhuis
On 9/23/2024 3:36 PM, Devon Sookhoo wrote: > I'm interested in implementing uncompressed MP4 parsing according to the > newly released ISO/IEC 23001-17:2024. Would anyone be interested in further > discussing this with me? Since, as far as I am aware, there are no official (or unofficial) test vec

Re: [FFmpeg-devel] [PATCH 10/14] swscale/range_convert: fix mpeg ranges in yuv range conversion for non-8-bit pixel formats

2024-09-24 Thread Derek Buitenhuis
On 9/23/2024 1:40 PM, Ramiro Polla wrote: > { > +#if 0 > int cpu_flags = av_get_cpu_flags(); Are these '#if 0' throughout this patch intended? - Derek ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org https://ffmpeg.org/mailman/listinfo/ffmpeg

Re: [FFmpeg-devel] [PATCH 03/13] avcodec/decode: support applying LCEVC enhacements

2024-09-02 Thread Derek Buitenhuis
On 8/31/2024 5:31 PM, James Almer wrote: > Signed-off-by: James Almer > --- > libavcodec/avcodec.c | 2 ++ > libavcodec/avcodec.h | 5 + > libavcodec/decode.c| 42 +- > libavcodec/internal.h | 2 ++ > libavcodec/lcevcdec.c |

Re: [FFmpeg-devel] [PATCH 07/13] avcodec/hevcdec: export LCEVC metadata as frame side data

2024-09-02 Thread Derek Buitenhuis
On 8/31/2024 5:31 PM, James Almer wrote: > Signed-off-by: James Almer > --- > configure | 1 + > libavcodec/hevc/hevcdec.c | 3 +++ > libavcodec/hevc/refs.c| 15 ++- > 3 files changed, 18 insertions(+), 1 deletion(-) Maybe I've misunderstood, but doesn't requiri

Re: [FFmpeg-devel] [PATCH v2] lavu/stereo3d: change the horizontal FOV field to a rational

2024-06-24 Thread Derek Buitenhuis
On 6/24/2024 1:13 AM, James Almer wrote: > If Derek is also ok with this then LGTM. I do not object. - Derek ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org https://ffmpeg.org/mailman/listinfo/ffmpeg-devel To unsubscribe, visit link above, or ema

Re: [FFmpeg-devel] [PATCH] lavu/stereo3d: change baseline and horizontal FOV to rationals

2024-06-23 Thread Derek Buitenhuis
On 6/23/2024 12:05 PM, Derek Buitenhuis wrote: > If I sound grumpy, it's because I am... the set sat commentless. The tone was unnecessary, sorry about that. - Derek ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org https://ffmpeg.org

Re: [FFmpeg-devel] [PATCH] lavu/stereo3d: change baseline and horizontal FOV to rationals

2024-06-23 Thread Derek Buitenhuis
On 6/22/2024 9:11 PM, Lynne via ffmpeg-devel wrote: > The API was committed 2 days ago, so changing these fields now > is within the realms of acceptable. > --- You also had weeks to comment on 3 revisions of the set, and it would have been nice to hear about this then instead of changing it after

Re: [FFmpeg-devel] [PATCH] avformat/dump: Print all possible Stereo3D info

2024-06-18 Thread Derek Buitenhuis
On 6/18/2024 7:34 PM, James Almer wrote: > nit: These three could be combined into a single av_log call. v2 sent. - Derek ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org https://ffmpeg.org/mailman/listinfo/ffmpeg-devel To unsubscribe, visit link

[FFmpeg-devel] [PATCH v2] avformat/dump: Print all possible Stereo3D info

2024-06-18 Thread Derek Buitenhuis
Signed-off-by: Derek Buitenhuis --- libavformat/dump.c | 11 ++- 1 file changed, 10 insertions(+), 1 deletion(-) diff --git a/libavformat/dump.c b/libavformat/dump.c index 059fb84522..61a2c6a29f 100644 --- a/libavformat/dump.c +++ b/libavformat/dump.c @@ -259,7 +259,16 @@ static void

[FFmpeg-devel] [PATCH] avformat/dump: Print all possible Stereo3D info

2024-06-18 Thread Derek Buitenhuis
Signed-off-by: Derek Buitenhuis --- As requested by James. --- libavformat/dump.c | 9 + 1 file changed, 9 insertions(+) diff --git a/libavformat/dump.c b/libavformat/dump.c index 059fb84522..8e8b9e1959 100644 --- a/libavformat/dump.c +++ b/libavformat/dump.c @@ -260,6 +260,15 @@ static

Re: [FFmpeg-devel] [PATCH v3 0/5] Vision Pro Spatial Data

2024-06-18 Thread Derek Buitenhuis
On 6/17/2024 8:20 PM, Derek Buitenhuis wrote: > 12 files changed, 490 insertions(+), 1 deletion(-) Will push later today if there are no objections. - Derek ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org https://ffmpeg.org/mailman/listi

Re: [FFmpeg-devel] [PATCH v3 4/5] avformat/mov: Add support for exporting Video Extension Usage info

2024-06-17 Thread Derek Buitenhuis
On 6/17/2024 9:33 PM, James Almer wrote: > Personally I'd say yes, but other similar lavu APIs (Mastering, Ambient > Viewing) don't seem to bother with it, so maybe just leave it as 0 > unless someone else has a strong opinion about it. I'll leave it for consistency then. - Derek __

Re: [FFmpeg-devel] [PATCH v3 4/5] avformat/mov: Add support for exporting Video Extension Usage info

2024-06-17 Thread Derek Buitenhuis
On 6/17/2024 9:07 PM, James Almer wrote: > nit: { 0, 1 }. If we set it to 0, 1 here, should it be set to that in the alloc function too? It'll be 0, 0 at alloc time due to use of av_mallocz. - Derek ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org

Re: [FFmpeg-devel] [PATCH v3 2/5] avutil/stereo3d: Fill out stereo info provided by Vision Pro files

2024-06-17 Thread Derek Buitenhuis
On 6/17/2024 9:03 PM, James Almer wrote: > Missing APIChanges entry (no need to resend, you can just amend before > pushing). Done locally for this and previous commit. - Derek ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org https://ffmpeg.org/ma

[FFmpeg-devel] [PATCH v3 5/5] avformat/mov: Add support for reading and exporting horizontal field of view

2024-06-17 Thread Derek Buitenhuis
These boxes are created by the Apple Vision Pro and the iPhone 15+ when capture for the Vision Pro is enabled. Based off of the swift API: * https://developer.apple.com/documentation/coremedia/kcmformatdescriptionextension_horizontalfieldofview Signed-off-by: Derek Buitenhuis

[FFmpeg-devel] [PATCH v3 4/5] avformat/mov: Add support for exporting Video Extension Usage info

2024-06-17 Thread Derek Buitenhuis
* https://developer.apple.com/documentation/videotoolbox/kvtcompressionpropertykey_heroeye Signed-off-by: Derek Buitenhuis --- libavformat/mov.c | 283 ++ 1 file changed, 283 insertions(+) diff --git a/libavformat/mov.c b/libavformat/mov.c index 9016cd5ad0

[FFmpeg-devel] [PATCH v3 3/5] fftools/ffprobe: Print more Stereo 3D info from side data

2024-06-17 Thread Derek Buitenhuis
Signed-off-by: Derek Buitenhuis --- fftools/ffprobe.c| 8 tests/ref/fate/matroska-spherical-mono | 2 ++ tests/ref/fate/matroska-spherical-mono-remux | 4 tests/ref/fate/matroska-stereo_mode | 8 tests/ref/fate/matroska-vp8-alpha

[FFmpeg-devel] [PATCH v3 2/5] avutil/stereo3d: Fill out stereo info provided by Vision Pro files

2024-06-17 Thread Derek Buitenhuis
://developer.apple.com/documentation/videotoolbox/kvtcompressionpropertykey_horizontaldisparityadjustment * https://developer.apple.com/documentation/coremedia/kcmformatdescriptionextension_horizontalfieldofview Signed-off-by: Derek Buitenhuis --- libavutil/stereo3d.c | 52

[FFmpeg-devel] [PATCH v3 1/5] avutil/spherical: Add more spherical types

2024-06-17 Thread Derek Buitenhuis
These originate from the Apple Vision Pro, and are documented here: https://developer.apple.com/documentation/coremedia/cmprojectiontype Signed-off-by: Derek Buitenhuis --- libavutil/spherical.c | 5 + libavutil/spherical.h | 16 libavutil/version.h | 2 +- 3 files

[FFmpeg-devel] [PATCH v3 0/5] Vision Pro Spatial Data

2024-06-17 Thread Derek Buitenhuis
Changes since v2: * horizontal display adjustment is now a rational Derek Buitenhuis (5): avutil/spherical: Add more spherical types avutil/stereo3d: Fill out stereo info provided by Vision Pro files fftools/ffprobe: Print more Stereo 3D info from side data avformat/mov: Add support for

Re: [FFmpeg-devel] [PATCH v2 2/5] avutil/stereo3d: Fill out stereo info provided by Vision Pro files

2024-06-17 Thread Derek Buitenhuis
On 6/17/2024 7:09 PM, James Almer wrote: > No, it's av_d2q(), av_q2d(), and av_rescale() as needed. Same as we do > for Mastering Display and Ambient Viewing Environment Metadata. > The reason to use AVRational is that in this specific spec the values > have a denominator of 1, but in others

Re: [FFmpeg-devel] [PATCH v2 2/5] avutil/stereo3d: Fill out stereo info provided by Vision Pro files

2024-06-17 Thread Derek Buitenhuis
On 6/17/2024 5:53 PM, James Almer wrote: > Maybe this should be an AVRational then. While that is probably 'more correct', it does mean that in 100% places this could be used, it'll have to be converted back to the -1 to 1 range. Is there a simple way to do that with an AVRational that doe

Re: [FFmpeg-devel] [PATCH v2 4/5] avformat/mov: Add support for exporting Video Extension Usage info

2024-06-17 Thread Derek Buitenhuis
On 6/17/2024 2:41 PM, Derek Buitenhuis wrote: > Signed-off-by: Derek Buitenhuis > --- > libavformat/mov.c | 279 ++ > 1 file changed, 279 insertions(+) Replaced by v3. - Derek ___ ffmpeg-devel

[FFmpeg-devel] [PATCH v3 4/5] avformat/mov: Add support for exporting Video Extension Usage info

2024-06-17 Thread Derek Buitenhuis
* https://developer.apple.com/documentation/videotoolbox/kvtcompressionpropertykey_heroeye Signed-off-by: Derek Buitenhuis --- Only difference from v2 is s/RECTANGULAR/RECTILINEAR/. --- libavformat/mov.c | 279 ++ 1 file changed, 279 insertions(+) diff

[FFmpeg-devel] [PATCH v2 5/5] avformat/mov: Add support for reading and exporting horizontal field of view

2024-06-17 Thread Derek Buitenhuis
These boxes are created by the Apple Vision Pro and the iPhone 15+ when capture for the Vision Pro is enabled. Based off of the swift API: * https://developer.apple.com/documentation/coremedia/kcmformatdescriptionextension_horizontalfieldofview Signed-off-by: Derek Buitenhuis

[FFmpeg-devel] [PATCH v2 4/5] avformat/mov: Add support for exporting Video Extension Usage info

2024-06-17 Thread Derek Buitenhuis
* https://developer.apple.com/documentation/videotoolbox/kvtcompressionpropertykey_heroeye Signed-off-by: Derek Buitenhuis --- libavformat/mov.c | 279 ++ 1 file changed, 279 insertions(+) diff --git a/libavformat/mov.c b/libavformat/mov.c index 9016cd5ad0

[FFmpeg-devel] [PATCH v2 3/5] fftools/ffprobe: Print more Stereo 3D info from side data

2024-06-17 Thread Derek Buitenhuis
Signed-off-by: Derek Buitenhuis --- fftools/ffprobe.c| 8 tests/ref/fate/matroska-spherical-mono | 2 ++ tests/ref/fate/matroska-spherical-mono-remux | 4 tests/ref/fate/matroska-stereo_mode | 8 tests/ref/fate/matroska-vp8-alpha

[FFmpeg-devel] [PATCH v2 2/5] avutil/stereo3d: Fill out stereo info provided by Vision Pro files

2024-06-17 Thread Derek Buitenhuis
://developer.apple.com/documentation/videotoolbox/kvtcompressionpropertykey_horizontaldisparityadjustment * https://developer.apple.com/documentation/coremedia/kcmformatdescriptionextension_horizontalfieldofview Signed-off-by: Derek Buitenhuis --- libavutil/stereo3d.c | 52

[FFmpeg-devel] [PATCH v2 1/5] avutil/spherical: Add more spherical types

2024-06-17 Thread Derek Buitenhuis
These originate from the Apple Vision Pro, and are documented here: https://developer.apple.com/documentation/coremedia/cmprojectiontype Signed-off-by: Derek Buitenhuis --- libavutil/spherical.c | 5 + libavutil/spherical.h | 16 libavutil/version.h | 2 +- 3 files

[FFmpeg-devel] [PATCH v2 0/5] Vision Pro Spatial Data

2024-06-17 Thread Derek Buitenhuis
* All comments applied from previous review. * Renamed rectangular to more industry standard rectilinear. Derek Buitenhuis (5): avutil/spherical: Add more spherical types avutil/stereo3d: Fill out stereo info provided by Vision Pro files fftools/ffprobe: Print more Stereo 3D info from side

Re: [FFmpeg-devel] [PATCH 1/5] avutil/spherical: Add more spherical types

2024-06-17 Thread Derek Buitenhuis
On 6/14/2024 2:03 PM, Derek Buitenhuis wrote: > I think setting it to 0 (AV_STEREO3D_2D) is correct rather than this (and is > already > done), and I don't think we need to reorder. > > I had discussed this with Vittorio and it was decided that it was good to > disambi

Re: [FFmpeg-devel] [PATCH 3/5] fftools/ffprobe: Print more Stereo 3D info from side data

2024-06-14 Thread Derek Buitenhuis
On 6/11/2024 7:43 PM, Derek Buitenhuis wrote: > Perhaps if type!=2D? Changed locally to this and updated the test. - Derek ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org https://ffmpeg.org/mailman/listinfo/ffmpeg-devel To unsubscribe, vi

Re: [FFmpeg-devel] [PATCH 1/5] avutil/spherical: Add more spherical types

2024-06-14 Thread Derek Buitenhuis
On 6/11/2024 7:44 PM, James Almer wrote: > This should ideally be the enum with value 0, but until next major when > such a change can happen, it would be IMO a good idea if you set > spherical->projection to AV_SPHERICAL_RECTANGULAR in av_spherical_alloc(). I think setting it to 0 (AV_STEREO3D_

Re: [FFmpeg-devel] [PATCH 3/5] fftools/ffprobe: Print more Stereo 3D info from side data

2024-06-11 Thread Derek Buitenhuis
On 6/11/2024 7:09 PM, Michael Niedermayer wrote: > side_data_type=Stereo 3D > type=2D > inverted=0 > +view=packed > +primary_eye=none Hmm. This does make me think we need a better way to print 'view' and 'inverted' in FFprobe. Perhaps if type!=2D? - Derek

Re: [FFmpeg-devel] [PATCH 4/5] avformat/mov: Add support for exporting Video Extension Usage info

2024-06-10 Thread Derek Buitenhuis
On 6/10/2024 8:38 PM, James Almer wrote: >> +remaining = atom.size; >> +while (remaining > 0) { > > Maybe this loop should call mov_read_default, with proj and eyes added > to mov_default_parse_table[]. Although i don't know if eyes may show up > as child for other parent boxes or not. >

[FFmpeg-devel] [PATCH 5/5] avformat/mov: Add support for reading and exporting horizontal field of view

2024-06-10 Thread Derek Buitenhuis
These boxes are created by the Apple Vision Pro and the iPhone 15+ when capture for the Vision Pro is enabled. Based off of the swift API: * https://developer.apple.com/documentation/coremedia/kcmformatdescriptionextension_horizontalfieldofview Signed-off-by: Derek Buitenhuis

[FFmpeg-devel] [PATCH 4/5] avformat/mov: Add support for exporting Video Extension Usage info

2024-06-10 Thread Derek Buitenhuis
* https://developer.apple.com/documentation/videotoolbox/kvtcompressionpropertykey_heroeye Signed-off-by: Derek Buitenhuis --- libavformat/mov.c | 279 ++ 1 file changed, 279 insertions(+) diff --git a/libavformat/mov.c b/libavformat/mov.c index 160e9626d7

[FFmpeg-devel] [PATCH 3/5] fftools/ffprobe: Print more Stereo 3D info from side data

2024-06-10 Thread Derek Buitenhuis
Signed-off-by: Derek Buitenhuis --- fftools/ffprobe.c | 8 1 file changed, 8 insertions(+) diff --git a/fftools/ffprobe.c b/fftools/ffprobe.c index 2d38e5dfdc..082cec8a64 100644 --- a/fftools/ffprobe.c +++ b/fftools/ffprobe.c @@ -2544,6 +2544,14 @@ static void print_pkt_side_data

[FFmpeg-devel] [PATCH 2/5] avutil/stereo3d: Fill out stereo info provided by Vision Pro files

2024-06-10 Thread Derek Buitenhuis
://developer.apple.com/documentation/videotoolbox/kvtcompressionpropertykey_horizontaldisparityadjustment * https://developer.apple.com/documentation/coremedia/kcmformatdescriptionextension_horizontalfieldofview Signed-off-by: Derek Buitenhuis --- libavutil/stereo3d.c | 52

[FFmpeg-devel] [PATCH 1/5] avutil/spherical: Add more spherical types

2024-06-10 Thread Derek Buitenhuis
These originate from the Apple Vision Pro, and are documented here: https://developer.apple.com/documentation/coremedia/cmprojectiontype Signed-off-by: Derek Buitenhuis --- libavutil/spherical.c | 3 +++ libavutil/spherical.h | 16 2 files changed, 19 insertions(+) diff

[FFmpeg-devel] [PATCH 0/5] Apple Spatial Metadata

2024-06-10 Thread Derek Buitenhuis
* I like the idea that the 'dadj' box sits inside a 'cmfy' box, it seems very wholesome. [1] https://developer.apple.com/av-foundation/HEVC-Stereo-Video-Profile.pdf Derek Buitenhuis (5): avutil/spherical: Add more spherical types avutil/stereo3d: Fill out stereo info

Re: [FFmpeg-devel] [RFC] STF 2025

2024-05-17 Thread Derek Buitenhuis
On 5/17/2024 7:50 PM, Michael Niedermayer wrote: > The people on the booth are predominantly male. Similarly ffmpeg-devel > is predominantly male. More gender diversity would be good. That I can agree with. - Derek ___ ffmpeg-devel mailing list ffmpeg-d

Re: [FFmpeg-devel] [RFC] STF 2025

2024-05-17 Thread Derek Buitenhuis
On 5/17/2024 2:49 PM, Michael Niedermayer wrote: > Also we need more cute girls on these events, everything i hear > its 100% male geeks/hackers. This is gross and sexist. - Derek ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org https://ffmpeg.org/

[FFmpeg-devel] [PATCH] fftools/ffprobe: Avoid overflow when calculating DAR

2024-05-03 Thread Derek Buitenhuis
Both the codecpar's width and height, and the SAR num and den are ints, which can overflow. Cast to int64_t, which is what av_reduce takes. Without this, occasionally, display_aspect_ratio can be negative in ffprobe's -show_stream output. Signed-off-by: Derek Buitenhuis --- fftools

Re: [FFmpeg-devel] [PATCH v3 0/2] HTTP Retry-After Support

2024-04-26 Thread Derek Buitenhuis
On 4/25/2024 9:22 PM, Martin Storsjö wrote: > Thanks, these patches LGTM. Pushed, thanks. - Derek ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org https://ffmpeg.org/mailman/listinfo/ffmpeg-devel To unsubscribe, visit link above, or email ffmpeg-d

Re: [FFmpeg-devel] [EXTERNAL] Request for Official GitHub Mirror of rtmpdump for Enhanced Security

2024-04-25 Thread Derek Buitenhuis
Hi, Replies inline. On 4/26/2024 12:10 AM, Javier Matos Denizac via ffmpeg-devel wrote: > Actually, I noticed that you publish release tarballs -> > http://rtmpdump.mplayerhq.hu/download/, but I don’t see a release tarball for > 2.4. Would y’all be willing to publish a release for 2.4 and maybe

[FFmpeg-devel] [PATCH v3 2/2] avformat/http: Add support for Retry-After header

2024-04-25 Thread Derek Buitenhuis
, chances of success are very unlikely. Some references: * https://datatracker.ietf.org/doc/html/rfc6585 * https://datatracker.ietf.org/doc/html/rfc7231#section-7.1.3 This adds an AVOption to respect that header. Signed-off-by: Derek Buitenhuis --- doc/protocols.texi| 5 + libavformat

[FFmpeg-devel] [PATCH v3 1/2] avformat/http: Rename parse_set_cookie_expiry_time to parse_http_date

2024-04-25 Thread Derek Buitenhuis
That is what it actually does, and it will be needed for more than the Expiry header soon. Signed-off-by: Derek Buitenhuis --- libavformat/http.c | 38 +++--- 1 file changed, 19 insertions(+), 19 deletions(-) diff --git a/libavformat/http.c b/libavformat/http.c

[FFmpeg-devel] [PATCH v3 0/2] HTTP Retry-After Support

2024-04-25 Thread Derek Buitenhuis
Changes since last set: * Updated commit message with RFC references. * Properly support Retry-After as both a date and integer number of seconds. I have tested this against both an HTTP-Date and seconds, and confirmed it to work. Derek Buitenhuis (2): avformat/http: Rename

Re: [FFmpeg-devel] [PATCH v2 0/9] HTTP rate limiting and retry improvements

2024-04-25 Thread Derek Buitenhuis
On 4/24/2024 8:43 PM, Derek Buitenhuis wrote: > Applied all your comments. > > Will wait a day and then push if no others appear. Pushed all except Retry-After support, which I need to change. While adding RFC references I noticed it can be in seconds, *or* a date... fun. Sending a

Re: [FFmpeg-devel] Request for Official GitHub Mirror of rtmpdump for Enhanced Security

2024-04-25 Thread Derek Buitenhuis
On 4/23/2024 10:46 PM, Michael Niedermayer wrote: > Can you elaborate what the problem is ? > I would have thought https://git.ffmpeg.org/rtmpdump.git > is secure I have to assume he means SHA-256, and not SHA-512. git apparently supports using SHA-256 instead of SHA-1 hashes, but support does no

Re: [FFmpeg-devel] [PATCH v2 0/9] HTTP rate limiting and retry improvements

2024-04-24 Thread Derek Buitenhuis
On 4/24/2024 12:13 PM, Martin Storsjö wrote: > I had a look over this patchset, and I had a handful of minor comments, > but overall, the patchset seems fine to me. Thanks! Applied all your comments. Will wait a day and then push if no others appear. - Derek

Re: [FFmpeg-devel] [PATCH v2 6/9] avformat/http: Add options to set the max number of connection retries

2024-04-24 Thread Derek Buitenhuis
On 4/24/2024 12:08 PM, Martin Storsjö wrote: > Minor inconsistency; the corresponding variable in the other function was > called conn_attempts, as a plural. Renamed to the plural version. - Derek ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org h

Re: [FFmpeg-devel] [PATCH v2 2/9] avformat/http: Use AVERROR_HTTP_TOO_MANY_REQUESTS

2024-04-24 Thread Derek Buitenhuis
On 4/24/2024 11:58 AM, Martin Storsjö wrote: > This function seems to handle both the literal status codes, like 429, and > also AVERROR style error codes, as when called from handle_http_errors, so > perhaps it would be good for consistency to add the AVERROR here too. Good catch. Added. - Der

Re: [FFmpeg-devel] [PATCH v2 2/9] avformat/http: Use AVERROR_HTTP_TOO_MANY_REQUESTS

2024-04-24 Thread Derek Buitenhuis
On 4/24/2024 11:53 AM, Martin Storsjö wrote: > Typo in the commit message Fixed locally. - Derek ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org https://ffmpeg.org/mailman/listinfo/ffmpeg-devel To unsubscribe, visit link above, or email ffmpeg-de

Re: [FFmpeg-devel] [PATCH v2 4/9] avformat/http: Add support for Retry-After header

2024-04-24 Thread Derek Buitenhuis
On 4/24/2024 12:06 PM, Martin Storsjö wrote: > Is this feature standardized in a RFC, or is it some other spec somewhere? > I think it would be nice with a link to a spec in the commit message here. It is in the RFC for 429 I noted in the commit I added that: RFC6585. It is also probably in the 5

Re: [FFmpeg-devel] [PATCH v2 1/9] avutil/error: Add HTTP 429 Too Many Requests AVERROR code

2024-04-22 Thread Derek Buitenhuis
On 4/22/2024 3:25 PM, Derek Buitenhuis wrote: > +{ ERROR_TAG(HTTP_TOO_MANY_REQUESTS), "Server returned 404 Too Many > Requests" }, Derp. Change locally to "Server returned 429 Too Many Requests". - Derek ___ ffmpeg

Re: [FFmpeg-devel] [PATCH 8/6] doc/protocols: Fill in missing HTTP options

2024-04-22 Thread Derek Buitenhuis
On 4/16/2024 6:13 PM, Stefano Sabatini wrote: >> +@item metadata >> +An exported dictionary containing Icecast metadata from the bitstream, if >> present. >> +Only useful with the C API. > > Probably best to use impersonal verbal mode: > Set an exported ... This is not quite right. This is not a

[FFmpeg-devel] [PATCH v2 7/9] avformat/http: Add option to limit total reconnect delay

2024-04-22 Thread Derek Buitenhuis
accomplished by the the existing reconnect_delay_max default of 120. Signed-off-by: Derek Buitenhuis --- libavformat/http.c| 12 ++-- libavformat/version.h | 2 +- 2 files changed, 11 insertions(+), 3 deletions(-) diff --git a/libavformat/http.c b/libavformat/http.c index 06bd3e340e

[FFmpeg-devel] [PATCH v2 6/9] avformat/http: Add options to set the max number of connection retries

2024-04-22 Thread Derek Buitenhuis
Not every use case benefits from setting retries in terms of the backoff. Signed-off-by: Derek Buitenhuis --- libavformat/http.c| 12 +--- libavformat/version.h | 2 +- 2 files changed, 10 insertions(+), 4 deletions(-) diff --git a/libavformat/http.c b/libavformat/http.c index

[FFmpeg-devel] [PATCH v2 5/9] avformat/http: Rename attempts to auth_attempts

2024-04-22 Thread Derek Buitenhuis
This accurately reflects what it does, as per e75bbcf493aeb549d04c56f49406aeee3950d93b. Signed-off-by: Derek Buitenhuis --- libavformat/http.c | 16 1 file changed, 8 insertions(+), 8 deletions(-) diff --git a/libavformat/http.c b/libavformat/http.c index 5ed481b63a

[FFmpeg-devel] [PATCH v2 9/9] doc/protocols: Fill in missing HTTP options

2024-04-22 Thread Derek Buitenhuis
Signed-off-by: Derek Buitenhuis --- doc/protocols.texi | 35 ++- 1 file changed, 34 insertions(+), 1 deletion(-) diff --git a/doc/protocols.texi b/doc/protocols.texi index 5ce1ddc8f4..ed70af4b33 100644 --- a/doc/protocols.texi +++ b/doc/protocols.texi @@ -492,6

[FFmpeg-devel] [PATCH v2 8/9] doc/protocols: Re-order HTTP options to match http.c order

2024-04-22 Thread Derek Buitenhuis
This makes the list easier to maintain. Signed-off-by: Derek Buitenhuis --- doc/protocols.texi | 112 ++--- 1 file changed, 56 insertions(+), 56 deletions(-) diff --git a/doc/protocols.texi b/doc/protocols.texi index f54600b846..5ce1ddc8f4 100644 --- a

[FFmpeg-devel] [PATCH v2 4/9] avformat/http: Add support for Retry-After header

2024-04-22 Thread Derek Buitenhuis
unlikely. This adds an AVOption to respect that header. Signed-off-by: Derek Buitenhuis --- libavformat/http.c| 12 libavformat/version.h | 2 +- 2 files changed, 13 insertions(+), 1 deletion(-) diff --git a/libavformat/http.c b/libavformat/http.c index e7603037f4..5ed481b63a

[FFmpeg-devel] [PATCH v2 3/9] avformat/http: Don't bail on parsing headers on "bad" HTTP codes

2024-04-22 Thread Derek Buitenhuis
Many "bad" HTTP codes like 429 and 503 may include important info in their headers. Also, in general, there is no purpose in bailing here. Signed-off-by: Derek Buitenhuis --- libavformat/http.c | 25 - 1 file changed, 20 insertions(+), 5 deletions(-) di

[FFmpeg-devel] [PATCH v2 2/9] avformat/http: Use AVERROR_HTTP_TOO_MANY_REQUESTS

2024-04-22 Thread Derek Buitenhuis
Added in thep previous commit. Signed-off-by: Derek Buitenhuis --- libavformat/http.c | 6 ++ 1 file changed, 6 insertions(+) diff --git a/libavformat/http.c b/libavformat/http.c index ed20359552..bbace2694f 100644 --- a/libavformat/http.c +++ b/libavformat/http.c @@ -286,6 +286,7

[FFmpeg-devel] [PATCH v2 1/9] avutil/error: Add HTTP 429 Too Many Requests AVERROR code

2024-04-22 Thread Derek Buitenhuis
This is a common error code from e.g. CDNs or cloud storage, and it is useful to be able to handle it differently to a generic 4XX code. Its source is RFC6585. Signed-off-by: Derek Buitenhuis --- libavutil/error.c | 1 + libavutil/error.h | 1 + libavutil/version.h | 2 +- 3 files changed

[FFmpeg-devel] [PATCH v2 0/9] HTTP rate limiting and retry improvements

2024-04-22 Thread Derek Buitenhuis
d use strtoull, per James' review. * Added docs, as per Stefano's reviews./ * Added a new option to limit the total reconnect delay. * Unfortunate, but HTTP connection management is messy business. Original set link: https://ffmpeg.org/pipermail/ffmpeg-devel/2024-April/325

Re: [FFmpeg-devel] Mailinglist conduct

2024-04-17 Thread Derek Buitenhuis
On 4/17/2024 8:33 PM, James Almer wrote: >>> This was an unnecessary personal attack, please don't do that again. Repeat >>> offense may result in temporary bans on the mailinglist and/or IRC. Please >>> keep it civil. >> >> That does not make it untrue, however. > > It's ok to have opinions. But

Re: [FFmpeg-devel] Mailinglist conduct

2024-04-17 Thread Derek Buitenhuis
On 4/17/2024 8:00 PM, Ronald S. Bultje wrote: > This was an unnecessary personal attack, please don't do that again. Repeat > offense may result in temporary bans on the mailinglist and/or IRC. Please > keep it civil. That does not make it untrue, however. I would have preferred a ban. - Derek _

Re: [FFmpeg-devel] FFmpeg table at NAB

2024-04-17 Thread Derek Buitenhuis
On 4/17/2024 2:54 PM, James Almer wrote: > But why were there GPAC people at the FFmpeg booth? > And i don't think a single person should represent the project in these > conferences to begin with. All this should go through the GA, including > who funds it and how. That is a good question, and

Re: [FFmpeg-devel] FFmpeg table at NAB

2024-04-17 Thread Derek Buitenhuis
On 4/17/2024 12:21 AM, Devin Heitmueller wrote: > Hello all, [...] Yeah, this is exactly what I was screaming into the void about for literal months, no literally 0 response. Look for the thread: [FFmpeg-devel] FFmpeg at NAB 2024 It spans months. My last mail is particularily relevant to your

Re: [FFmpeg-devel] [PATCH 4/6 v2] avformat/http: Add support for Retry-After header

2024-04-16 Thread Derek Buitenhuis
On 4/15/2024 6:33 PM, Stefano Sabatini wrote: > missing doc/protocols.texi update Sent patch 7 and 8 to address this. - Derek ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org https://ffmpeg.org/mailman/listinfo/ffmpeg-devel To unsubscribe, visit l

[FFmpeg-devel] [PATCH 8/6] doc/protocols: Fill in missing HTTP options

2024-04-16 Thread Derek Buitenhuis
Signed-off-by: Derek Buitenhuis --- doc/protocols.texi | 30 ++ 1 file changed, 30 insertions(+) diff --git a/doc/protocols.texi b/doc/protocols.texi index 5ce1ddc8f4..60c6d831dd 100644 --- a/doc/protocols.texi +++ b/doc/protocols.texi @@ -492,6 +492,10 @@ contains

[FFmpeg-devel] [PATCH 7/6] doc/protocols: Re-order HTTP options to match http.c order

2024-04-16 Thread Derek Buitenhuis
This makes the list easier to maintain. Signed-off-by: Derek Buitenhuis --- doc/protocols.texi | 112 ++--- 1 file changed, 56 insertions(+), 56 deletions(-) diff --git a/doc/protocols.texi b/doc/protocols.texi index f54600b846..5ce1ddc8f4 100644 --- a

  1   2   3   4   5   6   7   8   9   10   >