Skip to content

Fix/nvidia install#25

Merged
growlf merged 3 commits into
mainfrom
fix/nvidia-install
May 12, 2026
Merged

Fix/nvidia install#25
growlf merged 3 commits into
mainfrom
fix/nvidia-install

Conversation

@growlf
Copy link
Copy Markdown
Owner

@growlf growlf commented May 12, 2026

Summary

Type of change

  • Bug fix
  • New feature / enhancement
  • Documentation update
  • Refactor / cleanup
  • CI / tooling change

Related issues

Testing

  • docker compose config passes with no errors
  • Shell scripts pass shellcheck
  • Manually tested on Intel Arc hardware (if relevant)

Checklist

  • My changes follow the project's code style
  • I have updated documentation where necessary
  • I have not committed secrets or real credentials
  • .env.example is up to date (if env vars were added/removed)

Luma (Enclave AI) and others added 2 commits May 12, 2026 22:56
SVG file is already on main (via PR #23) but was not embedded in the
README — the placeholder comment was left behind. Replace it with the
actual image reference.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
install.sh was hardcoded for Intel Arc — check_intel_gpu() blocked
any non-Intel hardware with a fatal error. This adds auto-detection
(nvidia-smi → NVIDIA, Intel DRI vendor 0x8086 → Arc, fallback →
CPU-only), conditional GPU checks, and persists GPU_TYPE to .env.

start.sh updated to read GPU_TYPE from .env and handle all three
GPU paths. Systemd service template de-Arc'd (removed check-arc-gpu.sh
ExecStartPre and updated description).

Also: fixed docker exec ollama-arc → $OLLAMA_CONTAINER, removed
nonexistent qwen3.5:14b from MODELS_TO_PULL and OpenCode config.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@growlf growlf self-assigned this May 12, 2026
ShellCheck SC2034 warning: COMPOSE_FILES was set in a case block but
never referenced. The overlay selection is handled via GPU_TYPE written
to .env, which start.sh reads at runtime. Remove the dead variable.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@growlf growlf merged commit 622e584 into main May 12, 2026
1 check 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.

1 participant