Skip to content

Commit 4f7c970

Browse files
physicsrobclaude
andcommitted
Hygiene: revive the format-on-edit hook, gitignore dist and lint caches
The PostToolUse hook still invoked black at the pre-restructure venv path, silently no-opping since the ruff migration; it now runs ruff format from the workspace venv. Also corrects the test-onnx group comment: torchwright_doom's onnxruntime-gpu moved from its main dependencies to an opt-in test-onnx group. Test Suite Status: Unknown Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
1 parent ade21d4 commit 4f7c970

3 files changed

Lines changed: 8 additions & 6 deletions

File tree

.claude/settings.json

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -9,7 +9,7 @@
99
"hooks": [
1010
{
1111
"type": "command",
12-
"command": "F=$(jq -r '.tool_input.file_path // empty'); case \"$F\" in *.py) /home/rob/workspace/torchwright/.venv/bin/black --quiet --fast \"$F\" 2>/dev/null ;; esac; true"
12+
"command": "F=$(jq -r '.tool_input.file_path // empty'); case \"$F\" in *.py) /home/rob/workspace/torchdoom/.venv/bin/ruff format --quiet \"$F\" 2>/dev/null ;; esac; true"
1313
}
1414
]
1515
}

.gitignore

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -3,6 +3,9 @@ __pycache__
33
.venv
44
*.egg-info
55
.pytest_cache
6+
.mypy_cache
7+
.ruff_cache
8+
dist/
69
*.onnx
710
*.gif
811
*.png

pyproject.toml

Lines changed: 4 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -78,11 +78,10 @@ dev = [
7878
]
7979
# CPU onnxruntime for the ONNX-runtime and HF-parity tests (the OnnxTokenModule
8080
# oracle in onnx_load.py, and everything that compares against it). This is a
81-
# DEDICATED non-default group, deliberately NOT in `dev`: the umbrella's
82-
# `uv sync --all-packages` installs every member's dev group alongside
83-
# torchwright_doom's `onnxruntime-gpu` main dependency, and the CPU and GPU
84-
# builds collide on the same `onnxruntime` import path. Keeping it out of `dev`
85-
# lets the local workspace use the GPU build while the Modal test image
81+
# DEDICATED non-default group, deliberately NOT in `dev`: torchwright_doom
82+
# carries `onnxruntime-gpu` (in its own opt-in `test-onnx` group), and the CPU
83+
# and GPU builds collide on the same `onnxruntime` import path. Keeping it out
84+
# of `dev` lets the local workspace use the GPU build while the Modal test image
8685
# (modal_image.py) opts in explicitly via `--group test-onnx`. Pinned in
8786
# lockstep with the gpu extra so the CPU
8887
# oracle the parity tests compare against matches the 1.26.0 runtime contract;

0 commit comments

Comments
 (0)