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

conj() not correctly implemented for sparse complex matrices #8976

Closed
ntezak opened this issue Nov 11, 2014 · 2 comments
Closed

conj() not correctly implemented for sparse complex matrices #8976

ntezak opened this issue Nov 11, 2014 · 2 comments

Comments

@ntezak
Copy link

ntezak commented Nov 11, 2014

Element-wise complex conjugation is not supported for sparse matrices.
The following code should result in a single element sparse matrix with value 0im

conj(sparse([1im])) - sparse(conj([1im])) 
# should be sparse([0im]), but returns sparse([2im])

I think this used to be correctly defined but must have been broken recently.
I've added a pull request (issue #8977 )

ntezak pushed a commit to ntezak/julia that referenced this issue Nov 11, 2014
@ViralBShah
Copy link
Member

This is the problem:

julia> @which conj(sparse([1im]))
conj(x) at operators.jl:141

@ntezak
Copy link
Author

ntezak commented Nov 11, 2014

thanks!

waTeim pushed a commit to waTeim/julia that referenced this issue Nov 23, 2014
ViralBShah added a commit that referenced this issue Jan 4, 2015
partial cherry pick, just the added tests, from commit 67a1f68
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants