Conversational Speech generator, main script with shell arguments

Conversational Speech generator, main script with shell arguments

BUG=webrtc:7218
NOTRY=True

Review-Url: https://codereview.webrtc.org/2733863002
Cr-Commit-Position: refs/heads/master@{#17117}
Committed: https://chromium.googlesource.com/external/webrtc/+/1d395dfb36e9ada73fc16aff09c4db3f3c3f17df

[AleBzk, 2017-03-06 14:08:19]

Description was changed from

==========
Conversational Speech generator, main script with shell arguments

BUG=webrtc:7218
NOTRY=True
==========

to

==========
Conversational Speech generator, main script with shell arguments

BUG=webrtc:7218
NOTRY=True
==========

[AleBzk, 2017-03-06 14:08:19]

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

[AleBzk, 2017-03-06 14:15:51]

Patchset #1 (id:1) has been deleted

[AleBzk, 2017-03-06 14:20:13]

Patchset #1 (id:20001) has been deleted

[AleBzk, 2017-03-06 14:21:20]

Hi Ivo,

This is the 2nd CL for a new tool I'm making. Everything is explained in the
README.md file (you don;t need to read it to review the CL).
I just added the main script (only parsing arguments) and a simple unit test.

Note that the Python code style in Chromium wants that CapWords is used (see
https://chromium.googlesource.com/chromium/src/+/master/styleguide/styleguide.md)
except for main.

Alessio

[ivoc, 2017-03-08 13:15:51]

lgtm with one small nit.

https://codereview.webrtc.org/2733863002/diff/40001/webrtc/modules/audio_proc...
File
webrtc/modules/audio_processing/test/py_conversational_speech/generate_conversational_tracks.py
(right):

https://codereview.webrtc.org/2733863002/diff/40001/webrtc/modules/audio_proc...
webrtc/modules/audio_processing/test/py_conversational_speech/generate_conversational_tracks.py:13:
Usage: generate_convsersational_tracks.py
typo

[AleBzk, 2017-03-08 14:09:59]

The CQ bit was checked by alessiob@webrtc.org

[AleBzk, 2017-03-08 14:10:00]

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

[commit-bot: I haz the power, 2017-03-08 14:10:09]

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-03-08 14:12:24]

CQ is committing da patch.

Bot data: {"patchset_id": 60001, "attempt_start_ts": 1488982199837170,
"parent_rev": "a8d8aadba8c64fe7832339ef1e8e802d0718fe38", "commit_rev":
"1d395dfb36e9ada73fc16aff09c4db3f3c3f17df"}

[commit-bot: I haz the power, 2017-03-08 14:12:27]

Description was changed from

==========
Conversational Speech generator, main script with shell arguments

BUG=webrtc:7218
NOTRY=True
==========

to

==========
Conversational Speech generator, main script with shell arguments

BUG=webrtc:7218
NOTRY=True

Review-Url: https://codereview.webrtc.org/2733863002
Cr-Commit-Position: refs/heads/master@{#17117}
Committed:
https://chromium.googlesource.com/external/webrtc/+/1d395dfb36e9ada73fc16aff0...
==========

[commit-bot: I haz the power, 2017-03-08 14:12:28]

Committed patchset #2 (id:60001) as
https://chromium.googlesource.com/external/webrtc/+/1d395dfb36e9ada73fc16aff0...