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

Conversation

@aasseman
Copy link
Contributor

No description provided.

@aasseman
Copy link
Contributor Author

Not tested yet

@aasseman
Copy link
Contributor Author

Test by printing the targets and predictions.
We go from that:

TARGET: ['viens', 'ici', 'immediatement', '.', '<EOS>', '<PAD>', '<PAD>', '<PAD>', '<PAD>', '<PAD>']
PREDICTION: ['.', '<PAD>', '<PAD>', '<PAD>', '<PAD>', '<PAD>', '<PAD>', '<PAD>', '<PAD>', '<PAD>']
BLEU: 0.4111336169005197

To:

TARGET: ['la', 'plupart', 'd', 'entre', 'nous', 'etait', 'des', 'veterans', '.']
PREDICTION: ['je', 'est', 'est', 'a', '.', '.']
BLEU: 7.060301868108111e-232

So I believe that it is working correctly.

@aasseman aasseman added bug Something isn't working enhancement New feature or request labels Apr 30, 2019
@tkornuta-ibm tkornuta-ibm merged commit ff9ce7d into IBM:develop Apr 30, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

bug Something isn't working enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants