Skip to content
This repository was archived by the owner on Apr 23, 2026. It is now read-only.

Fix: Add missing torch and psutil dependencies to CUDA installation instructions - #3162

Merged
mergify[bot] merged 1 commit into
instructlab:mainfrom
jjaegii:main
Feb 20, 2025
Merged

Fix: Add missing torch and psutil dependencies to CUDA installation instructions#3162
mergify[bot] merged 1 commit into
instructlab:mainfrom
jjaegii:main

Conversation

@jjaegii

@jjaegii jjaegii commented Feb 18, 2025

Copy link
Copy Markdown
Contributor

📄 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

…nstructions

Signed-off-by: jjaegii <hn06038@gmail.com>
@mergify mergify Bot added the documentation Improvements or additions to documentation label Feb 18, 2025

@booxter booxter left a comment

Copy link
Copy Markdown
Contributor

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
@mergify mergify Bot removed the one-approval PR has one approval from a maintainer label Feb 20, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

documentation Improvements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants