NeMo icon indicating copy to clipboard operation
NeMo copied to clipboard

Data Simulator

Open chooper1 opened this issue 3 years ago • 4 comments

What does this PR do ?

Adds a multispeaker audio session simulator and corresponding documentation and tutorials.

Collections: ASR, Tools

Changelog

  • Adds data simulator
  • Adds a manifest_utils file and restructures speaker_utils
  • Adds config files for simulating data as well as a tutorial and documentation

Before your PR is "Ready for review"

Pre checks:

  • [ x] Make sure you read and followed Contributor guidelines
  • [ ] Did you write any new necessary tests?
  • [ x] Did you add or update any necessary documentation?
  • [ ] Does the PR affect components that are optional to install? (Ex: Numba, Pynini, Apex etc)
    • [ ] Reviewer: Does the PR have correct import guards for all optional libraries?

PR Type:

  • [ x] New Feature
  • [ ] Bugfix
  • [ x] Documentation

Who can review?

Anyone in the community is free to review the PR once the checks have passed. Contributor guidelines contains specific people who can review PRs to various areas.

chooper1 avatar Aug 05 '22 18:08 chooper1

This pull request introduces 25 alerts when merging 2be4fb592f3f7e776c1812b7b5b6aa70e068b0d2 into 543118c3a0eb1f0a824f77659f92e36d3318fc75 - view on LGTM.com

new alerts:

  • 20 for Unused import
  • 4 for Unused local variable
  • 1 for `__init__` method calls overridden method

lgtm-com[bot] avatar Aug 05 '22 18:08 lgtm-com[bot]

This pull request introduces 25 alerts when merging 6dd2e58bcb424b7f9e16fccd5c0dd150d9cb70ea into df335fe0ec110b0846521580734da447da06a24e - view on LGTM.com

new alerts:

  • 20 for Unused import
  • 4 for Unused local variable
  • 1 for `__init__` method calls overridden method

lgtm-com[bot] avatar Aug 05 '22 19:08 lgtm-com[bot]

This pull request introduces 25 alerts when merging 30e8656aa5e17aaf02df23d4bbb7933833bf3cde into df335fe0ec110b0846521580734da447da06a24e - view on LGTM.com

new alerts:

  • 20 for Unused import
  • 4 for Unused local variable
  • 1 for `__init__` method calls overridden method

lgtm-com[bot] avatar Aug 05 '22 20:08 lgtm-com[bot]

This pull request introduces 1 alert when merging d212ff84fc2ca42c53041b931fbb7721321e3fd1 into df335fe0ec110b0846521580734da447da06a24e - view on LGTM.com

new alerts:

  • 1 for Missing call to `__init__` during object initialization

lgtm-com[bot] avatar Aug 05 '22 23:08 lgtm-com[bot]

This pull request introduces 1 alert when merging f68b61327d04984232eb25278c50aedde9ff3a95 into f921ebe0436e55f7547b183ca83a623f6678422d - view on LGTM.com

new alerts:

  • 1 for Syntax error

lgtm-com[bot] avatar Aug 10 '22 18:08 lgtm-com[bot]

This pull request introduces 2 alerts when merging 5d731eeb9436006ad10a749a3d50d51cadeea21b into f7b7985ea8b42c958c133a6366fc69aaed9ec7c9 - view on LGTM.com

new alerts:

  • 2 for Variable defined multiple times

lgtm-com[bot] avatar Aug 15 '22 19:08 lgtm-com[bot]