Conversational Speech tool, WavReaderAdaptor and unit test

webrtc/modules/audio_processing/test/conversational_speech/generator_unittest.cc

 /*
  *  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.
  */
 
@@ skipping 15 matching lines @@
 // - A is the first speaking, B is the second one,
 // - each character after the speaker's letter indicates a time unit (e.g., 100
 //   ms),
 // - "*" indicates speaking, "." listening,
 // - numbers indicate the turn index in std::vector<Turn>.
 //
 // Note that the same speaker can appear in multiple lines in order to depict
 // cases in which there are wrong offsets leading to self cross-talk (which is
 // rejected).
 
+#define _USE_MATH_DEFINES

[kwiberg-webrtc, 2017/03/28 17:52:45]

A comment on what effect this has, and on which header file?

[AleBzk, 2017/03/29 08:56:44]

Done.

+
 #include <stdio.h>
+#include <cmath>
 #include <map>
 #include <memory>
 
 #include "webrtc/base/logging.h"
+#include "webrtc/base/pathutils.h"
 #include "webrtc/modules/audio_processing/test/conversational_speech/config.h"
 #include "webrtc/modules/audio_processing/test/conversational_speech/mock_wavreader_factory.h"
 #include "webrtc/modules/audio_processing/test/conversational_speech/multiend_call.h"
 #include "webrtc/modules/audio_processing/test/conversational_speech/timing.h"
+#include "webrtc/modules/audio_processing/test/conversational_speech/wavreader_adaptor.h"
 #include "webrtc/test/gmock.h"
 #include "webrtc/test/gtest.h"
 #include "webrtc/test/testsupport/fileutils.h"
 
 namespace webrtc {
 namespace test {
 namespace {
 
 using conversational_speech::LoadTiming;
 using conversational_speech::SaveTiming;
 using conversational_speech::MockWavReaderFactory;
 using conversational_speech::MultiEndCall;
 using conversational_speech::Turn;
 using conversational_speech::WavReaderAbstractFactory;
+using conversational_speech::WavReaderAdaptor;
 
 const char* const audiotracks_path = "/path/to/audiotracks";
 const char* const timing_filepath = "/path/to/timing_file.txt";
 const char* const output_path = "/path/to/output_dir";
 
 const std::vector<Turn> expected_timing = {
     {"A", "a1", 0},
     {"B", "b1", 0},
     {"A", "a2", 100},
     {"B", "b2", -200},
@@ skipping 19 matching lines @@
   {"t500", kMockWavReaderFactoryParams500ms},
   {"t1000", kMockWavReaderFactoryParams1000ms},
 };
 
 std::unique_ptr<MockWavReaderFactory> CreateMockWavReaderFactory() {
   return std::unique_ptr<MockWavReaderFactory>(
       new MockWavReaderFactory(kDefaultMockWavReaderFactoryParams,
                                kDefaultMockWavReaderFactoryParamsMap));
 }
 
+void CreateSineWavFile(const std::string& filepath,
+                       const MockWavReaderFactory::Params& params,
+                       float frequency = 440.0f) {
+  // Create samples.
+  const double two_pi = 2.0 * M_PI;

[kwiberg-webrtc, 2017/03/28 17:52:45]

constexpr

[AleBzk, 2017/03/29 08:56:44]

Done.

+  std::unique_ptr<int16_t[]> samples(new int16_t[params.num_samples]);

[kwiberg-webrtc, 2017/03/28 17:52:45]

It's easier to avoid bugs if you use e.g. a std::vector<int16_t>.

If you expect the number of samples to be large (or if performance was a
concern), it would've been better to allocate a fixed-size std::array on the
stack and repeatedly fill it and write it to disk.

[AleBzk, 2017/03/29 08:56:44]

Done.

+  for (std::size_t i = 0; i < params.num_samples; ++i) {
+    // TODO(alessiob): the produced tone is not pure, improve.
+    samples[i] = std::lround(32767.0f * std::sin(
+        two_pi * i * frequency / params.sample_rate));
+  }
+
+  // Write samples.
+  WavWriter wav_writer(filepath, params.sample_rate, params.num_channels);
+  wav_writer.WriteSamples(samples.get(), params.num_samples);
+}
+
 }  // namespace
 
 class ConversationalSpeechTest : public testing::Test {
  public:
   ConversationalSpeechTest() {
     rtc::LogMessage::LogToDebug(rtc::LS_VERBOSE);
   }
 };
 
 TEST_F(ConversationalSpeechTest, Settings) {
@@ skipping 320 matching lines @@
                                kDefaultMockWavReaderFactoryParamsMap));
 
   // There are two unique audio tracks to read.
   EXPECT_CALL(*mock_wavreader_factory, Create(testing::_)).Times(2);
 
   conversational_speech::MultiEndCall multiend_call(
       timing, audiotracks_path, std::move(mock_wavreader_factory));
   EXPECT_FALSE(multiend_call.valid());
 }
 
+TEST_F(ConversationalSpeechTest, MultiEndCallWavReaderAdaptorSine) {
+  // Parameters with which wav files are created.
+  constexpr int duration_seconds = 5;
+  const int sample_rates[] = {8000, 11025, 16000, 22050, 32000, 44100, 48000};
+
+  for (int sample_rate : sample_rates) {
+    const rtc::Pathname temp_filename(
+        OutputPath(), "TempSineWavFile_" + std::to_string(sample_rate)
+            + ".wav");
+
+    // Write wav file.
+    std::size_t num_samples = duration_seconds * sample_rate;

[kwiberg-webrtc, 2017/03/28 17:52:44]

const?

[AleBzk, 2017/03/29 08:56:44]

Done.

+    MockWavReaderFactory::Params params = {sample_rate, 1u, num_samples};
+    CreateSineWavFile(temp_filename.pathname(), params);
+    LOG(LS_VERBOSE) << "wav file @" << sample_rate << " Hz created ("
+        << num_samples << " samples)";
+
+    // Load wav file and check if params match.
+    WavReaderAdaptor wav_reader_adaptor(temp_filename.pathname());
+    EXPECT_EQ(sample_rate, wav_reader_adaptor.sample_rate());
+    EXPECT_EQ(1u, wav_reader_adaptor.num_channels());
+    EXPECT_EQ(num_samples, wav_reader_adaptor.num_samples());
+
+    // Clean up.
+    remove(temp_filename.pathname().c_str());
+  }
+}
+
 }  // namespace test
 }  // namespace webrtc