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

Editorial: Don't reference IDL boolean for values in tensors #735

Merged
merged 1 commit into from
Jul 23, 2024

Conversation

inexorabletash
Copy link
Member

@inexorabletash inexorabletash commented Jul 22, 2024

The spec should only reference IDL types in prose when dealing with binding, e.g. in unions, and maybe internal slots. When discussing values in tensors, just be explicit and don't try to link to a definition that doesn't really apply.


Preview | Diff

The spec should only reference IDL types in prose when dealing with
binding, e.g. in unions, and maybe internal slots. When discussing
values in tensors, just be explicit and don't try to link to a
definition that doesn't really apply.
@inexorabletash
Copy link
Member Author

Another simple one - @fdwr can you review/merge?

Copy link
Collaborator

@fdwr fdwr left a comment

Choose a reason for hiding this comment

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

👍

@fdwr
Copy link
Collaborator

fdwr commented Jul 23, 2024

@huningxin - this one is simple enough that I'll just merge it now. Please comment afterward if anything was dubious.

@fdwr fdwr merged commit 4b90605 into webmachinelearning:main Jul 23, 2024
2 checks passed
github-actions bot added a commit that referenced this pull request Jul 23, 2024
SHA: 4b90605
Reason: push, by fdwr

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Copy link
Contributor

@huningxin huningxin left a comment

Choose a reason for hiding this comment

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

LGTM!

@inexorabletash inexorabletash deleted the no-idl-bool branch July 23, 2024 01:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants