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.

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

tfdot would not work with Python3 #1

Open
rkuo opened this issue Jun 16, 2016 · 10 comments
Open

tfdot would not work with Python3 #1

rkuo opened this issue Jun 16, 2016 · 10 comments

Comments

@rkuo
Copy link

rkuo commented Jun 16, 2016

I installed TensorFlow with Anaconda Python3 version. But there is not output from tfdot. I do not know whether is just Python2/3 problem or not.

  • I removed op from TensorFlow's api in tfdot, which passed the compile error.
  • I got off some error message by modifying some print statements in notebook.

tfdot is very nice feature, which reduce the dependence on TensorBoard. Can you update tfdot?

@tjwei
Copy link
Owner

tjwei commented Jun 17, 2016

Just got my laptop back today and installed tensorflow. If it is not too difficult, I think it can be fixed by tomorrow.
The op issue indicates that there might be some internal API change.

@tjwei
Copy link
Owner

tjwei commented Jun 18, 2016

The nvidia driver was causing kernel issue on my laptop and upgrading ubuntu screwed up the system. Not a big deal but need some time.

@rkuo
Copy link
Author

rkuo commented Jun 18, 2016

Please do not feel pressured about this. I am happy you are willing to update this. I can wait.

@tjwei
Copy link
Owner

tjwei commented Jun 19, 2016

:) I am going to upgrade the system sooner or later anyway. I am actually happy to have some motivation to do it.

@tjwei
Copy link
Owner

tjwei commented Jun 21, 2016

Upgraded to ubuntu 16.04 and latest tensorflow. Now the two major notebook (TensorFlowBasic.ipynb and MNIST.ipynb) and tfdot should work for both python 2 and python 3.

@rkuo
Copy link
Author

rkuo commented Jun 21, 2016

I must made mistake somewhere. I re-clone the repo. I could run TensorFlowBasic without any error message, but I did not have any output from tfdot. Any tip for me to check my installation?

@tjwei
Copy link
Owner

tjwei commented Jun 21, 2016

You need to install graphviz binary and graphviz python module.
On ubuntu, you need both
sudo apt-get install graphviz
and
sudo pip3 install graphviz

@rkuo
Copy link
Author

rkuo commented Jun 21, 2016

I think I got it working. Thank you.

It printed out a lot more details than I expected (more than your demo video)
Since I used Mac, and I am new to all these, so I write down some notes here.

Reference from google:
http://enterprise-architecture.org/downloads?id=208
https://coderwall.com/p/8pajlg/fixing-library-not-loaded-usr-local-lib-libltdl-7-dylib

brew install graphviz  (I think I also did `conda install graphviz`, one of three steps is redundant!!)
pip install graphviz

check

➜  bin git:(master) pip list | grep graphviz
graphviz (0.4.10)
➜  bin git:(master) conda list | grep graphviz
graphviz                  2.38.0

try it

➜  bin git:(master) dot -h
dyld: Library not loaded: /usr/lib/libltdl.7.dylib
  Referenced from: /Users/rkuo/anaconda/bin/dot
  Reason: image not found
[1]    23225 trace trap  dot -h
➜  bin git:(master) brew install libtool --universal
...cut ...

got error msg, then

➜  bin git:(master) brew link libtool
Warning: Already linked: /usr/local/Cellar/libtool/2.4.6
To relink: brew unlink libtool && brew link libtool

test

➜  bin git:(master) dot -V
dot - graphviz version 2.38.0 (20140413.2041)

OK, run notebook, works!!! :-) Thank you again.

@rkuo
Copy link
Author

rkuo commented Jun 21, 2016

Can you double check the code? Why the system prints out randim_normal, weight, .. when we just add two matrices together?

@tjwei
Copy link
Owner

tjwei commented Jun 23, 2016

Maybe it is because tensorflow did a lot of work behind scenes. I will run and check it again.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants