Skip to content
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

A little confused about beam search #9

Open
hityzy1122 opened this issue Nov 13, 2018 · 0 comments
Open

A little confused about beam search #9

hityzy1122 opened this issue Nov 13, 2018 · 0 comments

Comments

@hityzy1122
Copy link

hityzy1122 commented Nov 13, 2018

Dear Author:
Thanks for your publiction of this code which is an excellent work.

However I am a little confused about the implement of beam search which is used in the file captionme.py at line 139 and 142.

That is: when I set the beam_size as 3, I got 4 different results in self.done_beams[i] (located at the line 109 in beamsearch.py). I wonder whether it is a correct result. If self.done_beams is used to record the beams per batch, why the number of results in self.done_beams[i] is different from beam_size

Looking forward for your reply!

Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant