Skip to content

Fix Texture total allocation size is too large - #64

Merged
benjaminleonard merged 1 commit into
mainfrom
max-sample-size
Jun 26, 2026
Merged

benjaminleonard merged 1 commit into
mainfrom
max-sample-size

Conversation

@benjaminleonard

Copy link
Copy Markdown
Collaborator

Fixes #62

The post-processing step uses anti-aliasing sized: width × height × 8 samples × bytes-per-pixel

On a large display, that buffer was too big for the GPU to allocate in one chunk.

The fix:

  1. Don't ask for more anti-aliasing than the GPU can actually give. Before rendering, we now do a test allocation: try 4×, then 2×, then none — and use the first one that succeeds
  2. Cap the default at 4× instead of 8×. Lowering it shrinks the buffer and is visually very similar

@vercel

vercel Bot commented Jun 26, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
rack-explorer Ready Ready Preview Jun 26, 2026 1:46pm

Request Review

@benjaminleonard
benjaminleonard merged commit dcf8546 into main Jun 26, 2026
2 of 3 checks passed

This branch was successfully deployed

1 active deployment
Preview — 45804fce Deployed Jun 26, 2026 by vercel[bot]
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.

Only Outline Displaying

1 participant