-
Notifications
You must be signed in to change notification settings - Fork 74.2k
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
Not a JPEG issue #9786
Comments
I'm guessing the problem that is your image is...drumroll...not a jpeg. It's probably a png. I recommend using |
@petewarden Any objections to me changing the example to |
@girving No, this seems like a good change to me, thanks! |
|
Here is a python script to identify those fault jpg images in a directory.
|
@Mohit-Ak Note that you don't need to do this anymore; Tensorflow's |
But I faced the same issue when I used the Object Detection API. This happens on the Cloud Training when it reads the ".record" file. I don't know the place where the object detection API uses it but it still has the issue. I can give you the logs if you want. |
Ah, the Object Detection API may be parsing the jpeg outside of TensorFlow. Not much TF can do about that. |
Here are the cloud logs. Just for the reference. We could do a ctrl+F for "JPEG" and find a lot of matches. 21:44:47.081 - Not a JPEG file: starts with 0x89 0x50 |
So that's either an older version of TensorFlow, a different of corruption (not related to PNGs), or a different bug. |
But I cloned it just yesterday and it got fixed after running the above script. |
Then it's a different bug. Care to diagnose why the format classification logic isn't working? https://github.com/tensorflow/tensorflow/blob/master/tensorflow/core/kernels/decode_image_op.cc#L156 |
@ Mohit-Ak |
Locking as this is a bug solved over 2 years ago. Please open a new issue, filling in all of the template |
How to fix this?
The text was updated successfully, but these errors were encountered: