-
-
Notifications
You must be signed in to change notification settings - Fork 1.4k
fix: pin torchvision per matrix entry to prevent ABI drift #3653
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
Changes from all commits
a03aace
fe5e8ac
3a4ca3b
bf030c1
e74abaf
9bde764
256e374
38927c3
1f95d49
f720ced
ecca61c
3dd6eee
4407920
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -23,6 +23,7 @@ | |
| "AXOLOTL_EXTRAS": os.environ.get("AXOLOTL_EXTRAS", ""), | ||
| "AXOLOTL_ARGS": os.environ.get("AXOLOTL_ARGS", ""), | ||
| "PYTORCH_VERSION": os.environ.get("PYTORCH_VERSION", "2.6.0"), | ||
| "TORCHVISION_VERSION": os.environ.get("TORCHVISION_VERSION", "0.21.0"), | ||
|
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. 🧩 Analysis chain🏁 Script executed: #!/bin/bash
set -euo pipefail
python - <<'PY'
import re
from pathlib import Path
pp = Path("pyproject.toml").read_text()
torch_min = re.search(r'"torch>=([^"]+)"', pp).group(1)
tv_min = re.search(r'"torchvision>=([^"]+)"', pp).group(1)
print(f"pyproject minimums: torch>={torch_min}, torchvision>={tv_min}")
for path in ("cicd/single_gpu.py", "cicd/multigpu.py"):
txt = Path(path).read_text()
t = re.search(r'"PYTORCH_VERSION":\s*os\.environ\.get\("PYTORCH_VERSION",\s*"([^"]+)"\)', txt)
v = re.search(r'"TORCHVISION_VERSION":\s*os\.environ\.get\("TORCHVISION_VERSION",\s*"([^"]+)"\)', txt)
print(f"{path} defaults: torch={t.group(1)}, torchvision={v.group(1)}")
PY
rg -n 'TORCHVISION_VERSION' .github/workflows cicd docker || trueRepository: axolotl-ai-cloud/axolotl Length of output: 1714 Align default PyTorch/TorchVision versions with pyproject.toml constraints across CI files. The defaults in Suggested changes# cicd/single_gpu.py & cicd/multigpu.py
- "PYTORCH_VERSION": os.environ.get("PYTORCH_VERSION", "2.6.0"),
- "TORCHVISION_VERSION": os.environ.get("TORCHVISION_VERSION", "0.21.0"),
+ "PYTORCH_VERSION": os.environ.get("PYTORCH_VERSION", "2.9.1"),
+ "TORCHVISION_VERSION": os.environ.get("TORCHVISION_VERSION", "0.24.1"),
# docker/Dockerfile-uv-base
-ARG TORCHVISION_VERSION="0.21.0"
+ARG TORCHVISION_VERSION="0.24.1"🤖 Prompt for AI Agents |
||
| "BASE_TAG": os.environ.get("BASE_TAG", "main-base-py3.11-cu126-2.6.0"), | ||
| "CUDA": os.environ.get("CUDA", "126"), | ||
| "GITHUB_REF": os.environ.get("GITHUB_REF", "refs/heads/main"), | ||
|
|
||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🧩 Analysis chain
🏁 Script executed:
Repository: axolotl-ai-cloud/axolotl
Length of output: 189
Clarify that cu128/cu130 wheels are unavailable entirely for torchvision 0.24.1, not just aarch64.
The verification confirms no cu128/cu130 wheels exist for torchvision 0.24.1 on PyPI. The comment is technically accurate (aarch64 wheels are indeed absent), but the wording suggests aarch64-specific unavailability when these CUDA versions aren't available for any platform. Update the comment to clarify:
# torchvision 0.24.1 does not have cu128/cu130 wheels availablerather than implying aarch64 is uniquely restricted.🤖 Prompt for AI Agents