Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Speedup RNN-T greedy decoding #7926
Speedup RNN-T greedy decoding #7926
Changes from 18 commits
9342489
7bcc4c0
7a0942f
26ec40c
1d556ea
cf631dd
a50965d
510eb90
659cfff
40d1568
ca2d94b
b328fac
7997bd6
6f7746b
95da9d1
ef35381
ca5779d
97092ff
c9785ff
1e09979
f4b7b68
d67b14b
c7d298d
2ea8f7f
5c8e18e
e8c43d0
ffe2a67
77bf674
02a9bbd
83c4793
430e159
266be2c
ce33493
9d545ee
9669149
1dbf29e
b4421cd
3e1ca1e
1b97e33
4429432
b7b83df
3df991a
31649fa
5f67c66
df86b17
0f4463b
c38f222
File filter
Filter by extension
Conversations
Jump to
There are no files selected for viewing
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Explain in docstring what this isc
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Done, missed the class docstring before)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Great work in documenting inline
Please be super explicit in documenting every line of decoding logic so that future reader has full knowledge of what is going on at every line.
It is necessary cause decoding loop is super complicated for rnnt
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Added comments for this
if
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Document line
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Added a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Document
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Added a comment (above this line)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Document
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Added a comment above
Check warning
Code scanning / CodeQL
Overwriting attribute in super-class or sub-class Warning
Check warning
Code scanning / CodeQL
Overwriting attribute in super-class or sub-class Warning