Patch for /cusp/detail/device/multiply.h

vkirangoud / cusp-library

Automatically exported from code.google.com/p/cusp-library

Apache License 2.0

0 stars 0 forks source link

Patch for /cusp/detail/device/multiply.h #100

Open GoogleCodeExporter opened 9 years ago

GoogleCodeExporter commented 9 years ago

Implementation of Dense Matrix-Vector Multiply using cublas

Original issue reported on code.google.com by alfredo....@gmail.com on 11 Oct 2012 at 7:43

Attachments:

multiply.h.patch

GoogleCodeExporter commented 9 years ago

Thanks for the patch. But I think you need to check wether the matrix is in 
column or row major format and act appropriately. Also some tests 
(testing/multiply.cu) would be good.

Original comment by filipe.c...@gmail.com on 11 Oct 2012 at 8:20

GoogleCodeExporter commented 9 years ago

Yes you are right, I forgot that cublas assumes column-major. How can I check 
if the matrix is in column or row major format?
I was thinking on something like:
if ( typeid(matrix.orientation) == typeid(cusp::column_major) )

but I'm quite noob of cusp

Original comment by alfredo....@gmail.com on 11 Oct 2012 at 9:59