Adding Hybrid RNNT-CTC model #5364

VahidooX · 2022-11-09T02:50:08Z

What does this PR do ?

This PR is a refactored version of the following PR created by https://github.com/iankur:
#4854

It adds the Hybrid RNNT/CTC model which has two decoders of CTC and RNNT(Transducer) over the encoder.
It enables to train a single model instead of two which works with both CTC and RNNT decoding. It also speeds up the convergence for CTC models.

Added the following examples:
./examples/asr/asr_hybrid_transducer_ctc/speech_to_text_hybrid_rnnt_ctc_char.py
./examples/asr/asr_hybrid_transducer_ctc/speech_to_text_hybrid_rnnt_ctc_bpe.py

Along with these sample configs for hybrid Conformer model:
./examples/asr/conf/conformer/hybrid_transducer_ctc/conformer_hybrid_transducer_ctc_char.yaml
./examples/asr/conf/conformer/hybrid_transducer_ctc/conformer_hybrid_transducer_ctc_bpe.yaml

Collection:
ASR

Changelog

Added support for Hybrid RNNT/CTC ASR model.

PR Type:

New Feature
Bugfix
Documentation

Signed-off-by: Vahid <vnoroozi@nvidia.com>

for more information, see https://pre-commit.ci

Signed-off-by: Vahid <vnoroozi@nvidia.com>

…nt_rnnt_ctc

for more information, see https://pre-commit.ci

Signed-off-by: Vahid <vnoroozi@nvidia.com>

…nt_rnnt_ctc

Signed-off-by: Vahid <vnoroozi@nvidia.com>

lgtm-com · 2022-11-09T03:29:10Z

This pull request introduces 16 alerts when merging fa92433 into 265056e - view on LGTM.com

new alerts:

12 for Unused import
4 for Unreachable code

titu1994

Really awesome. Needs plenty of cleanup especially for the subword model but it should be doable.

examples/asr/conf/conformer/hybrid_transducer_ctc/conformer_hybrid_transducer_ctc_bpe.yaml

nemo/collections/asr/models/hybrid_rnnt_ctc_bpe_models.py

nemo/collections/asr/models/hybrid_rnnt_ctc_models.py

Signed-off-by: Vahid <vnoroozi@nvidia.com>

for more information, see https://pre-commit.ci

Signed-off-by: Vahid <vnoroozi@nvidia.com>

…nt_rnnt_ctc

lgtm-com · 2022-11-15T21:28:39Z

This pull request introduces 6 alerts when merging 2b3902a into 5665f14 - view on LGTM.com

new alerts:

4 for Unreachable code
2 for Unused import

Heads-up: LGTM.com's PR analysis will be disabled on the 5th of December, and LGTM.com will be shut down ⏻ completely on the 16th of December 2022. Please enable GitHub code scanning, which uses the same CodeQL engine ⚙️ that powers LGTM.com. For more information, please check out our post on the GitHub blog.

Signed-off-by: Vahid <vnoroozi@nvidia.com>

for more information, see https://pre-commit.ci

Signed-off-by: Vahid <vnoroozi@nvidia.com>

…nt_rnnt_ctc

…rnnt_ctc

lgtm-com · 2022-11-15T22:00:00Z

This pull request introduces 9 alerts when merging bab1fc2 into 5665f14 - view on LGTM.com

new alerts:

5 for Unused import
4 for Unreachable code

Heads-up: LGTM.com's PR analysis will be disabled on the 5th of December, and LGTM.com will be shut down ⏻ completely on the 16th of December 2022. Please enable GitHub code scanning, which uses the same CodeQL engine ⚙️ that powers LGTM.com. For more information, please check out our post on the GitHub blog.

Signed-off-by: Vahid <vnoroozi@nvidia.com>

lgtm-com · 2022-11-15T22:18:39Z

This pull request introduces 9 alerts when merging a0d584e into 5665f14 - view on LGTM.com

new alerts:

5 for Unused import
4 for Unreachable code

Heads-up: LGTM.com's PR analysis will be disabled on the 5th of December, and LGTM.com will be shut down ⏻ completely on the 16th of December 2022. Please enable GitHub code scanning, which uses the same CodeQL engine ⚙️ that powers LGTM.com. For more information, please check out our post on the GitHub blog.

Signed-off-by: Vahid <vnoroozi@nvidia.com>

lgtm-com · 2022-11-16T00:20:41Z

This pull request introduces 8 alerts when merging 31d9f07 into 25f9ab9 - view on LGTM.com

new alerts:

4 for Unused import
4 for Unreachable code

Heads-up: LGTM.com's PR analysis will be disabled on the 5th of December, and LGTM.com will be shut down ⏻ completely on the 16th of December 2022. Please enable GitHub code scanning, which uses the same CodeQL engine ⚙️ that powers LGTM.com. For more information, please check out our post on the GitHub blog.

Signed-off-by: Vahid <vnoroozi@nvidia.com>

…nt_rnnt_ctc

Signed-off-by: Vahid <vnoroozi@nvidia.com>

lgtm-com · 2022-11-30T07:44:08Z

This pull request introduces 1 alert when merging b5a56b4 into 5c1d59e - view on LGTM.com

new alerts:

1 for Signature mismatch in overriding method

Heads-up: LGTM.com's PR analysis will be disabled on the 5th of December, and LGTM.com will be shut down ⏻ completely on the 16th of December 2022. It looks like GitHub code scanning with CodeQL is already set up for this repo, so no further action is needed 🚀. For more information, please check out our post on the GitHub blog.

Signed-off-by: Vahid <vnoroozi@nvidia.com>

for more information, see https://pre-commit.ci

Signed-off-by: Vahid <vnoroozi@nvidia.com>

…nt_rnnt_ctc

titu1994

Looks awesome, @bmwshop for final review

examples/asr/conf/conformer/hybrid_transducer_ctc/conformer_hybrid_transducer_ctc_bpe.yaml

examples/asr/conf/conformer/hybrid_transducer_ctc/conformer_hybrid_transducer_ctc_char.yaml

titu1994 · 2022-12-01T22:52:00Z

nemo/collections/asr/models/hybrid_rnnt_ctc_bpe_models.py

+        Returns: None
+
+        """
+        if isinstance(new_tokenizer_dir, DictConfig):


@bmwshop can you look into whether you can pull up this preconfig stuff into utility method of ASRBPEMixin in the future.

nemo/collections/asr/models/hybrid_rnnt_ctc_models.py

tests/collections/asr/test_asr_hybrid_rnnt_ctc_model_bpe.py

Signed-off-by: Vahid <vnoroozi@nvidia.com>

…nt_rnnt_ctc

Signed-off-by: Vahid <vnoroozi@nvidia.com>

titu1994

PR looks great, thanks ! We should add docstring to some functions, plus note it in README.md cause its a very cool functionality.

Minor comments

titu1994 · 2022-12-06T00:25:39Z

Jenkinsfile

+
+    // stage('L2: Hybrid ASR RNNT-CTC dev run') {
+    //   when {
+    //     anyOf {


Dont we want to uncomment the test now?

titu1994 · 2022-12-06T00:27:51Z

tests/collections/asr/test_asr_hybrid_rnnt_ctc_model_char.py

+
+
+class TestEncDecHybridRNNTCTCModel:
+    @pytest.mark.skipif(


Where is subword level test file?

titu1994

Looks awesome !

* added initial code. Signed-off-by: Vahid <vnoroozi@nvidia.com> * added the confs. Signed-off-by: Vahid <vnoroozi@nvidia.com> * added the confs. Signed-off-by: Vahid <vnoroozi@nvidia.com> * [pre-commit.ci] auto fixes from pre-commit.com hooks for more information, see https://pre-commit.ci * changed name from joint to hybrid. Signed-off-by: Vahid <vnoroozi@nvidia.com> * [pre-commit.ci] auto fixes from pre-commit.com hooks for more information, see https://pre-commit.ci * fixed format. Signed-off-by: Vahid <vnoroozi@nvidia.com> * fixed format. Signed-off-by: Vahid <vnoroozi@nvidia.com> * fixed bug. Signed-off-by: Vahid <vnoroozi@nvidia.com> * [pre-commit.ci] auto fixes from pre-commit.com hooks for more information, see https://pre-commit.ci * fixed bug. Signed-off-by: Vahid <vnoroozi@nvidia.com> * addressed comments. Signed-off-by: Vahid <vnoroozi@nvidia.com> * [pre-commit.ci] auto fixes from pre-commit.com hooks for more information, see https://pre-commit.ci * addressed comments. Signed-off-by: Vahid <vnoroozi@nvidia.com> * added docs. Signed-off-by: Vahid <vnoroozi@nvidia.com> * added docs. Signed-off-by: Vahid <vnoroozi@nvidia.com> * added docs. Signed-off-by: Vahid <vnoroozi@nvidia.com> * added docs. Signed-off-by: Vahid <vnoroozi@nvidia.com> * fixed bug. Signed-off-by: Vahid <vnoroozi@nvidia.com> * [pre-commit.ci] auto fixes from pre-commit.com hooks for more information, see https://pre-commit.ci * fixed bug. Signed-off-by: Vahid <vnoroozi@nvidia.com> * fixed bug. Signed-off-by: Vahid <vnoroozi@nvidia.com> * fixed bug. Signed-off-by: Vahid <vnoroozi@nvidia.com> * fixed bug. Signed-off-by: Vahid <vnoroozi@nvidia.com> * [pre-commit.ci] auto fixes from pre-commit.com hooks for more information, see https://pre-commit.ci * fixed bug. Signed-off-by: Vahid <vnoroozi@nvidia.com> * fixed bug. Signed-off-by: Vahid <vnoroozi@nvidia.com> * fixed bug. Signed-off-by: Vahid <vnoroozi@nvidia.com> * addec CI test. Signed-off-by: Vahid <vnoroozi@nvidia.com> * addec CI test. Signed-off-by: Vahid <vnoroozi@nvidia.com> * fixed bugs in change_vocabs. Signed-off-by: vahidoox <vnoroozi@nvidia.com> * fixed bugs in change_vocabs. Signed-off-by: vahidoox <vnoroozi@nvidia.com> * fixed style. Signed-off-by: vahidoox <vnoroozi@nvidia.com> * fixed style. Signed-off-by: vahidoox <vnoroozi@nvidia.com> * fixed style. Signed-off-by: vahidoox <vnoroozi@nvidia.com> * raise error for aux_ctc. Signed-off-by: Vahid <vnoroozi@nvidia.com> * raise error for aux_ctc. Signed-off-by: Vahid <vnoroozi@nvidia.com> * [pre-commit.ci] auto fixes from pre-commit.com hooks for more information, see https://pre-commit.ci * raise error for aux_ctc. Signed-off-by: Vahid <vnoroozi@nvidia.com> * raise error for aux_ctc. Signed-off-by: Vahid <vnoroozi@nvidia.com> * updated the streaming names. Signed-off-by: Vahid <vnoroozi@nvidia.com> * added unittests. Signed-off-by: Vahid <vnoroozi@nvidia.com> * added unittests. Signed-off-by: Vahid <vnoroozi@nvidia.com> * added unittests. Signed-off-by: Vahid <vnoroozi@nvidia.com> * fixed tests. Signed-off-by: Vahid <vnoroozi@nvidia.com> * fixed tests. Signed-off-by: Vahid <vnoroozi@nvidia.com> * fixed tests. Signed-off-by: Vahid <vnoroozi@nvidia.com> * fixed tests. Signed-off-by: Vahid <vnoroozi@nvidia.com> * fixed tests. Signed-off-by: Vahid <vnoroozi@nvidia.com> * [pre-commit.ci] auto fixes from pre-commit.com hooks for more information, see https://pre-commit.ci * fixed tests. Signed-off-by: Vahid <vnoroozi@nvidia.com> * added methods. Signed-off-by: Vahid <vnoroozi@nvidia.com> * added decoding. Signed-off-by: Vahid <vnoroozi@nvidia.com> * fxied the tests. Signed-off-by: Vahid <vnoroozi@nvidia.com> Signed-off-by: Vahid <vnoroozi@nvidia.com> Signed-off-by: vahidoox <vnoroozi@nvidia.com> Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com> Signed-off-by: Vladimir Bataev <vbataev@nvidia.com>

* added initial code. Signed-off-by: Vahid <vnoroozi@nvidia.com> * added the confs. Signed-off-by: Vahid <vnoroozi@nvidia.com> * added the confs. Signed-off-by: Vahid <vnoroozi@nvidia.com> * [pre-commit.ci] auto fixes from pre-commit.com hooks for more information, see https://pre-commit.ci * changed name from joint to hybrid. Signed-off-by: Vahid <vnoroozi@nvidia.com> * [pre-commit.ci] auto fixes from pre-commit.com hooks for more information, see https://pre-commit.ci * fixed format. Signed-off-by: Vahid <vnoroozi@nvidia.com> * fixed format. Signed-off-by: Vahid <vnoroozi@nvidia.com> * fixed bug. Signed-off-by: Vahid <vnoroozi@nvidia.com> * [pre-commit.ci] auto fixes from pre-commit.com hooks for more information, see https://pre-commit.ci * fixed bug. Signed-off-by: Vahid <vnoroozi@nvidia.com> * addressed comments. Signed-off-by: Vahid <vnoroozi@nvidia.com> * [pre-commit.ci] auto fixes from pre-commit.com hooks for more information, see https://pre-commit.ci * addressed comments. Signed-off-by: Vahid <vnoroozi@nvidia.com> * added docs. Signed-off-by: Vahid <vnoroozi@nvidia.com> * added docs. Signed-off-by: Vahid <vnoroozi@nvidia.com> * added docs. Signed-off-by: Vahid <vnoroozi@nvidia.com> * added docs. Signed-off-by: Vahid <vnoroozi@nvidia.com> * fixed bug. Signed-off-by: Vahid <vnoroozi@nvidia.com> * [pre-commit.ci] auto fixes from pre-commit.com hooks for more information, see https://pre-commit.ci * fixed bug. Signed-off-by: Vahid <vnoroozi@nvidia.com> * fixed bug. Signed-off-by: Vahid <vnoroozi@nvidia.com> * fixed bug. Signed-off-by: Vahid <vnoroozi@nvidia.com> * fixed bug. Signed-off-by: Vahid <vnoroozi@nvidia.com> * [pre-commit.ci] auto fixes from pre-commit.com hooks for more information, see https://pre-commit.ci * fixed bug. Signed-off-by: Vahid <vnoroozi@nvidia.com> * fixed bug. Signed-off-by: Vahid <vnoroozi@nvidia.com> * fixed bug. Signed-off-by: Vahid <vnoroozi@nvidia.com> * addec CI test. Signed-off-by: Vahid <vnoroozi@nvidia.com> * addec CI test. Signed-off-by: Vahid <vnoroozi@nvidia.com> * fixed bugs in change_vocabs. Signed-off-by: vahidoox <vnoroozi@nvidia.com> * fixed bugs in change_vocabs. Signed-off-by: vahidoox <vnoroozi@nvidia.com> * fixed style. Signed-off-by: vahidoox <vnoroozi@nvidia.com> * fixed style. Signed-off-by: vahidoox <vnoroozi@nvidia.com> * fixed style. Signed-off-by: vahidoox <vnoroozi@nvidia.com> * raise error for aux_ctc. Signed-off-by: Vahid <vnoroozi@nvidia.com> * raise error for aux_ctc. Signed-off-by: Vahid <vnoroozi@nvidia.com> * [pre-commit.ci] auto fixes from pre-commit.com hooks for more information, see https://pre-commit.ci * raise error for aux_ctc. Signed-off-by: Vahid <vnoroozi@nvidia.com> * raise error for aux_ctc. Signed-off-by: Vahid <vnoroozi@nvidia.com> * updated the streaming names. Signed-off-by: Vahid <vnoroozi@nvidia.com> * added unittests. Signed-off-by: Vahid <vnoroozi@nvidia.com> * added unittests. Signed-off-by: Vahid <vnoroozi@nvidia.com> * added unittests. Signed-off-by: Vahid <vnoroozi@nvidia.com> * fixed tests. Signed-off-by: Vahid <vnoroozi@nvidia.com> * fixed tests. Signed-off-by: Vahid <vnoroozi@nvidia.com> * fixed tests. Signed-off-by: Vahid <vnoroozi@nvidia.com> * fixed tests. Signed-off-by: Vahid <vnoroozi@nvidia.com> * fixed tests. Signed-off-by: Vahid <vnoroozi@nvidia.com> * [pre-commit.ci] auto fixes from pre-commit.com hooks for more information, see https://pre-commit.ci * fixed tests. Signed-off-by: Vahid <vnoroozi@nvidia.com> * added methods. Signed-off-by: Vahid <vnoroozi@nvidia.com> * added decoding. Signed-off-by: Vahid <vnoroozi@nvidia.com> * fxied the tests. Signed-off-by: Vahid <vnoroozi@nvidia.com> Signed-off-by: Vahid <vnoroozi@nvidia.com> Signed-off-by: vahidoox <vnoroozi@nvidia.com> Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com> Signed-off-by: andrusenkoau <andrusenkoau@gmail.com>

* added initial code. Signed-off-by: Vahid <vnoroozi@nvidia.com> * added the confs. Signed-off-by: Vahid <vnoroozi@nvidia.com> * added the confs. Signed-off-by: Vahid <vnoroozi@nvidia.com> * [pre-commit.ci] auto fixes from pre-commit.com hooks for more information, see https://pre-commit.ci * changed name from joint to hybrid. Signed-off-by: Vahid <vnoroozi@nvidia.com> * [pre-commit.ci] auto fixes from pre-commit.com hooks for more information, see https://pre-commit.ci * fixed format. Signed-off-by: Vahid <vnoroozi@nvidia.com> * fixed format. Signed-off-by: Vahid <vnoroozi@nvidia.com> * fixed bug. Signed-off-by: Vahid <vnoroozi@nvidia.com> * [pre-commit.ci] auto fixes from pre-commit.com hooks for more information, see https://pre-commit.ci * fixed bug. Signed-off-by: Vahid <vnoroozi@nvidia.com> * addressed comments. Signed-off-by: Vahid <vnoroozi@nvidia.com> * [pre-commit.ci] auto fixes from pre-commit.com hooks for more information, see https://pre-commit.ci * addressed comments. Signed-off-by: Vahid <vnoroozi@nvidia.com> * added docs. Signed-off-by: Vahid <vnoroozi@nvidia.com> * added docs. Signed-off-by: Vahid <vnoroozi@nvidia.com> * added docs. Signed-off-by: Vahid <vnoroozi@nvidia.com> * added docs. Signed-off-by: Vahid <vnoroozi@nvidia.com> * fixed bug. Signed-off-by: Vahid <vnoroozi@nvidia.com> * [pre-commit.ci] auto fixes from pre-commit.com hooks for more information, see https://pre-commit.ci * fixed bug. Signed-off-by: Vahid <vnoroozi@nvidia.com> * fixed bug. Signed-off-by: Vahid <vnoroozi@nvidia.com> * fixed bug. Signed-off-by: Vahid <vnoroozi@nvidia.com> * fixed bug. Signed-off-by: Vahid <vnoroozi@nvidia.com> * [pre-commit.ci] auto fixes from pre-commit.com hooks for more information, see https://pre-commit.ci * fixed bug. Signed-off-by: Vahid <vnoroozi@nvidia.com> * fixed bug. Signed-off-by: Vahid <vnoroozi@nvidia.com> * fixed bug. Signed-off-by: Vahid <vnoroozi@nvidia.com> * addec CI test. Signed-off-by: Vahid <vnoroozi@nvidia.com> * addec CI test. Signed-off-by: Vahid <vnoroozi@nvidia.com> * fixed bugs in change_vocabs. Signed-off-by: vahidoox <vnoroozi@nvidia.com> * fixed bugs in change_vocabs. Signed-off-by: vahidoox <vnoroozi@nvidia.com> * fixed style. Signed-off-by: vahidoox <vnoroozi@nvidia.com> * fixed style. Signed-off-by: vahidoox <vnoroozi@nvidia.com> * fixed style. Signed-off-by: vahidoox <vnoroozi@nvidia.com> * raise error for aux_ctc. Signed-off-by: Vahid <vnoroozi@nvidia.com> * raise error for aux_ctc. Signed-off-by: Vahid <vnoroozi@nvidia.com> * [pre-commit.ci] auto fixes from pre-commit.com hooks for more information, see https://pre-commit.ci * raise error for aux_ctc. Signed-off-by: Vahid <vnoroozi@nvidia.com> * raise error for aux_ctc. Signed-off-by: Vahid <vnoroozi@nvidia.com> * updated the streaming names. Signed-off-by: Vahid <vnoroozi@nvidia.com> * added unittests. Signed-off-by: Vahid <vnoroozi@nvidia.com> * added unittests. Signed-off-by: Vahid <vnoroozi@nvidia.com> * added unittests. Signed-off-by: Vahid <vnoroozi@nvidia.com> * fixed tests. Signed-off-by: Vahid <vnoroozi@nvidia.com> * fixed tests. Signed-off-by: Vahid <vnoroozi@nvidia.com> * fixed tests. Signed-off-by: Vahid <vnoroozi@nvidia.com> * fixed tests. Signed-off-by: Vahid <vnoroozi@nvidia.com> * fixed tests. Signed-off-by: Vahid <vnoroozi@nvidia.com> * [pre-commit.ci] auto fixes from pre-commit.com hooks for more information, see https://pre-commit.ci * fixed tests. Signed-off-by: Vahid <vnoroozi@nvidia.com> * added methods. Signed-off-by: Vahid <vnoroozi@nvidia.com> * added decoding. Signed-off-by: Vahid <vnoroozi@nvidia.com> * fxied the tests. Signed-off-by: Vahid <vnoroozi@nvidia.com> Signed-off-by: Vahid <vnoroozi@nvidia.com> Signed-off-by: vahidoox <vnoroozi@nvidia.com> Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>

VahidooX added 2 commits November 8, 2022 18:36

added initial code.

eae0620

Signed-off-by: Vahid <vnoroozi@nvidia.com>

added the confs.

b4c3f2a

Signed-off-by: Vahid <vnoroozi@nvidia.com>

VahidooX requested a review from titu1994 November 9, 2022 02:50

VahidooX and others added 8 commits November 8, 2022 18:50

added the confs.

a8b7886

Signed-off-by: Vahid <vnoroozi@nvidia.com>

[pre-commit.ci] auto fixes from pre-commit.com hooks

416af52

for more information, see https://pre-commit.ci

changed name from joint to hybrid.

384729f

Signed-off-by: Vahid <vnoroozi@nvidia.com>

Merge remote-tracking branch 'origin/add_joint_rnnt_ctc' into add_joi…

c84f981

…nt_rnnt_ctc

[pre-commit.ci] auto fixes from pre-commit.com hooks

3164ac6

for more information, see https://pre-commit.ci

fixed format.

4623bd6

Signed-off-by: Vahid <vnoroozi@nvidia.com>

Merge remote-tracking branch 'origin/add_joint_rnnt_ctc' into add_joi…

50f70bf

…nt_rnnt_ctc

fixed format.

fa92433

Signed-off-by: Vahid <vnoroozi@nvidia.com>

titu1994 reviewed Nov 11, 2022

View reviewed changes

VahidooX and others added 4 commits November 15, 2022 13:09

fixed bug.

512eef8

Signed-off-by: Vahid <vnoroozi@nvidia.com>

[pre-commit.ci] auto fixes from pre-commit.com hooks

edd79e3

for more information, see https://pre-commit.ci

fixed bug.

2fa7c57

Signed-off-by: Vahid <vnoroozi@nvidia.com>

Merge remote-tracking branch 'origin/add_joint_rnnt_ctc' into add_joi…

2b3902a

…nt_rnnt_ctc

VahidooX and others added 5 commits November 15, 2022 13:42

addressed comments.

3820433

Signed-off-by: Vahid <vnoroozi@nvidia.com>

[pre-commit.ci] auto fixes from pre-commit.com hooks

4b29ec7

for more information, see https://pre-commit.ci

addressed comments.

e7b1171

Signed-off-by: Vahid <vnoroozi@nvidia.com>

Merge remote-tracking branch 'origin/add_joint_rnnt_ctc' into add_joi…

ef5cca3

…nt_rnnt_ctc

Merge branch 'main' of https://github.com/NVIDIA/NeMo into add_joint_…

bab1fc2

…rnnt_ctc

added docs.

a0d584e

Signed-off-by: Vahid <vnoroozi@nvidia.com>

VahidooX added 2 commits November 15, 2022 16:06

added docs.

7ab6384

Signed-off-by: Vahid <vnoroozi@nvidia.com>

added docs.

31d9f07

Signed-off-by: Vahid <vnoroozi@nvidia.com>

added docs.

a84d29f

Signed-off-by: Vahid <vnoroozi@nvidia.com>

VahidooX added 7 commits November 29, 2022 17:05

added unittests.

39e6197

Signed-off-by: Vahid <vnoroozi@nvidia.com>

added unittests.

3c32e41

Signed-off-by: Vahid <vnoroozi@nvidia.com>

fixed tests.

a4499b1

Signed-off-by: Vahid <vnoroozi@nvidia.com>

Merge remote-tracking branch 'origin/add_joint_rnnt_ctc' into add_joi…

c239b3d

…nt_rnnt_ctc

fixed tests.

c991d18

Signed-off-by: Vahid <vnoroozi@nvidia.com>

fixed tests.

8c678ed

Signed-off-by: Vahid <vnoroozi@nvidia.com>

fixed tests.

b5a56b4

Signed-off-by: Vahid <vnoroozi@nvidia.com>

VahidooX and others added 5 commits November 30, 2022 10:57

fixed tests.

fb8e923

Signed-off-by: Vahid <vnoroozi@nvidia.com>

[pre-commit.ci] auto fixes from pre-commit.com hooks

b33a6aa

for more information, see https://pre-commit.ci

fixed tests.

ecf9f1d

Signed-off-by: Vahid <vnoroozi@nvidia.com>

Merge remote-tracking branch 'origin/add_joint_rnnt_ctc' into add_joi…

add76b3

…nt_rnnt_ctc

Merge branch 'main' into add_joint_rnnt_ctc

0ca5d2d

titu1994 previously approved these changes Dec 2, 2022

View reviewed changes

VahidooX and others added 4 commits December 2, 2022 14:02

added methods.

f6c5558

Signed-off-by: Vahid <vnoroozi@nvidia.com>

Merge remote-tracking branch 'origin/add_joint_rnnt_ctc' into add_joi…

aab6e4e

…nt_rnnt_ctc

Merge branch 'main' into add_joint_rnnt_ctc

ab072da

Merge remote-tracking branch 'origin/add_joint_rnnt_ctc' into add_joi…

853a490

…nt_rnnt_ctc

VahidooX dismissed titu1994’s stale review via 853a490 December 2, 2022 22:03

VahidooX and others added 5 commits December 2, 2022 15:24

added decoding.

f7361db

Signed-off-by: Vahid <vnoroozi@nvidia.com>

fxied the tests.

49c8c14

Signed-off-by: Vahid <vnoroozi@nvidia.com>

Merge branch 'main' into add_joint_rnnt_ctc

d27012a

Merge branch 'main' into add_joint_rnnt_ctc

52b84ba

Merge branch 'main' into add_joint_rnnt_ctc

78490e3

titu1994 reviewed Dec 6, 2022

View reviewed changes

titu1994 approved these changes Dec 6, 2022

View reviewed changes

titu1994 merged commit 786a850 into NVIDIA:main Dec 6, 2022

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Adding Hybrid RNNT-CTC model #5364

Adding Hybrid RNNT-CTC model #5364

VahidooX commented Nov 9, 2022 •

edited

Loading

lgtm-com bot commented Nov 9, 2022

titu1994 left a comment

lgtm-com bot commented Nov 15, 2022

lgtm-com bot commented Nov 15, 2022

lgtm-com bot commented Nov 15, 2022

lgtm-com bot commented Nov 16, 2022

lgtm-com bot commented Nov 30, 2022

titu1994 left a comment

titu1994 Dec 1, 2022

titu1994 left a comment

titu1994 Dec 6, 2022

titu1994 Dec 6, 2022

titu1994 left a comment

Adding Hybrid RNNT-CTC model #5364

Adding Hybrid RNNT-CTC model #5364

Conversation

VahidooX commented Nov 9, 2022 • edited Loading

What does this PR do ?

Changelog

lgtm-com bot commented Nov 9, 2022

titu1994 left a comment

Choose a reason for hiding this comment

lgtm-com bot commented Nov 15, 2022

lgtm-com bot commented Nov 15, 2022

lgtm-com bot commented Nov 15, 2022

lgtm-com bot commented Nov 16, 2022

lgtm-com bot commented Nov 30, 2022

titu1994 left a comment

Choose a reason for hiding this comment

titu1994 Dec 1, 2022

Choose a reason for hiding this comment

titu1994 left a comment

Choose a reason for hiding this comment

titu1994 Dec 6, 2022

Choose a reason for hiding this comment

titu1994 Dec 6, 2022

Choose a reason for hiding this comment

titu1994 left a comment

Choose a reason for hiding this comment

VahidooX commented Nov 9, 2022 •

edited

Loading