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

Branch yang #9

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Branch yang #9

wants to merge 3 commits into from

Conversation

jinglunYang
Copy link

I have pulled request again, please to check it. And i have checked the emd.cuh is in pkg/include/cuda/

JinglunYang added 3 commits May 29, 2019 17:12
…ired bugs about CUDA kernel failed when points sets(M,N>~2000).
…d bugs about CUDA kernel failed when points sets(M,N>~2000).
@louis-cl
Copy link

louis-cl commented Jun 5, 2019

The code outputs a value (hopefully correct ?) for big tensors but the gradient fails, even for small inputs.
In that case my error is:
THCudaCheck FAIL file=.../THCReduceAll.cuh line=327 error=4 : unspecified launch failure

@jinglunYang
Copy link
Author

Maybe you should adjust the version of your CUDA & Cudnn & Pytorch. They should be satisfied with a certainly relationship. Can you tell me about your environment?

@louis-cl
Copy link

I am using Python 3.6.8, Pytorch 1.1.0, CUDA 9.0.176 and Cudnn 7501.

@meder411
Copy link
Owner

Sorry I've been backed up with work. I will do my best to get to this ASAP.

@meder411
Copy link
Owner

This doesn't repair the bug necessarily. It just fixes the number of dimensions to be 3, as opposed to some arbitrary dimension D.

@rjtshrm
Copy link

rjtshrm commented Aug 5, 2019

Hi @jinglunYang , I picked you commit with changes. I can run test_emd_loss.py file. But while using the loss in my training set it gives me error RuntimeError: cuDNN error: CUDNN_STATUS_NOT_INITIALIZED. I already tested my training data with Chamfer Loss and it was working fine.

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.

4 participants