search

dmalhotra / pvfmm

A parallel kernel-independent FMM library for particle and volume potentials
http://pvfmm.org
GNU Lesser General Public License v3.0
52 stars 29 forks source link

SVD Fixed Indexerrors #1

Closed Schrigga closed 9 years ago

Schrigga commented 9 years ago

Fixed wrong indices in the diagonalisation of the SVD. Changed the bidiagonalization step on the end of the diagonalization to use two memset statements in a loop instead of two collapsed loops with an if-statement inside

yiorgb commented 9 years ago

No, it is someone else created a fork from pvfmm.

-Dhairya

On Jan 23, 2015, at 8:34 AM, George Biros gbiros@gmail.com wrote:

Hi Dhairya,

Is this from you? -g

Fixed wrong indices in the diagonalisation of the SVD. Changed the bidiagonalization step on the end of the diagonalization to use two memset statements in a loop instead of two collapsed loops with an if-statement inside

You can view, comment on, or merge this pull request online at:

https://github.com/dmalhotra/pvfmm/pull/1 https://github.com/dmalhotra/pvfmm/pull/1 Commit Summary

Accelerated SVD method Changed rebidiagonalization Fixed Indexerrors File Changes

M INSTALL https://github.com/dmalhotra/pvfmm/pull/1/files#diff-0 (5) M include/mat_utils.txx https://github.com/dmalhotra/pvfmm/pull/1/files#diff-1 (157) Patch Links:

https://github.com/dmalhotra/pvfmm/pull/1.patch https://github.com/dmalhotra/pvfmm/pull/1.patch https://github.com/dmalhotra/pvfmm/pull/1.diff https://github.com/dmalhotra/pvfmm/pull/1.diff — Reply to this email directly or view it on GitHub https://github.com/dmalhotra/pvfmm/pull/1.