Strip BLAS dependency - Githubissues

theochem / horton

HORTON: Helpful Open-source Research TOol for N-fermion systems

http://theochem.github.io/horton/

GNU General Public License v3.0

92 stars 40 forks source link

Strip BLAS dependency #249

Closed tovrstra closed 7 years ago

tovrstra commented 7 years ago

Motivation: a few percentages of performance gain in only one piece of code is not worth a BLAS dependency. I would not mind a BLAS dependency but too many users struggle with it.

Having BLAS as a dependency requires a more advanced build system that can resolve most of the common issues for users that do not understand how to link against their BLAS implementation and it should result in a significant overall performance gain. Neither conditions were met.

Fixes #237. Fixes #189. Fixes #220.

matt-chan commented 7 years ago

There are some whitespace problems in the cpplint. Should be easy to fix. Holding off merging...