search

hajicj / HAMR_2016

HAMR 2016 hack repo
MIT License
2 stars 1 forks source link

BLEU sometimes fails: division by zero #1

Open hajicj opened 8 years ago

hajicj commented 8 years ago

For some inputs, on Michele's NLTK (3.2.1), BLEU fails with a divide by zero error (Fraction class). Runs on Jan's NLTK 3.2.0.

Failing example: back on my grind and no more potent than Tylenol more focus on writing all the lyrics my mind store

MECHMulimedia commented 8 years ago

the actual lyrics are

back on my grid and all more potent than tylenol more focus on rhyming all the lyrics my mind can store

Sent from TypeMail

On Aug 7, 2016, 12:07 PM, at 12:07 PM, "Jan Hajič jr." notifications@github.com wrote:

For some inputs, on Michele's NLTK (3.2.1), BLEU fails with a divide by zero error (Fraction class). Runs on Jan's NLTK 3.2.0.

Failing example: back on my grind and no more potent than Tylenol more focus on writing all the lyrics my mind store

You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: https://github.com/hajicj/HAMR_2016/issues/1

mbuccoli commented 8 years ago

I suggest to create a script of rhyme.py with a more in-depth log, so we can compare ours logs and see what's not going on. Moreover, I can host the final application in my server, so maybe this problem won't happen.

hajicj commented 8 years ago

Bypassed by the pron_suffix_overlap() function instead of pron_bleu(). Needs testing.