Skip to content

fix(highlight): run inference under torch.inference_mode and cap context size - #2

Merged
TC407-api merged 1 commit into
masterfrom
fix/highlight-inference-mode
Oct 8, 2026
Merged

TC407-api merged 1 commit into
masterfrom
fix/highlight-inference-mode

Conversation

@TC407-api

Copy link
Copy Markdown
Owner

The model's batch inference path is not wrapped in no_grad, so autograd kept every
layer's activations and memory scaled with input size. Measured on M1 (CPU, 8 GB
sandbox): 12k chars peaked 7.9 GB, 24k was OOM-killed. On 2026-10-06 the live service
reached ~31 GB and was OOM-killed 11 times; the memory storm left M1's network links
in systemd-networkd 'failed' state and the box dropped offline two days later when the
DHCP leases expired.

With inference_mode: typical 2.0 GB, 12k 2.4 GB, 24k 3.4 GB, 192k 5.5 GB, settling back
to ~2 GB after each request (150-request soak flat). Contexts over
HIGHLIGHT_MAX_CONTEXT_CHARS (default 200000) are refused with 413, and each request's
size is logged so the next incident names its input.

…ext size

The model's batch inference path is not wrapped in no_grad, so autograd kept every
layer's activations and memory scaled with input size. Measured on M1 (CPU, 8 GB
sandbox): 12k chars peaked 7.9 GB, 24k was OOM-killed. On 2026-10-06 the live service
reached ~31 GB and was OOM-killed 11 times; the memory storm left M1's network links
in systemd-networkd 'failed' state and the box dropped offline two days later when the
DHCP leases expired.

With inference_mode: typical 2.0 GB, 12k 2.4 GB, 24k 3.4 GB, 192k 5.5 GB, settling back
to ~2 GB after each request (150-request soak flat). Contexts over
HIGHLIGHT_MAX_CONTEXT_CHARS (default 200000) are refused with 413, and each request's
size is logged so the next incident names its input.
@TC407-api
TC407-api merged commit 516a98a into master Oct 8, 2026
2 of 4 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.

1 participant