update label_image.py #15022
update label_image.py #15022
Conversation
freedomtan
commented
Dec 1, 2017
freedomtan
commented
- add build rule for label_image.py
- remove extraneous semicolons
add label_image_py to BUILD and update README.md accordingly
|
Can one of the admins verify this patch? |
|
@gunan the buildifier error seems to be suggesting a noop. What does that mean? |
| @@ -51,6 +51,16 @@ tf_cc_binary( | |||
| }), | |||
| ) | |||
|
|
|||
| py_binary( | |||
| name = "label_image_py", | |||
| main = "label_image.py", | |||
There was a problem hiding this comment.
Ahh interesting. @freedomtan can you change that.
|
Jenkins, test this please. |
1 similar comment
|
Jenkins, test this please. |
drpngx
added
kokoro:force-run
|
@gunan good to go if build passes? |
drpngx
added
awaiting testing (then merge)
and removed
awaiting review
|
Jenkins, test this please. |
|
@caisq looks like there are some failures on master, right? |
|
Some of the failures do not seem to be existing on master. For example this one: This one points to a nonexistent page: These looks strange, but I don't see them on Jenkins: Triggering test again. @tensorflow-jenkins test this please |
|
If the test doesn't refresh and the link is broken then it's probably an old Jenkins target that was moved to kokoro. |
|
Some tests appear to fail. I'm not sure they are related. Could you check? |
|
I don't think those failures are relevant. Anything I can do to clarify it? |
|
@freedomtan The test failures are unrelated. Mergin PR. Thanks. |
