-
Notifications
You must be signed in to change notification settings - Fork 15
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
AssertionError when calculating BLEU score #15
Comments
Hi @thechargedneutron , this is because that in valor's code when loading a data sample meets some wrong, it will automatically random choose another sample for replacement. THis process should only happened in training process but in the code i don't restrict it. So when testing meeting some wrong samples. it will randomly choose another sample from test set which cause dupilicate validation on chosen samples, results in the captioning bug. Solution: Line 369 in 7a047df
changing from 'if video_pixels is None:' to 'if video_pixels is None and self.training:' Line 377 in 7a047df
changing from 'if audio_spectrograms is None:' to 'if audio_spectrograms is None and self.training:' At this time, wrong samples in testing time will report a bug instead of sesearching for a replaced one, and you could fix the (video/audio) data according to the bug hint. To view the real bug information, you could comment out the 'try except' at Line 179 in d616e97
I will fix this in the latest code, thanks for pointing it. |
Thanks for the code and documentation. I am running the captioning finetuning experiment on MSRVTT. During the evaluation stage, the code stops with an AssertionError here. Seems like
hypo
variable contains repetition of the same sentence multiple times. Can you please tell if I missed any step of if not, why is this error coming and how to solve it?Here is the generated
hypo
variable and theref
variable output for video9894:Thanks!
The text was updated successfully, but these errors were encountered: