New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Attempt to make sense of reasoning for loss #20220
Conversation
I feel the document seems a bit weird. Adding "when" seems to make better sense of it. Open to suggestions on alternatives. Thanks
@@ -362,7 +362,7 @@ model's loss. This is the | |||
that will be optimized. | |||
|
|||
We can calculate the loss by calling @{tf.losses.sparse_softmax_cross_entropy}. | |||
The value returned by this function will be lowest, approximately 0, | |||
When the value returned by this function will be lowest, approximately 0, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
How about this:
The value returned by this function will be approximately 0 at lowest, when the probability of the correct class (at index label
') is near 1.0.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
That sounds better. Thanks.
Should I update the PR with it ?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I just patched it.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks helping us get this fixed.
@@ -362,7 +362,7 @@ model's loss. This is the | |||
that will be optimized. | |||
|
|||
We can calculate the loss by calling @{tf.losses.sparse_softmax_cross_entropy}. | |||
The value returned by this function will be lowest, approximately 0, | |||
When the value returned by this function will be lowest, approximately 0, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I just patched it.
I feel the document seems a bit weird. Adding "when" seems to make better sense of it. Open to suggestions on alternatives.
Thanks