Closed SkafteNicki closed 3 years ago
Hi! thanks for your contribution!, great first issue!
@SkafteNicki I would like to work on this issue.
I have moved the file functional/nlp.py
to functional/text/blue.py
. Where should I put the class implementation ??
Hi @karthikrangasai,
Please put the class implementation in a new folder text
inside a file called bleu
@SkafteNicki I have created a PR with the changes suggested.
I think tests and docs have to be updated, could you guide me on this ?
🚀 Feature
Motivation
Add Blue score to future NLP package:
functional/text/bleu.py
(for consistency)Pitch
Alternatives
Additional context
After merge to torchmetrics + official release the metric should be deprecated from flash + lightning-transformer.