Re: [FFmpeg-devel] [PATCH] lavc/libx265: flag as experimental

2024-10-17 Thread BUCCIANTINI Francesco via ffmpeg-devel
No, I strongly disagree. Marking a vital encoder as libx265 that thousands of people rely on as "experimental" after more than 10 years since its inclusion doesn't make any sense and would make more harm than good. Besides, the experimental flag is supposed to be used to indicate that the imple

[FFmpeg-devel] fix for defect #10531

2023-09-11 Thread Francesco Carusi
I attached a patch for defect [ #10531: vf_drawtext causing font rendering jitter after libharfbuz commit. ] Before the patch the width of a text line was measured up to the rightmost visible pixel of the last character. Pros: the right padding specified with the boxborderw parameter is exact

Re: [FFmpeg-devel] [PATCH] libavfilter/vf_drawtext: add letter_spacing as an evaluated parameter

2023-06-20 Thread Francesco Carusi
You can find an overview of the changes here: https://github.com/yethie/FFmpeg/blob/master/drawtext/CHANGES.md and the new code here: https://github.com/yethie/FFmpeg/blob/master/libavfilter/vf_drawtext.c On 19/06/2023 19:56, Mark Ren wrote: Ah alright I see. When might that push come throug

Re: [FFmpeg-devel] drawtext filter

2023-03-14 Thread Francesco Carusi
Can I help in any way in advancing this patch? On 03/02/2023 15:18, Francesco Carusi wrote: On 30/01/2023 13:19, Paul B Mahol wrote: On 1/30/23, Francesco Carusi wrote: On 28/01/2023 16:32, Paul B Mahol wrote: On 1/28/23, Francesco Carusi wrote: On 27/01/2023 18:31, Paul B Mahol wrote

Re: [FFmpeg-devel] drawtext filter

2023-02-03 Thread Francesco Carusi
On 30/01/2023 13:19, Paul B Mahol wrote: On 1/30/23, Francesco Carusi wrote: On 28/01/2023 16:32, Paul B Mahol wrote: On 1/28/23, Francesco Carusi wrote: On 27/01/2023 18:31, Paul B Mahol wrote: On 1/27/23, Francesco Carusi wrote: On 26/01/2023 17:37, Paul B Mahol wrote: On 1/26/23

Re: [FFmpeg-devel] drawtext filter

2023-01-30 Thread Francesco Carusi
On 28/01/2023 16:32, Paul B Mahol wrote: On 1/28/23, Francesco Carusi wrote: On 27/01/2023 18:31, Paul B Mahol wrote: On 1/27/23, Francesco Carusi wrote: On 26/01/2023 17:37, Paul B Mahol wrote: On 1/26/23, Francesco Carusi wrote: On 26/01/2023 14:21, Paul B Mahol wrote: On 1/26/23

Re: [FFmpeg-devel] drawtext filter

2023-01-28 Thread Francesco Carusi
On 27/01/2023 18:31, Paul B Mahol wrote: On 1/27/23, Francesco Carusi wrote: On 26/01/2023 17:37, Paul B Mahol wrote: On 1/26/23, Francesco Carusi wrote: On 26/01/2023 14:21, Paul B Mahol wrote: On 1/26/23, Francesco Carusi wrote: The drawtext reinit command is also used in the docs as

Re: [FFmpeg-devel] drawtext filter

2023-01-27 Thread Francesco Carusi
On 26/01/2023 17:37, Paul B Mahol wrote: On 1/26/23, Francesco Carusi wrote: On 26/01/2023 14:21, Paul B Mahol wrote: On 1/26/23, Francesco Carusi wrote: The drawtext reinit command is also used in the docs as an example for the sendcmd filter, so I thought it was fine to use commands in

Re: [FFmpeg-devel] drawtext filter

2023-01-26 Thread Francesco Carusi
On 26/01/2023 14:21, Paul B Mahol wrote: On 1/26/23, Francesco Carusi wrote: The drawtext reinit command is also used in the docs as an example for the sendcmd filter, so I thought it was fine to use commands in that way. In my opinion it is also a convenient way to modify multiple options at

Re: [FFmpeg-devel] drawtext filter

2023-01-26 Thread Francesco Carusi
? On 26/01/2023 11:50, Paul B Mahol wrote: On 1/26/23, Francesco Carusi wrote: Hi, I'm new to contributing to ffmpeg! I modified the drawtext filter to improve text rendering and add some features. You can find a high level description of the changes at this link: https://github.com/y

[FFmpeg-devel] drawtext filter

2023-01-26 Thread Francesco Carusi
@url{http://fribidi.org/}. +For more information about libharfbuzz, check: +@url{https://github.com/harfbuzz/harfbuzz}. + @section edgedetect Detect and draw edges. The filter uses the Canny Edge Detection algorithm. diff --git a/libavfilter/vf_drawtext.c b/libavfilter/vf_drawtext.c index

Re: [FFmpeg-devel] proposal for a minor change in the behavior of the drawtext filter

2022-04-26 Thread Francesco Carusi
As Michael Koch mentioned in his message, also the meaning of the 'y' parameter may be changed to improve the filter interface, but it would also break its compatibility with the current implementation. Thanks for reading! On 24/04/2022 12:07, Nicolas George wrote: Francesco Carusi (

Re: [FFmpeg-devel] proposal for a minor change in the behavior of the drawtext filter

2022-04-22 Thread Francesco Carusi
uot;. If I keep the current default, than what you're asking can be obtained using: y = - top_a I'm new to the ffmpeg devs community so I'm not sure whether it is preferred to keep backward compatibility or to change the behavior in the name of a better interface. What do

Re: [FFmpeg-devel] proposal for a minor change in the behavior of the drawtext filter

2022-04-22 Thread Francesco Carusi
On 22/04/2022 12:30, Michael Niedermayer wrote: On Fri, Apr 22, 2022 at 12:01:31PM +0200, Francesco Carusi wrote: Hello ffmpeg team, I'm working on an enhanced version of the drawtext filter and would like to discuss with you about a minor change in its behavior related to line sp

[FFmpeg-devel] proposal for a minor change in the behavior of the drawtext filter

2022-04-22 Thread Francesco Carusi
Hello ffmpeg team, I'm working on an enhanced version of the drawtext filter and would like to discuss with you about a minor change in its behavior related to line spacing management. In the current implementation the space between two lines of text is set equal to the height of the highest g

[FFmpeg-devel] Stream description

2018-02-21 Thread Francesco, Cuzzocrea
he Packet data buffer, I receive an error. How can I describe my stream as per field ? regards -- /***\ *Ing. Francesco Cuzzocrea *company:BLT Italia srlweb: http://www.blt.it *address:via Rosselli, 91 city: Lido

Re: [FFmpeg-devel] [PATCH] JPEG200 encoding : added option for changing default codeblock size

2017-08-30 Thread Francesco, Cuzzocrea
Height exponent of the code-blocks for the SPcod and SPcoc parameters) code-block exponent width  xcb and height ycb are defined as: xcb = value + 2   and  ycb = value+2 so xcb+ycb <=12 is the correct check. I will correct the patch. Francesco On 15/08/2017 23.57, Moritz Barsnick wrote:

[FFmpeg-devel] [PATCH] JPEG200 encoding : added option for changing default codeblock size

2017-08-07 Thread francesco
From: Francesco Cuzzocrea HI I think this time I've made correctly a patch. As in previous mail, I added option for changing codeblock size. I've inserted check on exponent sum as for ISO/IEC FCD 15444-1 --- libavcodec/j2kenc.c | 10 -- 1 file changed, 8 insertions(+), 2

Re: [FFmpeg-devel] [PATCH] JPEG2000 encoding with variable codeblock size

2017-08-04 Thread Francesco, Cuzzocrea
Sorry. I intend 32x128 On 04/08/2017 10.21, Francesco, Cuzzocrea wrote: The standard ISO/IEC FCD15444-1 specify limits for the exponent values and their sum. That is codeblock size xcb and ycb are defined as: xcb = 2^(xvalue+2) and ycb = 2^(yvalue+2) with values that ranges from a

[FFmpeg-devel] [PATCH 2/2] Inserted check on codeblock size, added limits on codeblock size Modified according to Niedermayer suggestions

2017-08-04 Thread francesco
From: Francesco Cuzzocrea --- libavcodec/j2kenc.c | 30 +- 1 file changed, 17 insertions(+), 13 deletions(-) diff --git a/libavcodec/j2kenc.c b/libavcodec/j2kenc.c index 1bd4fbd..20dda87 100644 --- a/libavcodec/j2kenc.c +++ b/libavcodec/j2kenc.c @@ -1121,8 +1121,6

Re: [FFmpeg-devel] [PATCH] JPEG2000 encoding with variable codeblock size

2017-08-04 Thread Francesco, Cuzzocrea
( xvalue + yvalue <=12). So 64x128 are allowed because xvalue = 4 and yvalue = 5. On 03/08/2017 19.24, Aaron Boxer wrote: On Aug 3, 2017 9:58 AM, wrote: From: Francesco Cuzzocrea Hi to all I've made some simple changes tha allow encoding with variable codeblock size. Default v

[FFmpeg-devel] [PATCH] JPEG2000 encoding with variable codeblock size

2017-08-03 Thread francesco
From: Francesco Cuzzocrea Hi to all I've made some simple changes tha allow encoding with variable codeblock size. Default value are the same as previous (16X16) but now setting them to 64x128 make generated codestream compatible with Analog Devices ADV212 video codec. --- libav