Single simulation runner

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

 # 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 logging
 import os
@@ skipping 81 matching lines @@
           self._simulate(noise_generator, input_filepath,
                          input_noise_cache_path, output_path, config_filepath)
 
   def _simulate(self, noise_generator, input_filepath, input_noise_cache_path,
                 output_path, config_filepath):
     """
     Simulates a given combination of APM configurations, probing signals, and
     noise generators. It iterates over the noise generator internal
     configurations.
     """
-    # TODO(alessio): implement.
-    pass
+    # Generate pairs of noisy input and reference signal files.
+    noise_generator.generate(
+        input_signal_filepath=input_filepath,

[ivoc, 2017/03/01 23:22:33]

Please use spaces around the '=', this happens several times in the file.

[AleBzk, 2017/03/02 08:30:08]

Thanks for your comment.
Python has its own style :)
https://engdoc.corp.google.com/eng/doc/devguide/py/styleguide.shtml?cl=head#w...
They admit that it looks strange to C++ developers.

[ivoc, 2017/03/02 19:16:26]

Ok, then we should keep it this way.

+        input_noise_cache_path=input_noise_cache_path,
+        base_output_path=output_path)
+
+    # For each input-reference pair, simulate a call and evaluate.
+    for noise_generator_config_name in noise_generator.config_names:
+      logging.info(' - noise config: <%s>', noise_generator_config_name)
+
+      # APM input and output signal paths.
+      noisy_signal_filepath = noise_generator.noisy_signal_filepaths[
+          noise_generator_config_name]
+      evaluation_output_path = noise_generator.output_paths[
+          noise_generator_config_name]
+
+      # Simulate a call using the audio processing module.
+      self._audioproc_wrapper.run(
+          config_filepath=config_filepath,
+          input_filepath=noisy_signal_filepath,
+          output_path=evaluation_output_path)
+
+      # Reference signal path for the evaluation step.
+      reference_signal_filepath = noise_generator.reference_signal_filepaths[
+          noise_generator_config_name]
+
+      # Evaluate.
+      self._evaluator.run(
+          evaluation_score_workers=self._evaluation_score_workers,
+          apm_output_filepath=self._audioproc_wrapper.output_filepath,
+          reference_input_filepath=reference_signal_filepath,
+          output_path=evaluation_output_path)
 
   @classmethod
   def _get_paths_collection(cls, filepaths):
     """
     Given a list of file paths, makes a collection with one pair for each item
     in the list where the key is the file name without extension and the value
     is the path.
     """
     filepaths_collection = {}
     for filepath in filepaths:
       name = os.path.splitext(os.path.split(filepath)[1])[0]
       filepaths_collection[name] = os.path.abspath(filepath)
     return filepaths_collection