[qcms] Use a static table in build_output_lut to invert para curves

[qcms] Use a static table in build_output_lut to invert para curves

Instead of allocating the uint16_t output table, use a static table
gamma_table_uint[256], to match the code in compute_precache().

Minor: adjust the fprintf text in the qcms_tests_output_trc test to
clarify the test output.

TBR=radu.velea@intel.com
BUG=600338
Committed: https://crrev.com/df1527ec51ed0079b792b260340918c9abfe551c
Cr-Commit-Position: refs/heads/master@{#390583}

[Noel Gordon, 2016-04-29 01:03:51]

Description was changed from

==========
boo


Merge branch 'master' of https://chromium.googlesource.com/chromium/src


[qcms] Make build_output_lut output 4096 points for parametric curves

build_output_lut inverts a gamma table. Make it return 4096
points for para curves to provide better accuracy.

Update the test to down-sample the output curve data to the
size of the input gamma table. Write the input and output
curve data results to a CSV output file.

Remove the gamma table size asserts(): the input and output
gamma table sizes of para curves might not be equal.

BUG=600338

patch from issue 1923873002 at patchset 140001
(http://crrev.com/1923873002#ps140001)
==========

to

==========
[qcms] Use a static table in build_output_lut to invert para curves

Instead of allocating the uint16_t output table, use a static table
gamma_table_uint[256], to match the code in compute_precache().

Minor: adjust the fprintf text in the qcms_tests_output_trc test to
clarify the test output.

BUG=600338
==========

[Noel Gordon, 2016-04-29 01:03:51]

noel@chromium.org changed reviewers:
+ radu.velea@intel.com

[Noel Gordon, 2016-04-29 01:06:13]

The CQ bit was checked by noel@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-04-29 01:06:53]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1929143003/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1929143003/1

[Noel Gordon, 2016-04-29 01:18:01]

Compare with the compute_precache() code:

https://code.google.com/p/chromium/codesearch#chromium/src/third_party/qcms/s...

Running the test:

./qcms_tests.exe -t qcms_test_output_trc -i itu-bt709.icc
Test qcms output gamma curve table integrity.
Output gamma table size = 4096
Detected parametric curve type = 3
Writing output gamma tables to
qcms-test-1461892182-parametric-gamma-output-BT.709 HDTV ITU .csv
gamma = 2.222229, a = 0.909912, b = 0.090088, c = 0.222214, d = 0.080994, e =
0.000000, f = 0.000000

Spreadsheet of the test results:

https://docs.google.com/spreadsheets/d/1tAn6NXczT1iiauDQgQwszVaCZ6tLnvL3j3eZQ...

[commit-bot: I haz the power, 2016-04-29 02:18:17]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-04-29 02:18:18]

Dry run: This issue passed the CQ dry run.

[Noel Gordon, 2016-04-29 02:43:47]

Description was changed from

==========
[qcms] Use a static table in build_output_lut to invert para curves

Instead of allocating the uint16_t output table, use a static table
gamma_table_uint[256], to match the code in compute_precache().

Minor: adjust the fprintf text in the qcms_tests_output_trc test to
clarify the test output.

BUG=600338
==========

to

==========
[qcms] Use a static table in build_output_lut to invert para curves

Instead of allocating the uint16_t output table, use a static table
gamma_table_uint[256], to match the code in compute_precache().

Minor: adjust the fprintf text in the qcms_tests_output_trc test to
clarify the test output.

TBR=radu.velea@intel.com
BUG=600338
==========

[Noel Gordon, 2016-04-29 02:44:49]

Will TBR for now, Radu please review when you have time.

[Noel Gordon, 2016-04-29 02:44:55]

The CQ bit was checked by noel@chromium.org

[commit-bot: I haz the power, 2016-04-29 02:45:08]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1929143003/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1929143003/1

[commit-bot: I haz the power, 2016-04-29 02:48:46]

Description was changed from

==========
[qcms] Use a static table in build_output_lut to invert para curves

Instead of allocating the uint16_t output table, use a static table
gamma_table_uint[256], to match the code in compute_precache().

Minor: adjust the fprintf text in the qcms_tests_output_trc test to
clarify the test output.

TBR=radu.velea@intel.com
BUG=600338
==========

to

==========
[qcms] Use a static table in build_output_lut to invert para curves

Instead of allocating the uint16_t output table, use a static table
gamma_table_uint[256], to match the code in compute_precache().

Minor: adjust the fprintf text in the qcms_tests_output_trc test to
clarify the test output.

TBR=radu.velea@intel.com
BUG=600338
==========

[commit-bot: I haz the power, 2016-04-29 02:48:47]

Committed patchset #1 (id:1)

[radu, 2016-04-29 06:28:31]

On 2016/04/29 02:48:47, commit-bot: I haz the power wrote:
> Committed patchset #1 (id:1)

lgtm

[commit-bot: I haz the power, 2016-04-30 17:24:04]

Patchset 1 (id:??) landed as
https://crrev.com/df1527ec51ed0079b792b260340918c9abfe551c
Cr-Commit-Position: refs/heads/master@{#390583}

[commit-bot: I haz the power, 2016-04-30 17:59:14]

Description was changed from

==========
[qcms] Use a static table in build_output_lut to invert para curves

Instead of allocating the uint16_t output table, use a static table
gamma_table_uint[256], to match the code in compute_precache().

Minor: adjust the fprintf text in the qcms_tests_output_trc test to
clarify the test output.

TBR=radu.velea@intel.com
BUG=600338
==========

to

==========
[qcms] Use a static table in build_output_lut to invert para curves

Instead of allocating the uint16_t output table, use a static table
gamma_table_uint[256], to match the code in compute_precache().

Minor: adjust the fprintf text in the qcms_tests_output_trc test to
clarify the test output.

TBR=radu.velea@intel.com
BUG=600338

Committed: https://crrev.com/df1527ec51ed0079b792b260340918c9abfe551c
Cr-Commit-Position: refs/heads/master@{#390583}
==========