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

Unimplement: Op TopK is not available in GraphDef version 8. #1040

Closed
ziiw opened this issue Feb 10, 2016 · 9 comments
Closed

Unimplement: Op TopK is not available in GraphDef version 8. #1040

ziiw opened this issue Feb 10, 2016 · 9 comments
Assignees
@girving

Comments

@ziiw
Copy link

ziiw commented Feb 10, 2016

Hi !

I've just finish to install TensorFlow on my OSX 10.11.13.

When I do: bazel-bin/tensorflow/examples/label_image

E tensorflow/core/common_runtime/executor.cc:275] Executor failed to create kernel. Unimplemented: Op TopK is not available in GraphDef version 8. It has been removed in version 7. Use TopKV2 instead. [[Node: top_k = TopK[T=DT_FLOAT, k=5, sorted=true, _device="/job:localhost/replica:0/task:0/cpu:0"](Const/_0)]] E tensorflow/examples/label_image/main.cc:321] Running print failed: Unimplemented: Op TopK is not available in GraphDef version 8. It has been removed in version 7. Use TopKV2 instead. [[Node: top_k = TopK[T=DT_FLOAT, k=5, sorted=true, _device="/job:localhost/replica:0/task:0/cpu:0"](Const/_0)]]

Did someone already seen this ?
@josh11b josh11b assigned girving and unassigned josh11b Feb 10, 2016
@girving
Copy link
Contributor

girving commented Feb 10, 2016

@petewarden is hitting the same thing. Looking at it now.

@girving
Copy link
Contributor

girving commented Feb 10, 2016

Diagnosed: we deprecated the C++ TopK op in favor of TopKV2 but that didn't affect the C++ graph builder API. Fix for label_image coming up, and I'll file a more detailed bug to add a deprecation feature when registering ops.

@girving
Copy link
Contributor

girving commented Feb 10, 2016

Fix in review.

@ziiw
Copy link
Author

ziiw commented Feb 11, 2016

Thx, how can I access this review ?

@nysalad
Copy link

nysalad commented Feb 11, 2016

FYI if you just want to get the label_image working locally,reverting the file works for me
git checkout 3ede550 tensorflow/core/kernels/topk_op.cc

@ziiw
Copy link
Author

ziiw commented Feb 11, 2016

Nice thx @nysalad :)

I've got some trouble installing tensorflow for label_images, I'm gonna uninstall all and re-install ;)

@girving
Copy link
Contributor

girving commented Feb 11, 2016

@vrv: Will there be a push to github soon? Looks like the last one was Tuesday.

@vrv
Copy link

vrv commented Feb 11, 2016

Probably some time today

On Thu, Feb 11, 2016 at 8:31 AM, Geoffrey Irving notifications@github.com
wrote:

@vrv https://github.com/vrv: Will there be a push to github soon? Looks
like the last one was Tuesday.

—
Reply to this email directly or view it on GitHub
#1040 (comment)
.

@vrv vrv closed this as completed in b2757fb Feb 11, 2016
@girving
Copy link
Contributor

girving commented Feb 11, 2016

@ziiw: master should work for you now, let me know if not.

tarasglek pushed a commit to tarasglek/tensorflow that referenced this issue Jun 20, 2017
@nealwu
Merge pull request tensorflow#1040 from aselle/inception_v2
9baf6ea 
Fixed concat order using tf_upgrade.py
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@girving girving

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
5 participants
@girving @vrv @ziiw @nysalad @josh11b