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.

Sign up for GitHub

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

Jump to bottom

#16147: Replace binary with binary_ng #17160

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from
Draft

#16147: Replace binary with binary_ng #17160

wants to merge 1 commit into from
+233 −361

Conversation

patrickroberts
Copy link
Contributor

@patrickroberts patrickroberts commented Jan 27, 2025 •
edited
Loading

Ticket

#16147

Problem description

binary_ng is intended to replace binary.

What's changed

Re-register binary composite ops with binary_ng device op.

Checklist

@patrickroberts patrickroberts force-pushed the proberts/16147-replace_binary_with_ng branch 4 times, most recently from 92c9b5e to f42134c Compare February 3, 2025 20:56
@patrickroberts patrickroberts mentioned this pull request Feb 4, 2025
Open
@patrickroberts patrickroberts force-pushed the proberts/16147-replace_binary_with_ng branch 2 times, most recently from 2e501d6 to 1f61716 Compare February 4, 2025 22:01
@patrickroberts patrickroberts force-pushed the proberts/16147-replace_binary_with_ng branch from 1f61716 to afd1250 Compare February 5, 2025 23:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@sjameelTT sjameelTT Awaiting requested review from sjameelTT sjameelTT will be requested when the pull request is marked ready for review sjameelTT is a code owner

@ntarafdar ntarafdar Awaiting requested review from ntarafdar ntarafdar will be requested when the pull request is marked ready for review ntarafdar is a code owner

@dchenTT dchenTT Awaiting requested review from dchenTT dchenTT will be requested when the pull request is marked ready for review dchenTT is a code owner

@ayerofieiev-tt ayerofieiev-tt Awaiting requested review from ayerofieiev-tt ayerofieiev-tt will be requested when the pull request is marked ready for review ayerofieiev-tt is a code owner

@dmakoviichuk-tt dmakoviichuk-tt Awaiting requested review from dmakoviichuk-tt dmakoviichuk-tt will be requested when the pull request is marked ready for review dmakoviichuk-tt is a code owner

@rfurko-tt rfurko-tt Awaiting requested review from rfurko-tt rfurko-tt will be requested when the pull request is marked ready for review rfurko-tt is a code owner

@cfjchu cfjchu Awaiting requested review from cfjchu cfjchu will be requested when the pull request is marked ready for review cfjchu is a code owner

@TT-BrianLiu TT-BrianLiu Awaiting requested review from TT-BrianLiu TT-BrianLiu will be requested when the pull request is marked ready for review TT-BrianLiu is a code owner

@razorback3 razorback3 Awaiting requested review from razorback3 razorback3 will be requested when the pull request is marked ready for review razorback3 is a code owner

@dongjin-na dongjin-na Awaiting requested review from dongjin-na dongjin-na will be requested when the pull request is marked ready for review dongjin-na is a code owner

@bbradelTT bbradelTT Awaiting requested review from bbradelTT bbradelTT will be requested when the pull request is marked ready for review bbradelTT is a code owner

At least 0 approving reviews are required to merge this pull request.

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@patrickroberts