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

can current fold work with tensorflow 1.3? Is it tested? #85

Open
xiaoyunwu opened this issue Sep 28, 2017 · 8 comments
Open

can current fold work with tensorflow 1.3? Is it tested? #85

xiaoyunwu opened this issue Sep 28, 2017 · 8 comments

Comments

@xiaoyunwu
Copy link

No description provided.

@evelkey
Copy link

evelkey commented Sep 28, 2017

No, I tried it and found out some cuda ops are not compatible.
For quick solution I merged different versions of tensorflow to have the functionality I need.

@delesley
Copy link
Contributor

delesley commented Sep 28, 2017 via email

@KazuhiraDZ
Copy link

Hi, I want to compile from source, can you tell me how to compile the latest version? is it the same as how we compile tensorflow from source?

@evelkey
Copy link

evelkey commented Sep 29, 2017

Hi, after compiling both tensorflow_fold and tensorflow from sources, it is working fine with tf version 1.3.
It's quite tricky to get it to work, because the tensorflow version which is included in fold has an error with a checksum.
You can find build instructions here: sources
Right now it's building with only bazel 5.4 for me.
To make it build you have to remove the checksum from tf with the following command:
sed -ri "/^\W+sha256 = \"[^\"]+\"\W+$/d" tensorflow/workspace.bzl (issue)

@KazuhiraDZ
Copy link

KazuhiraDZ commented Oct 3, 2017

thank you for help. I have run the code to remove the checksum,but, it still has something wrong,like:
Python Configuration Error: 'PYTHON_BIN_PATH' environment variable is not set
and referenced by '@org_tensorflow//util/python:python_headers'
ERROR: Analysis of target '//tensorflow_fold/loom:pywrapweaver' failed; build aborted
INFO: Elapsed time: 1.325s
FAILED: Build did NOT complete successfully (8 packages loaded)
Fetching http://mirror.bazel.build/.../swig/swig-3.0.8/swig-3.0.8.tar.gz
Fetching http://mirror.bazel.build/.../eigen/eigen/get/f3a22f35b044.tar.gz
Fetching http://mirror.bazel.build/.../backports.weakref-1.0rc1.tar.gz
Fetching http://mirror.bazel.build/.../archive/release-1.8.0.zip
ERROR: Couldn't start the build. Unable to run tests
can you help me the order how to build it?
and every time it have different log

@evelkey
Copy link

evelkey commented Oct 3, 2017

Try exporting the variable replacing "usr/bin/python3" with your own python path:
export PYTHON_BIN_PATH="/usr/bin/python3"

@KazuhiraDZ
Copy link

Thank you! It does work in cpu pc! I will install it in my gpu server...

@shengc
Copy link

shengc commented Oct 5, 2017

I have the following errors trying to build from source. Any idea how to fix them ?

/usr/include/string.h:544: error: '__locale_t' has not been declared
/usr/include/string.h:545: error: nonnull argument references non-pointer operand (argument 1, operand 3)
/usr/include/string.h:548: error: '__locale_t' has not been declared
/usr/include/string.h:549: error: nonnull argument references non-pointer operand (argument 1, operand 4)

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

5 participants