gonum / matrix

Matrix packages for the Go language [DEPRECATED]
446 stars 53 forks source link

fix issue #313 by increasing epsilon #367

Closed idavydov closed 8 years ago

idavydov commented 8 years ago

In some cases eigen hangs. https://gist.github.com/idavydov/eb4c88b4663a7240c92f It works ok with epsilon=1e-15 works, 1e-14 to be safe, as this code is going to be replaced in future.

kortschak commented 8 years ago

/cc @btracey

btracey commented 8 years ago

Lgtm. If only dgeev wasn't so much work...

btracey commented 8 years ago

Look good to you as well @kortschak ?

kortschak commented 8 years ago

Yes, there was not problem here for me, I just like to leave the finer points of numerical things to you since you have very much more experience.

LGTM