Skip to content
This repository was archived by the owner on Jul 7, 2023. It is now read-only.

Conversation

@martinpopel
Copy link
Contributor

Many researchers are still reporting approx_bleu and comparing it with proper BLEU reported in other papers.
We should guide them to not do this and minimize the risk of false-news papers, such as https://arxiv.org/abs/1712.09662.

Many researchers are still reporting approx_bleu and comparing it with proper BLEU reported in other papers.
We should guide them to not do this and minimize the risk of false-news papers, such as https://arxiv.org/abs/1712.09662.
@martinpopel martinpopel reopened this Jan 18, 2018
cat $DECODE_FILE.$MODEL.$HPARAMS.beam$BEAM_SIZE.alpha$ALPHA.decodes
# Eval BLEU
# (Always report proper BLEU in papers, not the internal approx_bleu.)
t2t-bleu --translation=translation.en --reference=ref-translation.de
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

so where is this ref-translation.de file? this command won't actually work for a user out of the box, right? or am i missing something?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

OK. Added echo -e 'Hallo Welt\nAuf Wiedersehen Welt' > ref-translation.de

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

perfect, thank you

So the Walkthrough works out of the box
@rsepassi rsepassi merged commit 02b903b into tensorflow:master Jan 24, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants