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

Remove async launch_op from TT-NN #17217

Draft
wants to merge 6 commits into
base: main
Choose a base branch
from
Draft

Remove async launch_op from TT-NN #17217

wants to merge 6 commits into from

Conversation

ayerofieiev-tt
Copy link
Member

@ayerofieiev-tt ayerofieiev-tt commented Jan 28, 2025 •
edited
Loading

Ticket

#17292

Problem description

Exploring possibility of removal of async launch from TT-NN

What's changed

Removed everything related to async from decorators.
Removed launch_op
Together with this removed handling of multiple ops on a level of an OP.
Multiple devices only handled on a level of Device Operation dispatch right now.

Checklist

  • Post commit CI
  • Blackhole Post commit (if applicable)
  • Model regression CI testing passes (if applicable)
  • Device performance regression CI testing passes (if applicable)

@ayerofieiev-tt ayerofieiev-tt changed the title [WIP] Remove async Remove async launch_op from TT-NN Jan 28, 2025
@ayerofieiev-tt ayerofieiev-tt mentioned this pull request Jan 29, 2025
Open
@ayerofieiev-tt ayerofieiev-tt linked an issue Jan 29, 2025 that may be closed by this pull request
Explore removal of async op launch from TT-NN #17292
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@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

@tt-aho tt-aho Awaiting requested review from tt-aho tt-aho will be requested when the pull request is marked ready for review tt-aho 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

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

@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

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

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

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

@yugaoTT yugaoTT Awaiting requested review from yugaoTT yugaoTT will be requested when the pull request is marked ready for review yugaoTT 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

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

@patrickroberts patrickroberts Awaiting requested review from patrickroberts patrickroberts will be requested when the pull request is marked ready for review patrickroberts 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

@jvegaTT jvegaTT Awaiting requested review from jvegaTT jvegaTT will be requested when the pull request is marked ready for review jvegaTT 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

@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

@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

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.

Explore removal of async op launch from TT-NN
1 participant
@ayerofieiev-tt