Skip to content

Add a buffer_view variant option to wireframe sample - #568

Merged
beaufortfrancois merged 5 commits into
webgpu:mainfrom
alan-baker:buffer-view-wireframe
Aug 5, 2026
Merged

Add a buffer_view variant option to wireframe sample#568
beaufortfrancois merged 5 commits into
webgpu:mainfrom
alan-baker:buffer-view-wireframe

Conversation

@alan-baker

Copy link
Copy Markdown
Contributor
  • Combines all model data into a single buffer
    • Beginning of the buffer has model metadata
    • vertex and index buffers for each draw set from offsets into the buffer
    • buffer_view shaders passed the whole buffer

* Combines all model data into a single buffer
  * Beginning of the buffer has model metadata
  * vertex and index buffers for each draw set from offsets into the
    buffer
  * buffer_view shaders passed the whole buffer
@alan-baker

Copy link
Copy Markdown
Contributor Author

@beaufortfrancois thoughts on this as the buffer_view example we discussed?

@beaufortfrancois beaufortfrancois 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 starting this!

Comment thread sample/wireframe/main.ts Outdated
Comment thread sample/wireframe/main.ts Outdated
Comment thread sample/wireframe/main.ts Outdated
Comment thread sample/wireframe/main.ts Outdated
Comment thread sample/wireframe/main.ts Outdated
Comment thread sample/wireframe/main.ts Outdated
Comment thread sample/wireframe/main.ts Outdated
Comment thread sample/wireframe/main.ts Outdated
@alan-baker
alan-baker marked this pull request as ready for review June 23, 2026 16:44
@beaufortfrancois
beaufortfrancois enabled auto-merge (squash) August 5, 2026 04:17
@beaufortfrancois
beaufortfrancois merged commit 4944c70 into webgpu:main Aug 5, 2026
1 check passed
@alan-baker
alan-baker deleted the buffer-view-wireframe branch August 5, 2026 15:14
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.

3 participants