-
Notifications
You must be signed in to change notification settings - Fork 72
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
How about results #4
Comments
I tried to reproduce results reported in the paper Get to the Point, using same parameters. But I was unsuccessful, results are much lower than expected, the ROUGE scores are half lower than the ones reported in the paper. Any idea / pretrained model available ? |
@colanim Have you got good results? |
Nope, I gave up and tried this repo, which is really easy to reproduce and gave better results anyway. |
@colanim Thanks a lot! |
@colanim @ymfa I have changed something which can help get better results. I hope this can help the one who will hold this repo in the future.
Line 127 in f6b3de3
Line 171 in f6b3de3
|
Good job. How about rouge scores?
The text was updated successfully, but these errors were encountered: