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

Update INSTALL.md with CUDA_HOME-related troubleshooting #140

Merged
merged 1 commit into from
Aug 6, 2024

Conversation

AmmoniumX
Copy link
Contributor

@AmmoniumX AmmoniumX commented Aug 4, 2024

This is referring to #137 , which in itself refers to a common problem during installation, mentioned on #19 .

Some users may encounter significant trouble installing the project, running into the error OSError: CUDA_HOME environment variable is not set. Please set it to your CUDA install root.. Simply adding the --no-build-isolation flag to the pip install, e.g. pip install --no-build-isolation -e ., usually solves this problem. However, this fix is not mentioned anywhere within the readmes or installation troubleshooting docs. This PR adds this recommendation into the INSTALL.md file under the "My installation failed with CUDA_HOME environment variable is not set " section, ensuring that more users are aware of this potential fix.

Examples of users experiencing related difficulties when installing:
#19
#41
#99
#133

@AmmoniumX AmmoniumX changed the title Updated INSTALL.md with CUDA_HOME-related troubleshooting Update INSTALL.md with CUDA_HOME-related troubleshooting Aug 4, 2024
Copy link
Contributor

@ronghanghu ronghanghu left a comment

Choose a reason for hiding this comment

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

Thank you!

@ronghanghu ronghanghu merged commit 511199d into facebookresearch:main Aug 6, 2024
1 check passed
fbcotter added a commit to wayveai/segment-anything-2 that referenced this pull request Sep 6, 2024
… into facebookresearch-main

* 'main' of github.com:facebookresearch/segment-anything-2: (40 commits)
  open `README.md` with unicode (to support Hugging Face emoji); fix various typos (facebookresearch#218)
  accept kwargs in auto_mask_generator
  Fix HF image predictor
  improving warning message and adding further tips for installation (facebookresearch#204)
  better support for non-CUDA devices (CPU, MPS) (facebookresearch#192)
  Update hieradet.py
  add Colab support to the notebooks; pack config files in `sam2_configs` package during installation (facebookresearch#176)
  also catch errors during installation in case `CUDAExtension` cannot be loaded (facebookresearch#175)
  Add interface for box prompt in SAM 2 video predictor (facebookresearch#174)
  Address comment
  Update hieradet.py
  Update docstrings
  Revert code snippet
  Updated INSTALL.md with CUDA_HOME-related troubleshooting (facebookresearch#140)
  Format using ufmt
  Update INSTALL.md (facebookresearch#156)
  Update README
  Make it optional to build CUDA extension for SAM 2; also fallback to all available kernels if Flash Attention fails (facebookresearch#155)
  Clean up
  Address comment
  ...
yanyilyu pushed a commit to yanyilyu/segment-anything-2 that referenced this pull request Sep 17, 2024
…search#140)

This is referring to facebookresearch#137 , which in itself refers to a common problem during installation, mentioned on facebookresearch#19 .

Some users may encounter significant trouble installing the project, running into the error `OSError: CUDA_HOME environment variable is not set. Please set it to your CUDA install root.`. Simply adding the `--no-build-isolation` flag to the pip install, e.g. `pip install --no-build-isolation -e .`, usually solves this problem. However, this fix is not mentioned anywhere within the readmes or installation troubleshooting docs. This PR adds this recommendation into the INSTALL.md file under the "My installation failed with `CUDA_HOME environment variable is not set` " section, ensuring that more users are aware of this potential fix.

Examples of users experiencing related difficulties when installing:
facebookresearch#19
facebookresearch#41
facebookresearch#99
facebookresearch#133
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