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

ell_to_csr conversion rounds ValueType to IndexType #3

Closed
GoogleCodeExporter opened this issue Jun 28, 2015 · 2 comments
Closed

ell_to_csr conversion rounds ValueType to IndexType #3

GoogleCodeExporter opened this issue Jun 28, 2015 · 2 comments

Comments

@GoogleCodeExporter
Copy link

Here's a quick patch:

Index: cusp/detail/host/conversion.h
===================================================================
--- cusp/detail/host/conversion.h   (revision 76)
+++ cusp/detail/host/conversion.h   (working copy)
@@ -328,7 +328,7 @@
         for(IndexType n = 0; n < num_entries_per_row; n++)
         {
             const IndexType j = src.column_indices(i,n);
-            const IndexType v = src.values(i,n);
+            const ValueType v = src.values(i,n);
             if(j != invalid_index)
             {
                 dst.column_indices[num_entries] = j;

Original issue reported on code.google.com by egastal on 15 Nov 2009 at 4:50

@GoogleCodeExporter
Copy link
Author

This issue was closed by revision r77.

Original comment by wnbell on 15 Nov 2009 at 5:32

  • Changed state: Fixed

@GoogleCodeExporter
Copy link
Author

The format conversion unit tests will now catch this sort of error too.

Original comment by wnbell on 15 Nov 2009 at 5:40

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

No branches or pull requests

1 participant