warp-rna icon indicating copy to clipboard operation
warp-rna copied to clipboard

CUDA_ERROR_ILLEGAL_ADDRESS, CPU support, TensorFlow bindings, some other changes

Open albertz opened this issue 2 years ago • 2 comments

Hey!

Thank you for this implementation!

I just wanted to say that we use your implementation and have done some changes, extensions and fixes to it. Unfortunately I don't really have time currently to prepare PRs for the individual changes, at least right now, and we anyway probably stay within our fork (but then I also don't expect that we will really do much further changes now). However, I still wanted to mention that we have done these changes, which are partly probably also interesting to you.

Our fork: https://github.com/rwth-i6/warp-rna

For example, we got a CUDA_ERROR_ILLEGAL_ADDRESS error, due to illegal memory access, reported here: https://github.com/rwth-i6/returnn/issues/1124. I think this was caused when you have cases where S > T. We fixed that now.

Further, we also implemented CPU support, in a somewhat hacky way, by wrapping up the CUDA code with some clever makros, which make it runnable on CPU without CUDA. See here to get an idea how this works.

And we also added TensorFlow bindings.

albertz avatar Sep 16 '22 10:09 albertz

Hey, Albert! I'm curious why you chose this implementation? Because it is a rare implementation of RNA or because of CUDA warps threads? In any case, I'm glad to see you use this project. I built this mostly as a toy example and do not use it anymore.

1ytic avatar Sep 17 '22 09:09 1ytic

Do you know a better RNA implementation? I don't really remember anymore why we took this one.

albertz avatar Sep 17 '22 11:09 albertz