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

coo2csr not working correctly for matrices where nnz < num_rows #153

Open
jpola opened this issue Sep 16, 2015 · 2 comments
Open

coo2csr not working correctly for matrices where nnz < num_rows #153

jpola opened this issue Sep 16, 2015 · 2 comments
Labels

Comments

@jpola
Copy link
Contributor

jpola commented Sep 16, 2015

I found a bug related to my indices_to_offsets function. If number of nonzeros < number of rows the transformation function gives wrong row_offsets. row_offsets in this situation is larger than row_indices in corresponding coo format.

I missed this due to fact that for large matrices this situation is not common, but for small matrices I found that very often.

@jpola jpola added the bug label Sep 16, 2015
@jpola
Copy link
Contributor Author

jpola commented Sep 29, 2015

I have solution for that. Fix require scatter operation which I don't want to write since we will be using boost compute in the near future? Isn't it?

@kknox
Copy link
Contributor

kknox commented Sep 29, 2015

Yes, I've not heard significant resistance to using boost.compute; it makes a lot of sense for clsparse.

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

No branches or pull requests

2 participants