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

Add cuda-toolkit package dependency to faiss-gpu and faiss-gpu-raft conda build recipes #3440

Closed
wants to merge 1 commit into from

Conversation

ramilbakhshyiev
Copy link
Contributor

Summary:

This change is required to unblock the migration to GitHub Actions. cuda-toolkit was only specified in the libfaiss package and it was not available in faiss-gpu or faiss-gpu-raft. This currently works on CircleCI because the runner image has CUDA toolkit of the needed version installed on the system and the build logic falls back to that but breaks on GitHub Actions because their runner images do not come with CUDA toolkit pre-installed.

Differential Revision: D57371597

@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D57371597

@ramilbakhshyiev ramilbakhshyiev changed the title Add cuda-toolkit package dependency to faiss-gpu and faiss-gpu-raft conda build recipes (#3436) Add cuda-toolkit package dependency to faiss-gpu and faiss-gpu-raft conda build recipes May 15, 2024
…onda build recipes

Summary: This change is required to unblock the migration to GitHub Actions. `cuda-toolkit` was only specified in the `libfaiss` package and it was not available in `faiss-gpu` or `faiss-gpu-raft`. This currently works on CircleCI because the runner image has CUDA toolkit of the needed version installed on the system and the build logic falls back to that but breaks on GitHub Actions because their runner images do not come with CUDA toolkit pre-installed.

Differential Revision: D57371597
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D57371597

ramilbakhshyiev added a commit that referenced this pull request May 15, 2024
…onda build recipes (#3440)

Summary:

This change is required to unblock the migration to GitHub Actions. `cuda-toolkit` was only specified in the `libfaiss` package and it was not available in `faiss-gpu` or `faiss-gpu-raft`. This currently works on CircleCI because the runner image has CUDA toolkit of the needed version installed on the system and the build logic falls back to that but breaks on GitHub Actions because their runner images do not come with CUDA toolkit pre-installed.

Differential Revision: D57371597
@facebook-github-bot
Copy link
Contributor

This pull request has been merged in 2050a03.

@ramilbakhshyiev ramilbakhshyiev deleted the export-D57371597 branch May 15, 2024 20:49
abhinavdangeti pushed a commit to blevesearch/faiss that referenced this pull request Jul 12, 2024
…onda build recipes (facebookresearch#3440)

Summary:
Pull Request resolved: facebookresearch#3440

This change is required to unblock the migration to GitHub Actions. `cuda-toolkit` was only specified in the `libfaiss` package and it was not available in `faiss-gpu` or `faiss-gpu-raft`. This currently works on CircleCI because the runner image has CUDA toolkit of the needed version installed on the system and the build logic falls back to that but breaks on GitHub Actions because their runner images do not come with CUDA toolkit pre-installed.

Reviewed By: junjieqi

Differential Revision: D57371597

fbshipit-source-id: 8bededd53e2528f033fac797b296d74b47f9403e
aalekhpatel07 pushed a commit to aalekhpatel07/faiss that referenced this pull request Oct 17, 2024
…onda build recipes (facebookresearch#3440)

Summary:
Pull Request resolved: facebookresearch#3440

This change is required to unblock the migration to GitHub Actions. `cuda-toolkit` was only specified in the `libfaiss` package and it was not available in `faiss-gpu` or `faiss-gpu-raft`. This currently works on CircleCI because the runner image has CUDA toolkit of the needed version installed on the system and the build logic falls back to that but breaks on GitHub Actions because their runner images do not come with CUDA toolkit pre-installed.

Reviewed By: junjieqi

Differential Revision: D57371597

fbshipit-source-id: 8bededd53e2528f033fac797b296d74b47f9403e
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.

2 participants