Total Harmonic Distorsion plus noise (THD+n) score in APM-QA.

Total Harmonic Distorsion plus noise (THD+n) score in APM-QA.

In order to compute a THD score, a pure tone must be used as input signal.
Also, its frequency must be known. For this reason, this CL adds a number of
changes in the APM-QA pipeline. More in detail, input signal metadata is loaded
and passed to the THD evaluation score instance. This makes the eval_scores
module less reusable, but it is fine since the module has been specifically
designed for the APM-QA module.

BUG=webrtc:7494
Review-Url: https://codereview.webrtc.org/3010413002
Cr-Commit-Position: refs/heads/master@{#19970}
Committed: https://webrtc.googlesource.com/src/+/5d26edcc02ac7545a0d73155eb92922cd2451a0f

[AleBzk, 2017-09-14 12:53:56]

Description was changed from

==========
Total Harmonic Distorsion plus noise (THD+n) score in APM-QA.

In order to compute a THD score, a pure tone must be used as input signal.
Also, its frequency must be known. For this reason, this CL adds a number of
changes in the APM-QA pipeline. More in detail, input signal metadata is loaded
and passed to the THD evaluation score instance. This makes the eval_scores
module less reusable, but it is fine since the module has been specifically
designed for the APM-QA module.

BUG=webrtc:7494
==========

to

==========
Total Harmonic Distorsion plus noise (THD+n) score in APM-QA.

In order to compute a THD score, a pure tone must be used as input signal.
Also, its frequency must be known. For this reason, this CL adds a number of
changes in the APM-QA pipeline. More in detail, input signal metadata is loaded
and passed to the THD evaluation score instance. This makes the eval_scores
module less reusable, but it is fine since the module has been specifically
designed for the APM-QA module.

BUG=webrtc:7494
==========

[AleBzk, 2017-09-14 12:53:56]

alessiob@webrtc.org changed reviewers:
+ aleloi@webrtc.org

[AleBzk, 2017-09-14 12:56:59]

Hi Alex,

As discussed offline, this CL comes from
https://chromium-review.googlesource.com/c/external/webrtc/+/609840/. It
contains all the changes you requested in there except for comparing our THD+n
implementation to an existing one.
If you wish, feel free to double-check.

I also documented how to run the unittests in README.md.

Alessio

[aleloi, 2017-09-14 12:59:34]

lgtm

[aleloi, 2017-09-15 13:27:10]

The CQ bit was checked by aleloi@webrtc.org to run a CQ dry run

[commit-bot: I haz the power, 2017-09-15 13:27:14]

Dry run: CQ is trying da patch.

Follow status at:
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.webrtc.org/...

[commit-bot: I haz the power, 2017-09-15 13:30:43]

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

[commit-bot: I haz the power, 2017-09-15 13:30:45]

Dry run: Try jobs failed on following builders:
  mac_baremetal on master.tryserver.webrtc (JOB_FAILED,
http://build.chromium.org/p/tryserver.webrtc/builders/mac_baremetal/builds/25057)

[AleBzk, 2017-09-26 12:23:17]

The CQ bit was checked by alessiob@webrtc.org

[AleBzk, 2017-09-26 12:23:18]

The patchset sent to the CQ was uploaded after l-g-t-m from aleloi@webrtc.org
Link to the patchset: https://codereview.webrtc.org/3010413002/#ps60001 (title:
"merge")

[commit-bot: I haz the power, 2017-09-26 12:23:27]

CQ is trying da patch.

Follow status at:
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.webrtc.org/...

[commit-bot: I haz the power, 2017-09-26 12:53:20]

CQ is committing da patch.

Bot data: {"patchset_id": 60001, "attempt_start_ts": 1506428597930730,
"parent_rev": "a42055116d3f0c20f7ddcb011926e97e2a6bcc08", "commit_rev":
"5d26edcc02ac7545a0d73155eb92922cd2451a0f"}

[commit-bot: I haz the power, 2017-09-26 12:53:25]

Description was changed from

==========
Total Harmonic Distorsion plus noise (THD+n) score in APM-QA.

In order to compute a THD score, a pure tone must be used as input signal.
Also, its frequency must be known. For this reason, this CL adds a number of
changes in the APM-QA pipeline. More in detail, input signal metadata is loaded
and passed to the THD evaluation score instance. This makes the eval_scores
module less reusable, but it is fine since the module has been specifically
designed for the APM-QA module.

BUG=webrtc:7494
==========

to

==========
Total Harmonic Distorsion plus noise (THD+n) score in APM-QA.

In order to compute a THD score, a pure tone must be used as input signal.
Also, its frequency must be known. For this reason, this CL adds a number of
changes in the APM-QA pipeline. More in detail, input signal metadata is loaded
and passed to the THD evaluation score instance. This makes the eval_scores
module less reusable, but it is fine since the module has been specifically
designed for the APM-QA module.

BUG=webrtc:7494

Review-Url: https://codereview.webrtc.org/3010413002
Cr-Commit-Position: refs/heads/master@{#19970}
Committed:
https://webrtc.googlesource.com/src/+/5d26edcc02ac7545a0d73155eb92922cd2451a0f
==========

[commit-bot: I haz the power, 2017-09-26 12:53:26]

Committed patchset #4 (id:60001) as
https://webrtc.googlesource.com/src/+/5d26edcc02ac7545a0d73155eb92922cd2451a0f