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

Support non-trivial strides when importing DLPack tensors #8082

Open
hawkinsp opened this issue Oct 4, 2021 · 3 comments
Open

Support non-trivial strides when importing DLPack tensors #8082

hawkinsp opened this issue Oct 4, 2021 · 3 comments
Assignees
@hawkinsp
Labels
enhancement New feature or request

Comments

@hawkinsp
Copy link
Member

hawkinsp commented Oct 4, 2021

No description provided.

@hawkinsp hawkinsp added the enhancement New feature or request label Oct 4, 2021
@hawkinsp hawkinsp self-assigned this Oct 4, 2021
@jakevdp
Copy link
Collaborator

jakevdp commented Oct 4, 2021

Related bug: #7657 The bug was closed due to a suitable workaround being proposed, but I think the underlying issue is still there.

@lucidrains
Copy link

i'm interested in this!

@mattjj
Copy link
Member

mattjj commented Oct 26, 2021

Just so we have a test case: running the code in this gist without the call to .contiguous() results in an error.

@tomatillos tomatillos mentioned this issue Sep 26, 2023
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@hawkinsp hawkinsp

Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
4 participants
@lucidrains @hawkinsp @jakevdp @mattjj