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

configure script hardcodes location of cuda that makes it fail on OSX #111

Closed
delip opened this issue Nov 11, 2015 · 8 comments
Closed

configure script hardcodes location of cuda that makes it fail on OSX #111

delip opened this issue Nov 11, 2015 · 8 comments

Comments

@delip
Copy link

delip commented Nov 11, 2015

Cuda installation on OSX is at $CUDA_TOOLKIT_PATH/lib (not lib64), and on OSX the shared libraries are end in .dylib (not .so).

  if [ -e "$CUDA_TOOLKIT_PATH/lib64/libcudart.so.7.0" ]; then
    break
  fi
  echo "Invalid path to CUDA 7.0 toolkit. ${CUDA_TOOLKIT_PATH}/lib64/libcudart.so.7.0 cannot be found"
@vrv
Copy link

vrv commented Nov 11, 2015

Hey @delip, we currently don't have support for GPUs on Mac OS X.

Out of curiosity, what GPU are you trying to get working with TensorFlow?

@vrv vrv added the cuda label Nov 11, 2015
@dhess
Copy link

dhess commented Nov 11, 2015

I'm not OP, but in my case, it would be the D700 and D500 as found on Mac Pro (Late 2013) models (which require OpenCL, of course).

@zheng-xq
Copy link
Contributor

We don't support OpenCL right now. You need a NV GPU with at least compute 3.0 capability.

@dhess
Copy link

dhess commented Nov 11, 2015

I understand. I was just replying to @vrv's question about GPUs people are using on OS X.

@delip
Copy link
Author

delip commented Nov 11, 2015

Hey @vrv, I'm on a Mac Pro (late 2013) like Drew. Also, a lot of my friends
develop on their macbook pros.

On Wed, Nov 11, 2015 at 1:03 AM, Drew Hess notifications@github.com wrote:

I understand. I was just replying to @vrv https://github.com/vrv's
question about GPUs people are using on OS X.


Reply to this email directly or view it on GitHub
#111 (comment)
.

@vrv
Copy link

vrv commented Nov 11, 2015

@delip: so do you also have AMD GPUs?

@delip
Copy link
Author

delip commented Nov 11, 2015

That's correct. I also tried this on the latest iMac 5K which also contains
a AMD GPU (AMD Radeon R9 M290X)

On Wed, Nov 11, 2015 at 5:24 PM, Vijay Vasudevan notifications@github.com
wrote:

@delip https://github.com/delip: so do you also have AMD GPUs?


Reply to this email directly or view it on GitHub
#111 (comment)
.

@vrv
Copy link

vrv commented Nov 11, 2015

Gotcha, so that would require OpenCL support, which we're tracking here: #22. I'll de-dupe with that feature request.

@vrv vrv closed this as completed Nov 11, 2015
ilblackdragon pushed a commit to ilblackdragon/tensorflow that referenced this issue Mar 9, 2016
rmlarsen pushed a commit that referenced this issue Aug 7, 2017
… (#12041)

* [OpenCL] Fix for //tensorflow/python/kernel_tests:image_ops_test

* Conforming to Google style comments

* [OpenCL] Fixed RGBToHSVTest
ishay2b pushed a commit to ishay2b/tensorflow that referenced this issue Aug 8, 2017
…assification

* commit '6e054dbd4b741d5b8fa8af93fdd7c9b74ae67ce0': (511 commits)
  Fix tensordot with list of ints as axes (tensorflow#11959)
  Fix segfault when recording raw allocation returns nullptr (tensorflow#12074)
  [OpenCL] Fix for //tensorflow/python/kernel_tests:image_ops_test (tensorflow#111) (tensorflow#12041)
  Removing visited_node hash table - fixing multinode shape mismatch issue (tensorflow#12044)
  [OpenCL] Fixes core_rnn_cell_tests (tensorflow#12076)
  Added Convolutional LSTM (tensorflow#8891)
  Update monitors_test.py (tensorflow#12062)
  fix a typo in tf.nn.separable_conv2d's doc (tensorflow#12067)
  Fix cmake builds: (tensorflow#12048)
  Fix typo in RELEASE.md (tensorflow#12064)
  Fix typo (tensorflow#12069)
  Handle case where init node is not present in the frozen graph.
  Fix typo in datasets docstring
  tfdbg: fix a bug in string representation of SparseTensors
  BUILD dependency cleanup in tensorflow/stream_executor/cuda
  BUILD dependency cleanups. Rename RecvTensorAsync method to GrpcRecvTensorAsync to fix shadowing of method in Worker with a different signature.
  [tpu:profiler] Dump gzipped json trace.
  Minor cleanup
  tf.nn.separable_conv2d now supports data_format. Avoid unnecessary transposes in tf.layers.separable_conv2d (and implicitly in tf.contrib.layers.separable_conv2d).
  Add an identity initializer that works with partitioned variables.
  ...
lukeiwanski pushed a commit to codeplaysoftware/tensorflow that referenced this issue Oct 26, 2017
…sorflow#111)

* [OpenCL] Fix for //tensorflow/python/kernel_tests:image_ops_test

* Conforming to Google style comments

* [OpenCL] Fixed RGBToHSVTest
tensorflow-copybara pushed a commit that referenced this issue Sep 14, 2019
- add pattern to canonicalize affine.for loop bounds (using
  canonicalizeMapAndOperands)
- rename AffineForLoopBoundFolder -> AffineForLoopBoundFolder for
  consistency

Signed-off-by: Uday Bondhugula <uday@polymagelabs.com>

Closes #111

COPYBARA_INTEGRATE_REVIEW=tensorflow/mlir#111 from bondhugula:bound-canonicalize ee8fb7f43a7ffd45f6df3f53c95098d8b7e494c7
PiperOrigin-RevId: 269041220
xinan-jiang pushed a commit to xinan-jiang/tensorflow that referenced this issue Oct 4, 2019
- add pattern to canonicalize affine.for loop bounds (using
  canonicalizeMapAndOperands)
- rename AffineForLoopBoundFolder -> AffineForLoopBoundFolder for
  consistency

Signed-off-by: Uday Bondhugula <uday@polymagelabs.com>

Closes tensorflow#111

COPYBARA_INTEGRATE_REVIEW=tensorflow/mlir#111 from bondhugula:bound-canonicalize ee8fb7f43a7ffd45f6df3f53c95098d8b7e494c7
PiperOrigin-RevId: 269041220
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

4 participants