Skip to content

Add roundEven(), isNaN(), isInfinite() to Operators - #936

Merged
fdwr merged 1 commit into
webmachinelearning:mainfrom
shubhamg13:add_ops
Aug 13, 2026
Merged

Add roundEven(), isNaN(), isInfinite() to Operators#936
fdwr merged 1 commit into
webmachinelearning:mainfrom
shubhamg13:add_ops

Conversation

@shubhamg13

@shubhamg13 shubhamg13 commented Jun 5, 2026

Copy link
Copy Markdown
Contributor

Addition of some operators to List.

Reference: #859

Non-Substantive as no new implementations


Preview | Diff

@w3cbot

w3cbot commented Jun 5, 2026

Copy link
Copy Markdown
Collaborator

shubhamg13 marked as non substantive for IPR from ash-nazg.

@shubhamg13

Copy link
Copy Markdown
Contributor Author

cc: @xiaochengh

@shubhamg13 shubhamg13 changed the title Add roundEven(), isNan(), isFinite() to Operators Add roundEven(), isNan(), isInfinite() to Operators Jun 5, 2026

@huningxin huningxin left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

The method names isNan() and isFinite() in the commit title should align with the method names in the code: isNaN() and isInfinite().

@shubhamg13 shubhamg13 changed the title Add roundEven(), isNan(), isInfinite() to Operators Add roundEven(), isNaN(), isInfinite() to Operators Aug 12, 2026
Signed-off-by: Shubham Gupta <shubham.gupta@chromium.org>
@shubhamg13

Copy link
Copy Markdown
Contributor Author

The method names isNan() and isFinite() in the commit title should align with the method names in the code: isNaN() and isInfinite().

Done as suggested. Please take a look.

@huningxin huningxin left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM, thanks!

@shubhamg13

Copy link
Copy Markdown
Contributor Author

Please help me to merge this PR

@huningxin

Copy link
Copy Markdown
Contributor

As WG process, it requires to approvals before mering, @fdwr @anssiko , could you please take a look?

@fdwr fdwr left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

馃憤 Thanks for filling the gaps.

@fdwr
fdwr merged commit 36224cf into webmachinelearning:main Aug 13, 2026
2 checks passed
github-actions Bot added a commit that referenced this pull request Aug 13, 2026
SHA: 36224cf
Reason: push, by fdwr

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
@shubhamg13
shubhamg13 deleted the add_ops branch August 13, 2026 03:34
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.

4 participants