APM Quality Assessment, simulation controller and libraries

webrtc/modules/audio_processing/test/py_quality_assessment/quality_assessment/data_access.py

Index: webrtc/modules/audio_processing/test/py_quality_assessment/quality_assessment/data_access.py
diff --git a/webrtc/modules/audio_processing/test/py_quality_assessment/quality_assessment/data_access.py b/webrtc/modules/audio_processing/test/py_quality_assessment/quality_assessment/data_access.py
new file mode 100644
index 0000000000000000000000000000000000000000..0c83123df3c6056001c2cbbf5227e5dcd32363dc
--- /dev/null
+++ b/webrtc/modules/audio_processing/test/py_quality_assessment/quality_assessment/data_access.py
@@ -0,0 +1,18 @@
+# Copyright (c) 2017 The WebRTC project authors. All Rights Reserved.
+#
+# Use of this source code is governed by a BSD-style license
+# that can be found in the LICENSE file in the root of the source
+# tree. An additional intellectual property rights grant can be found
+# in the file PATENTS.  All contributing project authors may
+# be found in the AUTHORS file in the root of the source tree.
+
+import os
+
+def make_directory(path):
+  """
+  Makes a directory (recursively) and hides OSError exceptions.

[aleloi2, 2017/03/02 01:14:03]

Why should OSError be hidden? I can't understand this from the usage in
simulation.py

[AleBzk, 2017/03/02 09:58:38]

Premise: in my tool I implicitly implemented checkpoints and cache, so a run may
reuse data from a previous one.
Answer: the directory might already exist, just wanted to avoid code duplication
every time I add directories.

To improve readability and avoid issues with missing write permissions, I
explicitly check if the directory already exists and just return if so.

Thanks for this comment!

[AleBzk, 2017/03/02 09:58:38]

Done.

+  """
+  try:
+    os.makedirs(path)
+  except OSError:
+    pass