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

Move conv specific ops from tensor_utils to conv2d. #16373

Merged
merged 3 commits into from
Jan 4, 2025

Conversation

nkpatel-tt
Copy link
Contributor

@nkpatel-tt nkpatel-tt commented Dec 31, 2024

Ticket

#15892

Problem description

Convolution related operations are part of general tensor_utils.

What's changed

Move conv specific code to conv2d folder.

Checklist

  • Post commit CI passes Link
  • Nightly fast dispatch Link1
  • Model regression CI testing passes Link
  • Device performance regression CI testing passes (if applicable) Link
  • (For models and ops writers) Full new models tests Link same as main

Copy link
Member

@ayerofieiev-tt ayerofieiev-tt left a comment

Choose a reason for hiding this comment

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

@nkpatel-tt , thank you!

@nkpatel-tt nkpatel-tt force-pushed the nkpatel/mv_conv_tensor_ops_to_conv2d branch 2 times, most recently from 18d31af to 6a02bae Compare January 2, 2025 05:49
Signed-off-by: Nilaykumar Patel <nkpatel@tenstorrent.com>
Signed-off-by: Nilaykumar Patel <nkpatel@tenstorrent.com>
Signed-off-by: Nilaykumar Patel <nkpatel@tenstorrent.com>
@nkpatel-tt nkpatel-tt force-pushed the nkpatel/mv_conv_tensor_ops_to_conv2d branch from ef9dd9f to bb22020 Compare January 4, 2025 04:52
@nkpatel-tt nkpatel-tt merged commit aaf2d73 into main Jan 4, 2025
243 of 244 checks passed
@nkpatel-tt nkpatel-tt deleted the nkpatel/mv_conv_tensor_ops_to_conv2d branch January 4, 2025 14:36
yugi957 pushed a commit that referenced this pull request Jan 9, 2025

Signed-off-by: Nilaykumar Patel <nkpatel@tenstorrent.com>
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.

3 participants