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

Extends ctranspose() to Tridiagonal matrices #2608

Closed
wants to merge 3 commits into from
Closed

Extends ctranspose() to Tridiagonal matrices #2608

wants to merge 3 commits into from

Conversation

jiahao
Copy link
Member

@jiahao jiahao commented Mar 19, 2013

This patch allows Tridiagonal matrices to be transposed.

@ViralBShah
Copy link
Member

What you have should be transpose(), and ctranspose should have a call to conj().

@jiahao
Copy link
Member Author

jiahao commented Mar 19, 2013

So if I change ctranspose() to transpose() and add a new ctranspose() which is conj(transpose()), this should be OK?

@ViralBShah
Copy link
Member

Is there a conj function defined on SymTridiagonal? It doesn't seem to be, and you may need to just do conj of the individual vectors instead.

Could you also add tests in test/linalg.jl?

Extends conj(), transpose() and ctranspose() to Tridiagonal matrices

Also adds very elementary tests of these functions
@jiahao
Copy link
Member Author

jiahao commented Mar 19, 2013

I added conj() as well... why not. Hopefully this time it all makes sense. (I haven't done anything for SymTridiagonal here, only Tridiagonal)

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

Successfully merging this pull request may close these issues.

2 participants