Opened 5 years ago

Closed 23 months ago

Last modified 23 months ago

#8128 closed defect (invalid)

geq filter, documentation

Reported by: Michael Koch Owned by: Gyan
Priority: minor Component: documentation
Version: git-master Keywords: geq
Cc: Blocked By:
Blocking: Reproduced by developer: no
Analyzed by developer: no

Description

Summary of the bug:

In the documentation for the "geq" filter all x and y must be replaced by capital letters X and Y, in the following lines. In the example section it's all written correct. It doesn't work with lower case.

p(x, y)

Return the value of the pixel at location (x,y) of the current plane.

lum(x, y)

Return the value of the pixel at location (x,y) of the luminance plane.

cb(x, y)

Return the value of the pixel at location (x,y) of the blue-difference chroma plane. Return 0 if there is no such plane.

cr(x, y)

Return the value of the pixel at location (x,y) of the red-difference chroma plane. Return 0 if there is no such plane.

r(x, y)
g(x, y)
b(x, y)

Return the value of the pixel at location (x,y) of the red/green/blue component. Return 0 if there is no such component.

alpha(x, y)

Return the value of the pixel at location (x,y) of the alpha plane. Return 0 if there is no such plane.

For functions, if x and y are outside the area, the value will be automatically clipped to the closer edge.

Change History (4)

comment:1 by Carl Eugen Hoyos, 5 years ago

Keywords: geq added

Please send your patch - made with git format-patch - to the FFmpeg development mailing list.

comment:2 by Carl Eugen Hoyos, 5 years ago

Priority: normalminor

comment:3 by Gyan, 5 years ago

Owner: set to Gyan
Status: newopen

comment:4 by Gyan, 23 months ago

Resolution: invalid
Status: openclosed

The lower case represents variables, not constants

i.e.

p(x,y) means user can supply p(64,24) to reference the value of pixel at X=64,Y=24.
p(X,Y) is a way to self-reference the pixel being evaluated.

Last edited 23 months ago by Gyan (previous) (diff)
Note: See TracTickets for help on using tickets.