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

Why not add contiguous check for tensor in cuda kernel? #211

Open
otakuxiang opened this issue Jul 7, 2021 · 0 comments
Open

Why not add contiguous check for tensor in cuda kernel? #211

otakuxiang opened this issue Jul 7, 2021 · 0 comments

Comments

@otakuxiang
Copy link

I just implement a InstanceNorm layer in SparseConv and it using permute and unsqueeze function.This will change the stride and make tensor no more contiguous. The convolution after the norm function will give a wrong answer because the stride changes. I see the source code and I find it always assume the input tensor is contiguous because it use stride(0) multiply row id to get the value.
So why not add a contiguous check to make it more robustly?

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

No branches or pull requests

1 participant