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

Try to build with Cuda 10.2 on Linux #111

Closed
wants to merge 1 commit into from
Closed

Conversation

karllessard
Copy link
Collaborator

DO NOT MERGE, this is just a test for now

@karllessard karllessard added DON'T MERGE This pull request is temporarily on hold CI build labels Sep 11, 2020
@roywei
Copy link
Contributor

roywei commented Sep 16, 2020

Hi, I was also looking into cuda 10.2 support. Even if I installed cuda 10.2 on a local machine, and successfully built from source. TF2.3 (dso_loader.cc) is still trying to load cuda 10.1 as it's hard coded. Need to symlink cuda10.2 as 10.1.

@karllessard
Copy link
Collaborator Author

Hi @roywei ,

As we have discussed with @frankfliu during our last meeting, we try to keep the build of the native libraries as close as what is being delivered in the Python wheels and try to avoid tampering with the source code of the main TF repository, since it would be hard for us to guarantee any stability by doing so.

We also have discussed starting to have additional "experimental" builds, in which case no stability will be guaranteed and users would be well aware of it. Cuda 10.2-enabled can be one of them. But right now, we already have a lot of issues with our build platform (GitHub actions) and adding more stress on it is probably not something that we will look at in the near future. So I would say that having custom builds (where you can patch the code as you want) is probably still the best approach until TensorFlow supports Cuda 10.2 officially.

There is already this issue reported in TensorFlow core where you can share your ideas as well: tensorflow/tensorflow#38194

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
DON'T MERGE This pull request is temporarily on hold
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants