-
Notifications
You must be signed in to change notification settings - Fork 48
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
Add device selection explainer #784
Add device selection explainer #784
Conversation
Signed-off-by: Zoltan Kis <[email protected]>
Signed-off-by: Zoltan Kis <[email protected]>
@zolkis thanks for this explainer! There's no good way to redirect existing explainers, so for consistency you can keep this file in the root. Perhaps explainer-device-selection.md -> device-selection-explainer.md. |
…ction explainer Signed-off-by: Zoltan Kis <[email protected]>
Signed-off-by: Zoltan Kis <[email protected]>
Signed-off-by: Zoltan Kis <[email protected]>
c401b84
to
f45266f
Compare
@mwyrzykowski could you please check if this is acceptable for merging, and whether the Minimum Viable Solution in the end is a good first step for making a PR (minus including the "low-latency" option which is a possible future discussion, but not now). |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you @zolkis for the writeup! Looks good!
…rnatives section Signed-off-by: Zoltan Kis <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
…evices Signed-off-by: Zoltan Kis <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍 Thanks for collecting the backstory and previous context links. If it was easy, we would have figured it out long ago, implying it's not easy 😉.
Corrections from Dwayne Co-authored-by: Dwayne Robinson <[email protected]>
Signed-off-by: Zoltan Kis <[email protected]>
Thanks @fdwr for the suggestions and comments, I have updated the text. Maybe it needs a proofreading before squash-merging. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@zolkis: 👍 LGTM.
Waiting on any other input? Otherwise we'll merge it.
We agreed on the last call that it could be merged - so go ahead, thanks! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I did a proofreading pass with some suggestions. Also added a "Participate" header.
As discussed on our call https://www.w3.org/2025/01/16-webmachinelearning-minutes.html#f000 this explainer is now ready to be merged.
Thank you @zolkis for putting this explainer together and everyone for your review and suggestions!
As usual, this doc will be updated whenever the group learns more about this topic.
Thank you Anssi! |
An initial draft to summarize the discussion.
Feel free to change it in any way, add commits here etc.
The main forums to discuss this are #749 and #759.
After we agree on a solution, this explainer can be archived.
@anssiko should we create an
explainers
folder?