Skip to content

feat(WebGPU): add vtkWebGPUConfiguration - #3649

Open
daker wants to merge 1 commit into
Kitware:masterfrom
daker:webgpu-shared-gpudevice
Open

daker wants to merge 1 commit into
Kitware:masterfrom
daker:webgpu-shared-gpudevice

Conversation

@daker

@daker daker commented Sep 11, 2026

Copy link
Copy Markdown
Collaborator

Context

Fixes #3648

Results

Changes

PR Checklist

  • GitHub Actions CI passed: semantic-release commit messages, lint, and tests
  • Test coverage added
  • Documentation and TypeScript definitions are updated to match these changes

@daker
daker force-pushed the webgpu-shared-gpudevice branch from 3f9670e to 9046596 Compare September 13, 2026 01:15
@daker daker changed the title feat(WebGPU): support shared GPUDevice across render windows feat(WebGPU): add vtkWebGPUConfiguration Sep 13, 2026
@daker
daker force-pushed the webgpu-shared-gpudevice branch 2 times, most recently from 163233b to 22791dc Compare September 13, 2026 01:26
@daker

daker commented Sep 13, 2026

Copy link
Copy Markdown
Collaborator Author

@PaulHax

@daker
daker force-pushed the webgpu-shared-gpudevice branch from 22791dc to f8d1203 Compare September 13, 2026 01:30
Comment thread Sources/Rendering/WebGPU/Configuration/index.d.ts Outdated
Comment thread Sources/Rendering/WebGPU/Configuration/index.d.ts Outdated

@PaulHax PaulHax left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for pushing on this. I don't have a project using WebGPU (at the moment) so I'm not really in a good position to give good feedback on this important foundational work.

@daker
daker force-pushed the webgpu-shared-gpudevice branch from f8d1203 to b02066f Compare September 13, 2026 20:38
@daker

daker commented Sep 18, 2026

Copy link
Copy Markdown
Collaborator Author

@finetjul can i get a review on this one when you have time ?

@finetjul

Copy link
Copy Markdown
Member

@finetjul can i get a review on this one when you have time ?

That's more a topic for @sankhesh

Comment thread Sources/Rendering/WebGPU/Configuration/index.js
Comment thread Sources/Rendering/WebGPU/Configuration/index.js
Comment thread Sources/Rendering/WebGPU/Configuration/index.js Outdated
Comment thread Sources/Rendering/WebGPU/Configuration/index.js
Comment thread Sources/types.d.ts
@sankhesh

Copy link
Copy Markdown
Collaborator

Several windows now share one vtkWebGPUDevice (pipelines, shader and buffer/texture caches). No cache is released or reference-counted when one window releases. Please confirm that a window's releaseGraphicsResources() can't destroy cached GPU objects another window still uses. It would be good to add a two-window test that includes device loss.

@daker
daker force-pushed the webgpu-shared-gpudevice branch from b02066f to db5d889 Compare September 19, 2026 22:49
@daker
daker requested a review from sankhesh September 19, 2026 22:50
@daker

daker commented Sep 22, 2026

Copy link
Copy Markdown
Collaborator Author

@sankhesh i have addressed your concerns

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.

[Feature] Support shared GPUDevice instances across WebGPURenderWindows

4 participants