Skip to content

Conversation

@sagudev
Copy link
Collaborator

@sagudev sagudev commented Jan 3, 2026

Description
Spec also allows obtaining adapter_info from device: https://www.w3.org/TR/webgpu/#dom-gpudevice-adapterinfo, so I implemented this. Ironically web-sys only allows obtaining adapter_info from device. We could implemented this without any changes to core (by caching adapter_info from adapter), but I decided against this for consistency (we now have device_adapter_info on global).

Testing
None.

Squash or Rebase?
Squash

Checklist

  • Run cargo fmt.
  • Run taplo format.
  • Run cargo clippy --tests. If applicable, add:
    • --target wasm32-unknown-unknown
  • Run cargo xtask test to run tests.
  • If this contains user-facing changes, add a CHANGELOG.md entry.

Signed-off-by: sagudev <[email protected]>
Copy link
Member

@ErichDonGubler ErichDonGubler left a comment

Choose a reason for hiding this comment

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

LGTM, minus a nit.

@ErichDonGubler ErichDonGubler self-assigned this Jan 5, 2026
@inner-daemons inner-daemons self-requested a review January 7, 2026 16:27
@inner-daemons inner-daemons self-assigned this Jan 7, 2026
@inner-daemons inner-daemons removed their request for review January 8, 2026 08:14
@inner-daemons inner-daemons removed their assignment Jan 8, 2026
Copy link
Member

@ErichDonGubler ErichDonGubler left a comment

Choose a reason for hiding this comment

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

Approving to signal that we should merge this, once the TODO is replaced. We should still take this, if it takes longer than a day or so for @sagudev to get to this (because being busy is a thing!).

Signed-off-by: sagudev <[email protected]>
@ErichDonGubler ErichDonGubler merged commit 1166617 into gfx-rs:trunk Jan 9, 2026
52 of 53 checks passed
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