Skip to content

Fix Windows CI: add Mesa software OpenGL for headless napari tests - #181

Merged
nfahlgren merged 1 commit into
mainfrom
copilot/fix-github-actions-build-failure
Aug 13, 2026
Merged

Fix Windows CI: add Mesa software OpenGL for headless napari tests#181
nfahlgren merged 1 commit into
mainfrom
copilot/fix-github-actions-build-failure

Conversation

Copilot AI commented Aug 13, 2026

Copy link
Copy Markdown
Contributor

The build (3.13, windows-latest) deploy job fails because napari/vispy calls glGetIntegerv(GL_MAX_TEXTURE_SIZE) during Viewer initialization, but GitHub Actions Windows runners have no GPU, so no OpenGL context exists and PyOpenGL raises GL_INVALID_OPERATION (error 1282). The prior NAPARI_TESTING=1 fix was insufficient — it sets napari's test mode but doesn't provide an OpenGL implementation.

Changes

  • .github/workflows/deploy.yml: Add ssciwr/setup-mesa-opengl@v1 as a Windows-only step before tests run. This installs Mesa3D's CPU-based opengl32.dll on the runner, giving napari/vispy a valid software OpenGL context to initialize against.
- name: Setup Mesa OpenGL for headless rendering (Windows only)
  if: runner.os == 'Windows'
  uses: ssciwr/setup-mesa-opengl@v1

@deepsource-io

deepsource-io Bot commented Aug 13, 2026

Copy link
Copy Markdown

DeepSource Code Review

We reviewed changes in ed5d10b...fd650e2 on this pull request. Below is the summary for the review, and you can see the individual issues we found as inline review comments.

See full review on DeepSource ↗

PR Report Card

Overall Grade   Security  

Reliability  

Complexity  

Hygiene  

Coverage  

Code Review Summary

Analyzer Status Updated (UTC) Details
Python Aug 13, 2026 1:15a.m. Review ↗
Code coverage Aug 13, 2026 1:22a.m. Review ↗

Code Coverage Summary

Language Line Coverage (Overall)
Aggregate
100%
Python
100%

➟ Additional coverage metrics may have been reported. See full coverage report ↗


Important

AI Review is run only on demand for your team. We're only showing results of static analysis review right now. To trigger AI Review, comment @deepsourcebot review on this thread.

@nfahlgren
nfahlgren merged commit 9b4c2cc into main Aug 13, 2026
5 checks passed
@nfahlgren
nfahlgren deleted the copilot/fix-github-actions-build-failure branch August 13, 2026 01:23
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.

2 participants