Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@jjaegii
Copy link
Contributor

@jjaegii jjaegii commented Feb 18, 2025

📄 Description of Changes:

This PR updates the Install with Nvidia CUDA section in the README to prevent installation failures caused by missing dependencies.

Issue

Users attempting to install instructlab[cuda] may encounter the following errors:

  • ModuleNotFoundError: No module named 'torch'
  • ModuleNotFoundError: No module named 'psutil'

Fix

  • Updated the installation steps to explicitly install torch and psutil before running pip install instructlab[cuda].
  • Ensured that these dependencies are in place to prevent package resolution failures.

Changes Made

  • Modified the Install with Nvidia CUDA section in README to include:
    pip install torch psutil
    before installing instructlab[cuda].

Issue resolved by this Pull Request:
Resolves # (Add issue number if applicable)


🛠️ How to Test

  1. Set up a fresh virtual environment:
    python -m venv venv && source venv/bin/activate
  2. Follow the updated installation steps from the README.
  3. Verify that torch and psutil are installed correctly:
    python -c "import torch; print(torch.__version__)"
    Expected output: Torch version installed correctly.
  4. Verify that instructlab[cuda] installs without issues.

✅ Checklist

@mergify mergify bot added the documentation Improvements or additions to documentation label Feb 18, 2025
Copy link
Contributor

@booxter booxter left a comment

Choose a reason for hiding this comment

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

Thanks. This is the same issue that is tracked in #1821

@mergify mergify bot added the one-approval PR has one approval from a maintainer label Feb 18, 2025
@mergify mergify bot merged commit 41daf12 into instructlab:main Feb 20, 2025
7 checks passed
@mergify mergify bot removed the one-approval PR has one approval from a maintainer label Feb 20, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants