search

asanakoy / deeppose_tf

DeepPose implementation on TensorFlow. Original Paper http://arxiv.org/abs/1312.4659
Other
144 stars 61 forks source link

Fix syntax to be compatible with Tensorflow v1.0 #1

Closed resonatevision closed 7 years ago

resonatevision commented 7 years ago

Remove deprecated function calls.

asanakoy commented 7 years ago

@resonatevision, thanks for the pull request! Have you tested the changes?

resonatevision commented 7 years ago

Yes. Tested on Tensorflow 1.0.1. Couldn't get the training to run without the changes. Also, there were some WARNINGS about the deprecating apis this month so I changed those too.

resonatevision commented 7 years ago

Actually I am having trouble with chainer now. Don't know if it is related to the changes I made to global counter. It gives me I/O error in self._invoke_prefetch() when evaluate_pcp is called. Could be completely different issue. But you can hold off on merging if you have never seen this before. Update: I reverted back the global variable changes. Still seeing the evaluate_pcp problem. It's something else. Any thoughts?

asanakoy commented 7 years ago

This is not related, I think. As far as I remember your problem is related to chainer multi-threading.

resonatevision commented 7 years ago

Yes. Exactly. Any thoughts on how you solved it?

resonatevision commented 7 years ago

I fixed it by replacing the MultiprocessIterator with SerialIterator. Obviously will impact training speed but that's the best I could do for now.

asanakoy commented 7 years ago

Sorry, I'm quiet busy now with other project, but I will check your pull request and your problem in 2 weeks :)

asanakoy commented 7 years ago

I upgraded to tensorflow r1.0 using script provided with tensorflow.