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

Could you show more detail parameters about ROUGE and BLEU? #6

Open
kugwzk opened this issue Nov 20, 2019 · 4 comments
Open

Could you show more detail parameters about ROUGE and BLEU? #6

kugwzk opened this issue Nov 20, 2019 · 4 comments

Comments

@kugwzk
Copy link

kugwzk commented Nov 20, 2019

Hello, @ByronPan . I tried your code and got a pred.txt, and also get similar results about the diversity. However, I only got BLEU = 3.93, 38.4/6.9/2.6/1.3 (BP=0.716, ratio=0.750, hyp_len=38839, ref_len=51813) and for ROUGE ROUGE(1/2/3/L/SU4): 19.61/5.65/1.85/19.32/8.48. I think you said use a smooth BLEU so how could I got this script? And as for the rouge, could you please show more detail about the test_rouge script.
Thanks a lot.

@zzs-NLP
Copy link

zzs-NLP commented Jul 6, 2020

Hi~ I have a similar problem but I don't know how to solve it. TypeError: forward() missing 1 required positional argument: 'history_lengths' How did you solve it? Thank you very much!

@zzs-NLP
Copy link

zzs-NLP commented Jul 6, 2020

@kugwzk

@kugwzk
Copy link
Author

kugwzk commented Jul 7, 2020

@zzs-NLP , I didn't remember I met this problem. Maybe you can add 'history_len' in the parameter list?

@zzs-NLP
Copy link

zzs-NLP commented Jul 8, 2020

@kugwzk Ok, thank you very much for your advice, I will try it again~

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

2 participants