Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

The bind group in set_bind_group should be optional #4738

Open
nical opened this issue Nov 21, 2023 · 4 comments
Open

The bind group in set_bind_group should be optional #4738

nical opened this issue Nov 21, 2023 · 4 comments
Assignees
Labels
area: api Issues related to API surface

Comments

@nical
Copy link
Contributor

nical commented Nov 21, 2023

Spec: https://gpuweb.github.io/gpuweb/#gpubindingcommandsmixin-setbindgroup

The bind group is nullable which in rust parlance means we should take an Option<BindGroupId> and be able to "unset" a bindgroup at an index.

@teoxoy teoxoy added the area: api Issues related to API surface label Nov 21, 2023
@teoxoy teoxoy added this to the WebGPU Specification V1 milestone Nov 21, 2023
@jimblandy
Copy link
Member

It's unclear what the underlying platform APIs' expectations are around removing bindgroups, so we're going to need to investigate this a bit before we can tell how much work it will be.

@teoxoy
Copy link
Member

teoxoy commented Dec 15, 2023

Relevant spec PR: gpuweb/gpuweb#3938

@Wumpf
Copy link
Member

Wumpf commented Nov 19, 2024

@Wumpf Wumpf closed this as completed Nov 19, 2024
@github-project-automation github-project-automation bot moved this from In Progress to Done in WebGPU for Firefox Nov 19, 2024
@Wumpf
Copy link
Member

Wumpf commented Nov 19, 2024

ah I spoke too soon, HAL doesn't handle this yet it seems?

@Wumpf Wumpf reopened this Nov 19, 2024
@github-project-automation github-project-automation bot moved this from Done to In Progress in WebGPU for Firefox Nov 19, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area: api Issues related to API surface
Projects
Status: In Progress
Development

No branches or pull requests

5 participants