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

WIP: Cleanup of Tridiagonal code #5749

Closed
wants to merge 1 commit into from
Closed

WIP: Cleanup of Tridiagonal code #5749

wants to merge 1 commit into from

Conversation

jiahao
Copy link
Member

@jiahao jiahao commented Feb 9, 2014

The eventual goal of this PR is to implement the missing functionality referenced by #3688, #5429.

Currently, this just strips out the temporary working variables as fields from within Tridiagonal.

@andreasnoack
Copy link
Member

I'm fine with this.

I realise that you had a broader perspective with this pr that might overlap with mine (#5759) from last night. Just to indicate what's in the pipe line, I also have a function for real symmetric tridiagonalization in the Quaternions package, but right now it only supports storage in the lower triangle.

@IainNZ
Copy link
Member

IainNZ commented Aug 11, 2015

@jiahao is this still needed?

@jakebolewski
Copy link
Member

no

@tkelman
Copy link
Contributor

tkelman commented Aug 11, 2015

I guess #6265 did half of this? #8240 could eventually subsume tridiagonal and others as special cases.

@jiahao jiahao deleted the cjh/tridiag branch October 22, 2015 02:56
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.

5 participants