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

beam search #11

Open

wasd12345 opened this issue Dec 13, 2018 · 0 comments

wasd12345 commented Dec 13, 2018

The To Do mentions "Finish implementing beam search decoding to support > 1 beam".

It looks like you have various beam width variables in different places of the code. Was your note because:

you implemented beam search but it didn't seem to work as well when width>1 and you wanted to investigate further, or
there are still some places in the code that would need to be modified to before using beam search?

If 2), then which places would need to be modified?

Thanks!

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