Skip to content

Switch from TensorDescriptor to tl.make_tensor_descriptor #232

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
Merged
merged 1 commit into from
Jul 1, 2025
Merged

Switch from TensorDescriptor to tl.make_tensor_descriptor #232

merged 1 commit into from
Jul 1, 2025

Conversation

jansel
Copy link
Contributor

@jansel jansel commented Jul 1, 2025 •
edited
Loading

@jansel
Switch from TensorDescriptor to tl.make_tensor_descriptor
fed3cf5 
This rewrites a lot of the tensor descriptor handling to properly apply
requirements.

stack-info: PR: #232, branch: jansel/stack/76
jansel added a commit that referenced this pull request Jul 1, 2025
@jansel
Switch from TensorDescriptor to tl.make_tensor_descriptor
fa1615b 
This rewrites a lot of the tensor descriptor handling to properly apply
requirements.

stack-info: PR: #232, branch: jansel/stack/76
@jansel jansel force-pushed the jansel/stack/76 branch from 0b0adb8 to fa1615b Compare July 1, 2025 15:08
This was referenced Jul 1, 2025
Merged
Merged
Merged
@facebook-github-bot facebook-github-bot added the CLA Signed This label is managed by the Meta Open Source bot. label Jul 1, 2025
This was referenced Jul 1, 2025
Merged
Merged
@jansel jansel requested a review from yf225 July 1, 2025 15:10
@jansel jansel force-pushed the jansel/stack/74 branch from be9721c to 70e1d89 Compare July 1, 2025 15:10
jansel added a commit that referenced this pull request Jul 1, 2025
@jansel
Switch from TensorDescriptor to tl.make_tensor_descriptor
58cd6f1 
This rewrites a lot of the tensor descriptor handling to properly apply
requirements.

stack-info: PR: #232, branch: jansel/stack/76
@jansel jansel force-pushed the jansel/stack/76 branch from fa1615b to 58cd6f1 Compare July 1, 2025 15:10
@jansel jansel force-pushed the jansel/stack/74 branch from 70e1d89 to aca25e0 Compare July 1, 2025 15:12
jansel added a commit that referenced this pull request Jul 1, 2025
@jansel
Switch from TensorDescriptor to tl.make_tensor_descriptor
d9209b2 
This rewrites a lot of the tensor descriptor handling to properly apply
requirements.

stack-info: PR: #232, branch: jansel/stack/76
@jansel jansel force-pushed the jansel/stack/76 branch from 58cd6f1 to d9209b2 Compare July 1, 2025 15:12
@jansel jansel force-pushed the jansel/stack/74 branch from aca25e0 to 0365bf0 Compare July 1, 2025 15:13
jansel added a commit that referenced this pull request Jul 1, 2025
@jansel
Switch from TensorDescriptor to tl.make_tensor_descriptor
75a48f5 
This rewrites a lot of the tensor descriptor handling to properly apply
requirements.

stack-info: PR: #232, branch: jansel/stack/76
@jansel jansel force-pushed the jansel/stack/76 branch from d9209b2 to 75a48f5 Compare July 1, 2025 15:13
@jansel jansel force-pushed the jansel/stack/74 branch from 0365bf0 to bf14a24 Compare July 1, 2025 15:14
jansel added a commit that referenced this pull request Jul 1, 2025
@jansel
Switch from TensorDescriptor to tl.make_tensor_descriptor
b99d7c2 
This rewrites a lot of the tensor descriptor handling to properly apply
requirements.

stack-info: PR: #232, branch: jansel/stack/76
@jansel jansel force-pushed the jansel/stack/76 branch from 75a48f5 to b99d7c2 Compare July 1, 2025 15:14
@jansel jansel force-pushed the jansel/stack/76 branch from b99d7c2 to fed3cf5 Compare July 1, 2025 22:46
@jansel jansel changed the base branch from jansel/stack/74 to main July 1, 2025 22:46
@jansel jansel merged commit 0996865 into main Jul 1, 2025
5 of 9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@yf225 yf225 yf225 approved these changes

Assignees
No one assigned
Labels
CLA Signed This label is managed by the Meta Open Source bot.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@jansel @yf225 @facebook-github-bot