-
-
Notifications
You must be signed in to change notification settings - Fork 606
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
tensorflow error: you must feed a value for placeholder tensor 'placeholder_8' with dtype float #33
Comments
Yes, this issue is a valid one with Flux. We changed our approach to gradients to make things more consistent but were waiting on malmaud/TensorFlow.jl#215 to complete it. Should be fixed soon. |
Thanks for the update. Indeed it seems that TensorFlow merged their pull request already. In that case I'll wait rather than trying to get this going through some hack. By the way, the warnings about the dot operators seem very hard to fix. Will this require some way of passing multiple function arguments to |
That's one option. Right now I'm trying to implement #31 in DataFlow.jl, which is a slightly different design. With that you'd dispatch on |
So now DataFlow just interprets |
I am still having this issue, has it been resolved on some branch other than the latest tagged version perhaps? |
This should be fixed in the latest release. |
I'm probably getting a bit ahead of things here since I'm using master of
TensorFlow.jl
as I'm on 0.6 and the latest release doesn't work. TensorFlow itself seems to be working just fine on master.Anyway, I get the following error when attempting to train the MNIST example:
As I suspect this is mostly the fault of my insisting on using TensorFlow master as a result of using 0.6, I'll probably look into patching this myself until the 0.6 ecosystem matures.
Update: It seems that
placeholder_8
is a placeholder that is getting added to the TensorFlow graph object, but is disconnected from the actual graph (i.e. not used by any functions). Again, this is in the basic MNIST example. Apparently, it is the placeholder for the gradients.The text was updated successfully, but these errors were encountered: