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

Eigenvalues/vectors in PolyMath #15

Closed
SergeStinckwich opened this issue Dec 6, 2016 · 13 comments
Closed

Eigenvalues/vectors in PolyMath #15

SergeStinckwich opened this issue Dec 6, 2016 · 13 comments

Comments

@SergeStinckwich
Copy link
Member

SergeStinckwich commented Dec 6, 2016

There are 2 classes for supporting eigenvalues/eigenvectors at the moment: DhbJacobiTransformation and DhbLargestEigenValueFinder.

These classes are not documented and no examples are provided.

Some examples are here :

@SergeStinckwich
Copy link
Member Author

SergeStinckwich commented Dec 6, 2016

In fact, one example is available in the class comment of DhbJacobiTransformation:

| m jacobi eigenvalues eigenvectors |
m := DhbSymmetricMatrix rows: #((84 -79 58 55)
                                 (-79 84 -55 -58)
                                 (58 -55 84 79)
                                 (55 -58 79 84)).
jacobi := DhbJacobiTransformation matrix: m.
eigenvalues := jacobi evaluate.
eigenvectors := jacobi transform columnsCollect: [ :each | each].

This example should be move in the class and a proper class comment should be provided.

@oshyshkin
Copy link

How can I assign this task?

@SergeStinckwich
Copy link
Member Author

You want to assign this task to who ?

@oshyshkin
Copy link

To me)
I can not find any button or smth like this. I don't use git often, so probably I just missed something.

@SergeStinckwich
Copy link
Member Author

Because you are not of the PolyMath org. I add you

@oshyshkin oshyshkin self-assigned this Jan 24, 2017
@Uko
Copy link

Uko commented Apr 10, 2017

Hey guys, how is the work going on? I've heard that @ShyshkinOleh implemented the SVD decomposition. Is it integrated? Is something missing?

@SergeStinckwich
Copy link
Member Author

Normally this is integrated except the tests ...

@Uko
Copy link

Uko commented Apr 10, 2017

@ShyshkinOleh, can you push tests so this task will be closed before Easter?

@SergeStinckwich
Copy link
Member Author

I try to extract the tests but actually quite difficult to see the diff on metacello when you rename classes ...

@oshyshkin
Copy link

@SergeStinckwich @Uko
Currently, I have one test, which I showed to Yura, I'll push it soon and will write in Slack about it. I don't know if I can do more tests because, in general, we need to compare the final result, else should track by tests of methods I use.

@nicolas-cellier-aka-nice
Copy link
Contributor

@SergeStinckwich
Copy link
Member Author

Having more benchmarks will be nice for sure.

@oshyshkin
Copy link

I added a test. Sorry for the delay, I had trouble with a system (reinstalling).

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

No branches or pull requests

4 participants