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

Add metrics to compute fluency of references #129

Closed
poojah15 opened this issue Jan 13, 2022 · 6 comments
Closed

Add metrics to compute fluency of references #129

poojah15 opened this issue Jan 13, 2022 · 6 comments

Comments

@poojah15
Copy link

Apart from the content preservation metrics that this repo contains, it is also common to check the fluency/ coherence of the generated outputs. Forward PPL using a pretrained language model is generally used to measure the fluency of the outputs but there seem to be other newly introduced metrics as well... I was wondering if anyone knows of some tools to compute fluency of sentences apart from PPL? It would be a useful feature to add to this repository as well. Any help is much appreciated!

@sharan21
Copy link

Hey, you can check out the "Naturalness" metric as a substitute for perplexity as it correlates better with human judgment in measuring the fluency or coherence of a sentence. https://github.com/passeul/style-transfer-model-evaluation
Ill check it out myself and add this feature as I feel it will be useful as well!

@poojah15
Copy link
Author

Thanks! It seems to be just the tool I need. Yes, it will be useful to integrate it as a feature to this repo.

@sharan21
Copy link

I tested the naturalness metric in https://github.com/passeul/style-transfer-model-evaluation and it works quite nicely. It should be useful in Text Style Transfer and Summarization tasks or even just Generation tasks to check the quality. Integrating it into this repo should be straightforward as well. Ill link a PR to this issue and merge it for others to use :)

@sharan21
Copy link

I have tested the new feature on my branch and it is working. This is the pull request for the issue. Feel free to clone my branch and use it incase it takes time for the pull request to get merged.

@poojah15
Copy link
Author

Thanks a lot!

@temporaer
Copy link
Member

@poojah15 , were you able to successfully run from @sharan21 's code?

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

3 participants