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.

Sign up for GitHub

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

Jump to bottom

fix len greedy decode #106

Open

vinhtran2611 wants to merge 1 commit into harvardnlp:master from vinhtran2611:master

vinhtran2611 commented Feb 23, 2023

No description provided.


          fix len greedy decode

c8037cd

kuraga commented Sep 14, 2024

Have doubts. Seems like start_symbol is the numeric value of the <SOS>/<BOS> symbol instead of... the index we start, etc.

kuraga commented Sep 14, 2024

See also #116.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet