Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Don't imply top_k is nondifferentiable #5726

Closed
girving opened this issue Nov 20, 2016 · 2 comments
Closed

Don't imply top_k is nondifferentiable #5726

girving opened this issue Nov 20, 2016 · 2 comments
Assignees
@xmbrst
Labels
type:docs-bug Document issues

Comments

@girving
Copy link
Contributor

girving commented Nov 20, 2016

top_k is in the Evaluation section of the documentation, which says

The evaluation ops are useful for measuring the performance of a network. Since they are
nondifferentiable, they are typically used at evaluation time.

This is confusing, since top_k is differentiable. Pointed out by @nmduc: #288 (comment).
@prb12 prb12 added the type:docs-bug Document issues label Nov 21, 2016
@caisq caisq closed this as completed in 92a2758 Nov 25, 2016
@kkarrancsu
Copy link

Hello, it looks like the latest docs (API r0.12) still has the old documentation that implies that top_k is non-differentiable. See docs

@girving
Copy link
Contributor Author

girving commented Feb 13, 2017

They are fixed at HEAD: https://www.tensorflow.org/versions/master/api_docs/python/nn/evaluation.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@xmbrst xmbrst

Labels
type:docs-bug Document issues
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
4 participants
@girving @kkarrancsu @prb12 @xmbrst