On Wed, Sep 25, 2019 at 09:26:58PM +0530, Gyan wrote:
> Ok, the formatting is fine except for one. See below.
> 
> On 25-09-2019 03:56 AM, lance.lmw...@gmail.com wrote:
> >From: Limin Wang <lance.lmw...@gmail.com>
> >
> >Reviewed-by: Gyan <ffm...@gyani.pro>
> >Signed-off-by: Limin Wang <lance.lmw...@gmail.com>
> >---
> >  doc/filters.texi | 42 +++++++++++++++++++++++++++++++++++++++++-
> >  1 file changed, 41 insertions(+), 1 deletion(-)
> >
> >diff --git a/doc/filters.texi b/doc/filters.texi
> >index e41384a..a8342c0 100644
> >--- a/doc/filters.texi
> >+++ b/doc/filters.texi
> >@@ -19057,6 +19057,13 @@ terminates. Default value is 0.
> >  @itemize
> >  @item
> >  Display 4 inputs into 2x2 grid.
> >+
> >+Below is the 4 inputs position:
> >+@example
> >+input1(0, 0)  | input3(w0, 0)
> >+input2(0, h0) | input4(w0, h0)
> >+@end example
> >+
> >  Note that if inputs are of different sizes unused gaps might appear,
> >  as not all of output video is used.
> >  @example
> >@@ -19065,6 +19072,12 @@ xstack=inputs=4:layout=0_0|0_h0|w0_0|w0_h0
> >  @item
> >  Display 4 inputs into 1x4 grid.
> >+
> >+Below is the 4 inputs position:
> >+@example
> >+input1(0, 0) | input2(0, h0) | input3(0, h0+h1) | input4(0, h0+h1+h2)
> Since this is 1 column x 4 rows, these should be different lines
> without the separator.
> 
> >+@end example
> >+
> >  Note that if inputs are of different sizes unused gaps might appear,
> >  as not all of output video is used.
> >  @example
> >@@ -19073,11 +19086,38 @@ xstack=inputs=4:layout=0_0|0_h0|0_h0+h1|0_h0+h1+h2
> >  @item
> >  Display 9 inputs into 3x3 grid.
> >+
> >+Below is the 9 inputs position:
> >+@example
> >+input1(0, 0)       | input4(w0, 0)      | input7(w0+w1, 0)
> >+input2(0, h0)      | input5(w0, h0)     | input8(w0+w1, h0)
> >+input3(0, h0+h1)   | input6(w0, h0+h1)  | input9(w0+w1, h0+h1)
> 
> Shouldn't the third column be at x = w0+w3? w1 is width of input2, right?
> 
> >+@end example
> >+
> >  Note that if inputs are of different sizes unused gaps might appear,
> >  as not all of output video is used.
> >  @example
> >-xstack=inputs=9:layout=w3_0|w3_h0+h2|w3_h0|0_h4|0_0|w3+w1_0|0_h1+h2|w3+w1_h0|w3+w1_h1+h2
> >+xstack=inputs=9:layout=0_0|0_h0|0_h0+h1|w0_0|w0_h0|w0_h0+h1|w0+w1_0|w0+w1_h0|w0+w1_h0+h1
> >  @end example
> >+
> >+@item
> >+Display 16 inputs into 4x4 grid.
> >+
> >+Below is the 16 inputs position:
> >+@example
> >+input1(0, 0)       | input5(w0, 0)       | input9 (w0+w1, 0)       | 
> >input13(w0+w1+w2, 0)
> >+input2(0, h0)      | input6(w0, h0)      | input10(w0+w1, h0)      | 
> >input14(w0+w1+w2, h0)
> >+input3(0, h0+h1)   | input7(w0, h0+h1)   | input11(w0+w1, h0+h1)   | 
> >input15(w0+w1+w2, h0+h1)
> >+input4(0, h0+h1+h2)| input8(w0, h0+h1+h2)| input12(w0+w1, h0+h1+h2)| 
> >input16(w0+w1+w2, h0+h1+h2)
> Same as above.
> 
> You should also note that when videos are of different sizes, the
> videos may overlap e.g. if height of input5 is greater than input1,
> then input6 will be overlaid on top of part of input5.
I have tested with different width videos, it's better to same width for every 
column.
Please check my last update version, please ignore the first one, it's wrong 
version.

> 
> Gyan
> _______________________________________________
> ffmpeg-devel mailing list
> ffmpeg-devel@ffmpeg.org
> https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
> 
> To unsubscribe, visit link above, or email
> ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".
_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel

To unsubscribe, visit link above, or email
ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".

Reply via email to