Commit 979bc7b
committed
Show the clamp comparison as a picture, since the table alone asks people to take it on trust
The decode comparison was three numbers in a table. The figure carries the same measurement in a form someone
can check by eye: one LTX-2.5 latent, one frame, the decode path as the only variable. A patch enlarged from
both, 11606 distinct levels inside a 0.1-wide band from the stock decode against 584790 from ours, with the
stepping visible in one and absent in the other. A map of exactly which samples the clamp removes, 0.48% above
white and 0.46% below black. A difference histogram, and a scanline through the patch where the two decodes
track each other everywhere the clamp does not bite, which is the part that says nothing else moved.
Its numbers stand on their own: ranges 0.0000 to 1.0000 for the stock path against -0.0352 to +1.0408 for
ours, over 121 frames at 1280x704.
A second figure covering the write path was left out on purpose. It labels our ProRes output "12-bit", which
is the claim this release corrected: ffmpeg encodes ProRes at 10-bit whatever the format is nominally worth.
Its other numbers are sound, and its point survives without the label, but shipping a figure that repeats a
mistake the docs just fixed is worse than shipping no figure. It can be redrawn when there is a script to
redraw it with.1 parent bef46d2 commit 979bc7b
2 files changed
Lines changed: 11 additions & 0 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
437 | 437 | | |
438 | 438 | | |
439 | 439 | | |
| 440 | + | |
| 441 | + | |
| 442 | + | |
| 443 | + | |
| 444 | + | |
| 445 | + | |
| 446 | + | |
| 447 | + | |
| 448 | + | |
| 449 | + | |
| 450 | + | |
440 | 451 | | |
441 | 452 | | |
442 | 453 | | |
| |||
Loading
0 commit comments