why don't remove <end> for References and Hypotheses in validate ?

[Lzhushuai]

Hi, thank you for the nice work. I'm new to image caption and the work helps me a lot. from Line 298 to Line in train.py when you wrap the list for Hypotheses and References , why don't you remove the , this tag is not part of the caption same to and . Thank, you!

[soloist97]

Maybe that's because we want to check whether the model can generate '<end>' token at the last of a sentence. As '<end>' is used to indicate the end of the generation process.

[sgrvinod]

Hi @Lzhushuai, sorry for the really delayed reply. Yes, what @soloist97 said is right. I do, however, remove it in the final evaluation in eval.py so it doesn't contribute to the final BLEU scores.

[LealemTilahun]

Hi @sgrvinod, i tried to run eval.py but "ValueError: max() arg is an empty sequence" error msg is displaying. how can i fix this error please!