Skip to content

Elementary operations for Tridiagonal and SymTridiagonal matrices#2611

Merged
ViralBShah merged 4 commits into
JuliaLang:masterfrom
jiahao:tridiag-simpleops
Mar 19, 2013
Merged

Elementary operations for Tridiagonal and SymTridiagonal matrices#2611
ViralBShah merged 4 commits into
JuliaLang:masterfrom
jiahao:tridiag-simpleops

Conversation

@jiahao

@jiahao jiahao commented Mar 19, 2013

Copy link
Copy Markdown
Member

Fleshed out elementary +,-,x,\ operations on Tridiagonal and SymTridiagonal matrices as well as transpose(), conj() and ctranspose()

Where necessary (esp. in matrix multiply) the output type changes so that the output is still preserved gracefully. In particular where needed:

  • SymTridiagonal is converted to Tridiagonal or dense
  • Tridiagonal is converted to dense

Fixed apparent bug in copy(Symdiagonal)

Also some light cleanup of tridiag.jl

Replaces pull request #2608.

jiahao added 4 commits March 19, 2013 01:56
Extends conj(), transpose() and ctranspose() to Tridiagonal matrices

Also adds very elementary tests of these functions
…agonal matrices

Where necessary (esp. in matrix multiply) the output type changes so that the output is still preserved gracefully. In particular where needed:
- SymTridiagonal is converted to Tridiagonal or dense
- Tridiagonal is converted to dense

Fixed apparent bug in copy(Symdiagonal)

Also some light cleanup of tridiag.jl
ViralBShah added a commit that referenced this pull request Mar 19, 2013
Elementary operations for Tridiagonal and SymTridiagonal matrices
@ViralBShah
ViralBShah merged commit 8b29497 into JuliaLang:master Mar 19, 2013
@ViralBShah

Copy link
Copy Markdown
Member

Did you do a make testall to check that all tests pass? All the tests fail for me. I have currently disabled the tests.

@jiahao
jiahao deleted the tridiag-simpleops branch March 22, 2013 22:44
@jiahao

jiahao commented Mar 22, 2013

Copy link
Copy Markdown
Member Author

Ah, Tridiagonal doesn't have == defined. I'll put this in for Tridiagonal and SymTridiagonal.

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