Skip to content

Comments

fix: update WebGPU embedding benchmark to prefer GPUAdapter.info over requestAdapterInfo#1098

Closed
ErichDonGubler wants to merge 2 commits intohuggingface:mainfrom
erichdongubler-mozilla:patch-1
Closed

fix: update WebGPU embedding benchmark to prefer GPUAdapter.info over requestAdapterInfo#1098
ErichDonGubler wants to merge 2 commits intohuggingface:mainfrom
erichdongubler-mozilla:patch-1

Conversation

@ErichDonGubler
Copy link

Migrates to use the new spec.-compliant way of getting adapter info in WebGPU (see also microsoft/onnxruntime#21065).

This has been removed by both Chrome and Firefox, so there might be enough reason to simply stop using requestAdapterInfo, but I wasn't sure.

…er `requestAdapterInfo`

Migrates to use the new spec.-compliant way of getting adapter info in WebGPU (see also <microsoft/onnxruntime#21065>).

This has been removed by both Chrome and Firefox, so there might be enough reason to simply stop using `requestAdapterInfo`, but I wasn't sure.
@ErichDonGubler
Copy link
Author

This might not be ready to merge yet, if the main library doesn't yet expose the info member. I wasn't sure about that, either. 😅

@xenova
Copy link
Collaborator

xenova commented Dec 23, 2024

Thanks for the heads up! I'm busy migrating all our examples to http://github.com/huggingface/transformers.js-examples, and will update them to use adapter.info when necessary!

@xenova
Copy link
Collaborator

xenova commented Feb 17, 2026

(closed recently by #1382)

@xenova xenova closed this Feb 17, 2026
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