Skip to content

fix(env): add cuda path and fix quote escaping#7

Merged
Simon-Tl merged 4 commits intomainfrom
dev/bing_li
Oct 29, 2025
Merged

fix(env): add cuda path and fix quote escaping#7
Simon-Tl merged 4 commits intomainfrom
dev/bing_li

Conversation

@libing0526
Copy link
Collaborator

Where:

  • type is one of: feat, fix, docs, refactor, perf, test, chore.
  • scope is optional and describes the part of the codebase affected (e.g., auth, ui, api).
  • concise message is a short description of the change (max 50 chars).

📝 Change Type

Please select the type of change this PR introduces (choose one or more):

  • feat: New feature.
  • fix: Bug fix.
  • docs: Documentation only changes.
  • refactor: A code change that neither fixes a bug nor adds a feature.
  • perf: Performance improvement.
  • test: Adding missing tests or correcting existing tests.
  • chore: Maintenance tasks (e.g., updating dependencies).

💡 Description

Fixed CUDA environment setup and PowerShell/CMD quote escaping issues for WSL command execution.

Key Changes

  1. Added dynamic PATH filtering to remove Windows paths and prepend CUDA path
  2. Fixed quote escaping issues by using single quotes for environment variables
  3. Implemented CUDA environment setup in venv activation for all command types

🔗 Related Issues

List any issues this PR closes or relates to:

  • Fixes CUDA toolkit not found in WSL environment
  • Resolves bash syntax errors caused by Windows PATH inheritance

✅ Checklist

Please ensure the following points are addressed before merging:

  • I have performed a self-review of my own code.
  • I have added/updated tests that prove my fix or feature works (if applicable).
  • I have updated the documentation (if necessary).
  • My code follows the project's style guidelines.

@libing0526 libing0526 requested a review from Simon-Tl October 29, 2025 14:32
@libing0526 libing0526 changed the title fix(env): add CUDA path and fix quote escaping fix(env): add cuda path and fix quote escaping Oct 29, 2025
@Simon-Tl Simon-Tl merged commit 60e3f03 into main Oct 29, 2025
2 of 3 checks passed
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