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

cublasCgemmBatched, cublasZgemmBatched wrappers #52

Merged
merged 2 commits into from Apr 17, 2014

Conversation

Projects
None yet
2 participants
@benanne
Contributor

benanne commented Apr 14, 2014

I noticed that the wrappers for cublasCgemmBatched and cublasZgemmBatched were missing (the float and double versions are wrapped already). I need cublasCgemmBatched, so I added them.

I didn't write any tests for this since the modifications are pretty trivial (I just copied from cublasSgemmBatched and cublasCgemm), but I can have a go at writing one if it is necessary.

I am currently using scikits.cuda to build an FFT-based convolution operator for Theano. I need to do a batch of dot products in the Fourier domain to implement an input domain convolution, that's why I need cublasCgemmBatched. It's here, in case anyone is interested: https://github.com/benanne/theano_fftconv

@lebedov lebedov merged commit e9f8468 into lebedov:master Apr 17, 2014

@lebedov

This comment has been minimized.

Show comment
Hide comment
@lebedov

lebedov Apr 17, 2014

Owner

Thanks for the new bindings. Unit tests for these functions would be great if you can add them.

Owner

lebedov commented Apr 17, 2014

Thanks for the new bindings. Unit tests for these functions would be great if you can add them.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment