search

k2-fsa / sherpa

Speech-to-text server framework with next-gen Kaldi
https://k2-fsa.github.io/sherpa
Apache License 2.0
534 stars 107 forks source link

Remove the name Rnnt from class names #194

Open csukuangfj opened 1 year ago

csukuangfj commented 1 year ago

The name Rnnt in the code is confusing. We are not using recurrent connections (i.e., RNN, GRU, LSTM) in our transducer models and we should use transducer to replace Rnnt in the code.

danpovey commented 1 year ago

OK. If it spans across multiple codebases, e.g. icefall vs. k2, we may need to introduce aliases for compatibility.

csukuangfj commented 1 year ago

OK. If it spans across multiple codebases, e.g. icefall vs. k2, we may need to introduce aliases for compatibility.

I am mostly concerning with the code in sherpa. The changes would be implementation details and won't be visible to end users.