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

Consistent name for CompositeToLayoutOp #1090

Closed
jnie-TT opened this issue Oct 28, 2024 · 0 comments · Fixed by #1105
Closed

Consistent name for CompositeToLayoutOp #1090

jnie-TT opened this issue Oct 28, 2024 · 0 comments · Fixed by #1105
Assignees

Comments

@jnie-TT
Copy link
Contributor

jnie-TT commented Oct 28, 2024

Followup issue from #840, we want to have a ToLayout op that captures all the parameters up front, effectively doing what the current CompositeToLayoutOp does (maybe set a couple parameters as optional since ttnn::to_layout miight not support them yet), and we would just use the same op for tilize/untilze. And eventually we can merge the decomposed ops into this op as ttnn supports them, and make subsequent arguments required instead of optional.

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 a pull request may close this issue.

1 participant