-
Notifications
You must be signed in to change notification settings - Fork 89
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
Support for integers in sparse matrix types #153
Comments
Thanks ;-) Since vectors and dense matrices already support integers, I adjusted the title accordingly. |
@karlrupp the matrix objects can be created but the methods don't appear to exist. For example, if I try to multiply two integer matrices together I get the following error:
|
Thanks, @cdeterman , frankly I was not aware of the limitation for GEMM in OpenCL (i.e. I haven't written that code) |
Hi, Since I have written this code, I can give a bit more explanations :p The Philippe On Mon, Aug 3, 2015 at 1:24 PM, Karl Rupp notifications@github.com wrote:
|
@PhilippeTillet perhaps this could be a separate issue? It seems to me (perhaps a bit naively) that a safe default would be to have the accumulator default to |
Contrary to CUDA, the size of an int in OpenCL is well defined (32 bits). Additionally, it is very possible that GEMM on integer types would not On Tue, Aug 4, 2015 at 8:41 AM, Charles Determan notifications@github.com
|
@PhilippeTillet : Keep in mind that ViennaCL does not allow mixing numeric types in computations.
should have accumulator semantics other than 'int'. In fact, it would be quite surprising to see float or double as accumulator here. Moreover, non-optimal performance is better than no performance at all ;-) |
I read on the viennacl mailing list a mention of adding additional templates such as
int
so I am creating this issue to formally request it as well :-)The text was updated successfully, but these errors were encountered: