-
Notifications
You must be signed in to change notification settings - Fork 74k
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
cuda/cuda_config.h missing when compiling custom ops with nvcc #12860
Comments
the same problem, I want to know if there is any better way to solve this problem |
@allenlavoie, have you looked at this as part of your library cleanup? |
I haven't. I may have run into it when I was following our custom op documentation. It sounds like worst case we could add a copy in https://github.com/tensorflow/tensorflow/blob/master/tensorflow/tools/pip_package/build_pip_package.sh ? Or maybe a genrule before that. |
I have the same problem while compiling with nvcc. Also is because But when I copy the file, i get the following error:
|
The commit 2c598e8 destroys my makefile, as well. This commit consistly changes in several places the includes -#include "third_party/gpus/cuda/include/cuda.h"
+#include "cuda/include/cuda.h" Which gives me the error message:
compiling user-ops with GPU does not work since 1.3.1. Where lives that guy, tensorflower-gardener? He is the one who usually introduces those breaking changes! |
@PatWie +1 tensorflow/core/kernels/lrn_op.cc:34:10: fatal error: 'cuda/include/cuda.h' file not found |
With TF 1.3, I only find
|
I also encounter the same problem in TF 1.3.0. This error is introduced with |
I also encounter this problem in TF 1.4.0, any clue ? |
Also stucking on this problem in TF 1.4.0... |
@CR-Ko Downgrading to 1.2.0 "solved" the issue for me |
I have a similar issue, resulting in: |
No change here. I currently stick to TF1.2 compiled from source (I always compile the library from source). It is frustrating. @allenlavoie or @aselle can you give us at least a hint? edit: I tracked it down to commit 2c598e8. This is a MWE for v1.2 for reproducing the error. It works under TF1.2 but not TF1.4. |
@Queequeg92 installing from source did not fix the problem, ended up manually adding paths to .h files as additional include directories and copying |
It has been 14 days with no activity and the |
Hello. I also have the problem with T.F 1.4 although it worked in T.F 1.2. Have anyone fix it with T.F 1.4 |
Folding all the duplicates into #15002 |
@Queequeg92 I have it working under v1.5rc0 now, see https://github.com/cgtuebingen/tf_custom_op which requires installing from source or copying the |
@PatWie Thank you! I'll have a try. |
@PatWie I have installed v1.5 from source successfully. No cuda_config.h error, but still cuda.h error. 😫 |
I solved this problem by commenting the line *** #include "cuda/cuda_config.h" *** in the "dso_loader.h" file. And the custom op works normally. |
@Queequeg92 # use cuda
find_package(CUDA 9.0 EXACT REQUIRED)
set(CUDA_SAMPLE_INC "${CUDA_INCLUDE_DIRS}/../samples/common/inc")
message(STATUS "CUDA_INCLUDE_DIRS: ${CUDA_INCLUDE_DIRS}")
include_directories(SYSTEM "${CUDA_INCLUDE_DIRS}/../../") are for. But the latest version of TF is broken again (#15002 (comment)) @xysmlx |
System information
Yes, see below.
Ubuntu 16.04
Source
v1.3.0-0-g9e76bf324 1.3.0
3.5.2
0.5.4
8.0.44 / 5.1.5
Any.
See below.
Describe the problem
When compiling a custom op using nvcc, which includes
tensorflow/core/util/cuda_kernel_helper.h
, I get the following error:Copying
cuda_config.h
to/site-packages/tensorflow/include/tensorflow/stream_executor/cuda
solves the problem.The same issue has been observed by several other users in #6602 (see the comments added after the issue was closed).
The text was updated successfully, but these errors were encountered: