add epsilon to RankCrossEntropyLoss to avoid NAN

NTMC-Community / MatchZoo

Facilitating the design, comparison and sharing of deep text matching models.

Apache License 2.0

3.84k stars 897 forks source link

add epsilon to RankCrossEntropyLoss to avoid NAN #776

Closed jibrilfrej closed 5 years ago

uduse commented 5 years ago

@jibrilfrej It's ok to merge with two approved reviews, and I just invited you as a collaborator. Feel free to merge it once you accept the invitation.

jibrilfrej commented 5 years ago

I accepted the invitation but I still cannot merge.

uduse commented 5 years ago

Oh it's my fault. I forgot the master is protected. Would you mind making the change on branch 2.2-dev instead?

jibrilfrej commented 5 years ago

I created a new pull request : #779