-
Notifications
You must be signed in to change notification settings - Fork 751
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
Add tensorflow with gpu support for windows #567
Conversation
Unfortunately, it's not currently possible to "merge" annotation values as it would make it hard to override them. If you have a good idea to enhance JavaCPP that would allow both concatenating and overriding results easily with annotations like this, let me know! Other than that, we'll need to leave |
Added the PYTHON_EXECUTABLE again. |
I can't get it to build here. The Python script don't seem to be running properly. Which distribution of Python are you using? |
Anaconda with python 3.6 was in the path of the system. Which python script has problems? |
No, it doesn't seem to be because of the version of Python, it doesn't work with 3.6.5 either. I'm getting this kind of error:
A Python script somewhere is supposed to create those .cmake.pre-gen.tmp files, but they don't get (all) created for some reason here on my installation of Windows 10. |
All the failing paths are longer than 260 characters, so this has something to do with the MAX_PATH limit: |
I had path length problems somewhere else, so my directory is just called: f:/G/JP/tensorflow/. But if this is really an issue we should check it somewhere and tell the user beforehand. Instead of building tensorflow in contrib/cmake/build, we could build it somewhere else. The CI scripts of tensorflow for example, create a "cmake_build" directory at the top level directory. |
This is working well for me, thanks! I've uploaded snapshots manually and will keep doing that until we figure out what to do about the CI service (issue #568). Let's hope they get Bazel working before long; a build time of 4 hours is pretty ridiculous. As far as I know, we're only missing support for static libraries: bazelbuild/bazel#1920 BTW, keep an eye on tensorflow/tensorflow#17390. Google is expending effort to build a community around support for Java. |
@saudet Thanks for pointing me to #17390. There are too many Tensorflow Github tickets to follow. I will be at a conference in Yokohama (Japan) next week and have same holidays afterwards. When I'm back in office I could start working on examples for feature.pb.h, example.pb.h, record_reader.h, record_writer.h. The other day I was building Tensorflow with Bazel but had the feeling it takes much longer on Windows and uses less available resources as cmake. |
Which conference is that BTW? I live in Japan, not too far from Yokohama/Tokyo... |
It is the ICMR 2018 from 11th to 14th of June. After some traveling around the country I will be back in Yokohama from 25th-30th and will be in Tokyo from 30th-7th. If you want to meet we better discuss the details via email. |
Tested with cuda 9.0 + cudnn 7.0 and cuda 9.2 + cudnn 7.1
Is it possible to merge attributes from other platform annotation? Right now I had to copy the "link" attributes from the "windows" platform to the "windows-x86_64" with "-gpu" extension.
Furthermore it might be better to exchange the "-gpu" extension with "-cuda90" or "-cuda92". On the other hand all the different configurations are horrible (e.g. cuda version, cudnn version, cpu instruction set) to maintain.