Skip to content

Conversation

malfet
Copy link
Contributor

@malfet malfet commented Nov 16, 2021

Add CUDA_HOME to PATH to fix Cmake builds
Define FORCE_CUDA only when CUDA_PATH is present (or if RocM)

cc @seemethere

Add CUDA_HOME to PATH to fix Cmake builds
Define `FORCE_CUDA` only when CUDA_PATH is present (or if RocM)
@facebook-github-bot
Copy link
Contributor

facebook-github-bot commented Nov 16, 2021

💊 CI failures summary and remediations

As of commit 317b187 (more details on the Dr. CI page):


  • 6/6 failures introduced in this PR

6 failures not recognized by patterns:

Job Step Action
CircleCI torch_onnx_test Checkout code 🔁 rerun
CircleCI python_type_check Checkout code 🔁 rerun
CircleCI unittest_linux_cpu_py3.8 Checkout code 🔁 rerun
CircleCI circleci_consistency Checkout code 🔁 rerun
CircleCI binary_linux_conda_py3.6_cu102 Checkout code 🔁 rerun
CircleCI clang_format Checkout code 🔁 rerun

This comment was automatically generated by Dr. CI (expand for details).

Please report bugs/suggestions to the (internal) Dr. CI Users group.

Click here to manually regenerate this comment.

Copy link
Contributor

@datumbox datumbox left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, thanks @malfet.

@malfet malfet merged commit 81fe60c into main Nov 16, 2021
@malfet malfet deleted the malfet/add-CUDA_HOME-to-PATH branch November 16, 2021 17:35
@github-actions
Copy link

Hey @malfet!

You merged this PR, but no labels were added. The list of valid labels is available at https://github.com/pytorch/vision/blob/main/.github/process_commit.py

facebook-github-bot pushed a commit that referenced this pull request Nov 17, 2021
…4944)

Summary:
Add CUDA_HOME to PATH to fix Cmake builds
Define `FORCE_CUDA` only when CUDA_PATH is present (or if RocM)

Reviewed By: datumbox

Differential Revision: D32470483

fbshipit-source-id: 08c6cdbba6b9e3fed353bdeb350a47f2907461b6
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants