-
-
Notifications
You must be signed in to change notification settings - Fork 5.5k
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
eigs(speye(21)) does not work #4246
Comments
Here is the ARPACK comment on return value 3: c = 3: No shifts could be applied during a cycle of the Here is "remark 4" c 4. At present there is no a-priori analysis to guide the selection Presently, ncv is set in the first line of aupd_wrapper() in arpack.jl:
Maybe the caller should have access to setting the parameter ncv... |
Yes I thought about the caller having access to ncv, but this really should work out of the box. |
Do we know how to fix this? |
With commit 96dcf38 this now works when |
@alanedelman Any insights here? I wonder if there is an error with our ARPACK interface or if this has something to do with the Arnoldi factorization. |
This command works in octave and they are also using ARPACK. Maybe look though the octave interface? http://octave.sourceforge.net/doxygen/html/d3/d5d/eigs-base_8cc_source.html At first glance, it looks like the ncv is chosen in the same way: 00815 if (p < 0) Sorry I don't have time to look at this any further.. |
I did take a quick look at octave, but could not find the relevant code. Thanks for pointing to it. I wonder what we are doing differently in the problem setup. |
Look at the link to the octave source above, lines 938-1002 |
That does not seem to be the right thing though.
|
You're right. I also just tried it and didn't get the right answer (last eigenvalue/vector was wrong)… Am 20.09.2013 um 06:07 schrieb Viral B. Shah:
|
tolerance is an output parameter from the first iteration, which is needed in all future uses of the data in arpack. we were resetting it to 0 (if it couldn't find a solution in the first pass), which would certainly be tough to meet at any size. |
Thanks. This is awesome detective work. |
eigs
does not work on identity matrices of size 21 and higher for the default 6 eigenvalues.The text was updated successfully, but these errors were encountered: