alsa_conformance_test: Disable merging points function
The merging points feature may merge the wrong points so disable it
temporarily until we fix the issue.
BUG=b:145486343
TEST=build code and run test
Change-Id: I18498857540053098bc391f77a5a2c4f17040bc0
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/audiotest/+/1980200
Reviewed-by: Judy Hsiao <judyhsiao@chromium.org>
Commit-Queue: Yu-Hsuan Hsu <yuhsuan@chromium.org>
Tested-by: Yu-Hsuan Hsu <yuhsuan@chromium.org>
diff --git a/alsa_conformance_test.md b/alsa_conformance_test.md
index 6930d14..0ea7d2f 100644
--- a/alsa_conformance_test.md
+++ b/alsa_conformance_test.md
@@ -104,7 +104,7 @@
```
+ --merge_threshold
+ Merge points with TIME_DIFF less than merge_threshold. Only the latter point is
- counted in linear regression(default: 0.0001)
+ counted in linear regression. (default: 0)
## Results
These are the functions that ALSA conformance test covers.
diff --git a/alsa_conformance_test/alsa_conformance_args.c b/alsa_conformance_test/alsa_conformance_args.c
index 663c5f4..6317ad1 100644
--- a/alsa_conformance_test/alsa_conformance_args.c
+++ b/alsa_conformance_test/alsa_conformance_args.c
@@ -50,7 +50,12 @@
args->device_file = NULL;
args->dev_info_only = false;
args->iterations = 1;
- args->merge_threshold = 0.0001;
+ /*
+ * TODO(yuhsuan): This feature is not stable. Sometimes it may merge
+ * wrong points if the step size is small. Disable merging points until
+ * the issue is fixed. (b/145486343)
+ */
+ args->merge_threshold = 0;
return args;
}
diff --git a/alsa_conformance_test/alsa_conformance_test.c b/alsa_conformance_test/alsa_conformance_test.c
index 81a5036..36ba6c7 100644
--- a/alsa_conformance_test/alsa_conformance_test.c
+++ b/alsa_conformance_test/alsa_conformance_test.c
@@ -50,8 +50,8 @@
"Number of times to run the tests specified. (default: 1)\n");
printf("\t--merge_threshold: "
"Merge points with TIME_DIFF less than merge_threshold. "
- "Only the latter point is counted in linear regression "
- "(default: 0.0001)\n");
+ "Only the latter point is counted in linear regression. "
+ "(default: 0)\n");
printf("\t--device_file:\n"
"\t\tDevice file path. It will load devices from the file. "
"File format:\n"
