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

Tensorflow 1.10 C++ project errors : logging.h error in line 229 #22646

Closed
EdwardVincentMa opened this issue Oct 1, 2018 · 17 comments
Closed
Assignees
Labels
1.9 stale This label marks the issue/pr stale - to be closed automatically if no activity stat:awaiting response Status - Awaiting response from author subtype:bazel Bazel related Build_Installation issues subtype:windows Windows Build/Installation Issues

Comments

@EdwardVincentMa
Copy link

EdwardVincentMa commented Oct 1, 2018

I have generated libtensorflow_cc.so(V1.9, V1.10 and V1.11) file successully with bazel in Win10 enviroment, but when I use this TensorFlow API in VS2017, there are errors as bellow:

image

image

I also build TensorFlow1.8 C++ version API, and it used OK without these errors in VS2017.

C:\tf\tensorflow1.10\bazel-tensorflow1.10\tensorflow\core\platform\default\logging.h
I don not know why it occur these errors. Tensorflow1.8 did not have these errors.
I build tensorflow1.10 by bazel with Windows10-gpu version.

@EdwardVincentMa EdwardVincentMa changed the title Tensorflow 1.10 : logging.h error in line 229 Tensorflow 1.10 C++: logging.h error in line 229 Oct 1, 2018
@EdwardVincentMa EdwardVincentMa changed the title Tensorflow 1.10 C++: logging.h error in line 229 Tensorflow 1.10 C++ project errors : logging.h error in line 229 Oct 2, 2018
@tensorflowbutler tensorflowbutler added the stat:awaiting response Status - Awaiting response from author label Oct 2, 2018
@tensorflowbutler
Copy link
Member

Thank you for your post. We noticed you have not filled out the following field in the issue template. Could you update them if they are relevant in your case, or leave them as N/A? Thanks.
Have I written custom code
OS Platform and Distribution
TensorFlow installed from
TensorFlow version
Bazel version
CUDA/cuDNN version
GPU model and memory
Exact command to reproduce
Mobile device

@Harshini-Gadige
Copy link

@EdwardVincentMa Hi, have you tried with tensorflow V 1.11 ? Let us know if you are facing similar issue with V 1.11

@EdwardVincentMa
Copy link
Author

@Harshini-Gadige Yes, V1,11 has the same issues.

@EdwardVincentMa
Copy link
Author

@Harshini-Gadige And where is the TensorFlow's lib(build by bazel) ?

@tensorflowbutler tensorflowbutler removed the stat:awaiting response Status - Awaiting response from author label Oct 5, 2018
@EdwardVincentMa
Copy link
Author

@Harshini-Gadige Hello, how to solve this problem ?

@Harshini-Gadige
Copy link

@aselle Hi, could you please look into this ?

@EdwardVincentMa
Copy link
Author

@aselle Hello, how to solve this problem?

@yifeif
Copy link
Contributor

yifeif commented Oct 15, 2018

That file has not changed for a while. @meteorcloudy any clue? Also cc @gunan.

@gunan
Copy link
Contributor

gunan commented Oct 16, 2018

I think in visual studio, you may need to add search directories for include paths.
Our integration tests that use custom ops has been passing on this, and other users have not reported similar issues. So I am pretty sure this is not a bug in TF.

I would recommend reaching out to stackoverflow, or using google for looking for how to set up visual studio header include directories.

@meteorcloudy
Copy link
Member

@EdwardVincentMa How did you build the TensorFlow1.8 C++ version API? With CMake? Can you be more specific so I can understand what's missing in the bazel build.

@visionbang
Copy link

@gunan I check the project. The problem is always exist.

@HackersSpirit
Copy link

@gunan: Getting the similar issue. I using Qt creator insted of visual studio

@wang1ang
Copy link

wang1ang commented Nov 16, 2018

Add NOMINMAX in Preprocessor Definitions (/DNOMINMAX) when compiling.
image

@Fighting-JJ
Copy link

Add NOMINMAX in Preprocessor Definitions (/DNOMINMAX) when compiling.
image

加上这行代码,可以work。不过类似于非极大值抑制之类的操作会出现问题

@mohantym mohantym added the subtype:windows Windows Build/Installation Issues label Dec 3, 2021
@mohantym
Copy link
Contributor

mohantym commented Dec 3, 2021

Hi @EdwardVincentMa !
It seems you are using older versions(1.x versions) of Tensorflow which is not supported any more. Have you tried latest version yet ? You can use tested configurations to build older version using Bazel.Thanks!

@mohantym mohantym added the stat:awaiting response Status - Awaiting response from author label Dec 3, 2021
@google-ml-butler
Copy link

This issue has been automatically marked as stale because it has no recent activity. It will be closed if no further activity occurs. Thank you.

@google-ml-butler google-ml-butler bot added the stale This label marks the issue/pr stale - to be closed automatically if no activity label Dec 10, 2021
@google-ml-butler
Copy link

Closing as stale. Please reopen if you'd like to work on this further.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
1.9 stale This label marks the issue/pr stale - to be closed automatically if no activity stat:awaiting response Status - Awaiting response from author subtype:bazel Bazel related Build_Installation issues subtype:windows Windows Build/Installation Issues
Projects
None yet
Development

No branches or pull requests