Skip to content
This repository has been archived by the owner on Nov 22, 2022. It is now read-only.

fixed overflow error in lm reporting #831

Closed

Conversation

shreydesai
Copy link
Contributor

Summary: Fixes overflow error in perplexity reporting by showing "inf" instead of OverflowError

Differential Revision: D16495447

@facebook-github-bot facebook-github-bot added the CLA Signed Do not delete this pull request or issue due to inactivity. label Jul 25, 2019
shreydesai added a commit to shreydesai/pytext that referenced this pull request Jul 25, 2019
Summary:
Pull Request resolved: facebookresearch#831

Fixes overflow error in perplexity reporting by showing "inf" instead of OverflowError

Differential Revision: D16495447

fbshipit-source-id: 9e9252b59f933eb688275fd0df2c266a501c6f32
shreydesai added a commit to shreydesai/pytext that referenced this pull request Jul 26, 2019
Summary:
Pull Request resolved: facebookresearch#831

Fixes overflow error in perplexity reporting by showing "inf" instead of OverflowError

Differential Revision: D16495447

fbshipit-source-id: ad4ccf19f8df8b87901d1330110c38f6523ca30f
shreydesai added a commit to shreydesai/pytext that referenced this pull request Jul 27, 2019
Summary:
Pull Request resolved: facebookresearch#831

Fixes overflow error in perplexity reporting by showing "inf" instead of OverflowError

Differential Revision: D16495447

fbshipit-source-id: 303a7888255e8db6c4baf19f8a09104ffedc5dfc
Summary:
Pull Request resolved: facebookresearch#831

Fixes overflow error in perplexity reporting by showing "inf" instead of OverflowError

Reviewed By: arbabu123

Differential Revision: D16495447

fbshipit-source-id: 20f64910c07386c22f3ba6e728b40764f3d8f15a
@facebook-github-bot
Copy link
Contributor

This pull request has been merged in 9f3b2b0.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
CLA Signed Do not delete this pull request or issue due to inactivity. Merged
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants