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

More portable conda environment #8

Open
wants to merge 308 commits into
base: main
Choose a base branch
from
Open

Conversation

imgeorgiev
Copy link

I tried installing the conda environment coming with this project on my fairly standard machine:

  • Distributor ID: Ubuntu
  • Description: Ubuntu 22.04.1 LTS
  • Release: 22.04
  • Codename: jammy
  • Kernel: 5.15.0-52-generic
  • Nvidia Driver Version: 515.65.01
  • CUDA Version: 11.7
  • RTX 2080 Ti

However, I ran into two cascading issues:

  1. The repository seems to assume that you have the complete CUDA dev kit installed on a system level. Although this is a fair assumption, I think it decreases the portability of the project, especially in the current world of researchers having dozens of different conda envs with different CUDA versions. I addressed this issue by adding the cuda-toolkit=11.3 package from the official nvidia conda channel
  2. After installing the correct toolkit for this project, I found out that it was incompatible with my system-installed gcc v11.3.0 (standard ubuntu distribution). The nvcc version coming with CUDA 11.3 is only compatible with gcc/g++ <10.0. Thus I added the cxx-compiler dependency from the conda-forge channel.

(Please correct me if my understanding of these issues is wrong)

I tested this on the machine listed above as well as an Ubuntu 20.04 system. Seems to be working as expected using test_env.py and train_shac.py

To summarise, this PR only increases the portability of the project with respect with respect to its dependencies and doesn't alter the functionality.

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

Successfully merging this pull request may close these issues.

2 participants