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

a problem and some suggesion #1

Closed
GoogleCodeExporter opened this issue Dec 24, 2015 · 1 comment
Closed

a problem and some suggesion #1

GoogleCodeExporter opened this issue Dec 24, 2015 · 1 comment

Comments

@GoogleCodeExporter
Copy link

in the class EigenvalueDecomposition,the function getEigenVectors() return the 
eigenvectors of the matrix.I think each row is one eigenvector,but in 
FaceRec.sortEigenVectors(),you regarded each column as a eigenvector.Is that a 
problem?
Moreover,in FaceRec.updateEigenVectors(),if there are two different 
eigenvectors corresponding to the same eigenvale,it doesn't work.
in the code,most matrices are transposed,not the same as those in paper.it's a 
little confused.
I suggest to set the build path in Eclipse using relative path,It will 
convenient to import the project.
I am glad to see the GUI,but when I run it,it is like this(attach),in ubuntu or 
winxp.
Your code has  clarity structure using short functions,I have learned a lot 
from it,Thanks!

Original issue reported on code.google.com by NeuHanK...@gmail.com on 29 Dec 2010 at 6:55

Attachments:

@GoogleCodeExporter
Copy link
Author

thanks for pointing out the bugs..was away for a long time..Changed the code a 
bit..As for the gui layout:-) will work on it..

Original comment by sajanjoseph@gmail.com on 14 Feb 2011 at 9:48

  • Changed state: Fixed

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

1 participant