FIX:default to WebGL over WebGPU on Linux - #934
Conversation
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Advanced Run ID: 📒 Files selected for processing (3)
Included review availability: Your plan provides up to 10 included reviews per hour; 9 remain after this review. 📝 WalkthroughWalkthroughThe exporter adds platform-aware default backend ordering. Linux selects WebGL before WebGPU. Other platforms select WebGPU before WebGL. The renderer applies this order when no explicit backend preference exists. ChangesLightning backend order
Priority: ➖ Normal Estimated code review effort: 2 (Simple) | ~8 minutes Change: Bug fix Suggested reviewers: Merge Risk: ⚪ Minimal · up to The change is narrowly scoped, preserves explicit preferences and non-Linux behavior, and is ready to merge after normal checks. 🚥 Pre-merge checks | ✅ 4 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (4 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Description
Changes the default render backend order for Lightning exports on Linux so that WebGL is tried before WebGPU, instead of the other way around.
Motivation
On Linux, the WebGPU backend (Dawn/Vulkan) can corrupt GPU resource imports for VideoFrame-backed textures partway through an export. This crashes PixiJS's WebGPU bind group system well after initialization has already succeeded, so the failure isn't caught by existing init-time checks. Defaulting Linux to WebGL avoids this instability until it's resolved upstream, while other platforms keep preferring WebGPU.
Type of Change
Related Issue(s)
N/A
Screenshots / Video
N/A — this is an internal backend-selection change with no UI impact.
Testing Guide
npm test -- backendPolicy— coversgetDefaultLightningRenderBackendOrder, which returns["webgl", "webgpu"]on Linux and["webgpu", "webgl"]on other platforms.preferredRenderBackendoverride) and completes without the mid-export GPU resource crash.Checklist
Summary by CodeRabbit