romeric / Fastor

A lightweight high performance tensor algebra framework for modern C++
MIT License
752 stars 70 forks source link

Pivoting speed #172

Closed rfabbri closed 8 months ago

rfabbri commented 1 year ago

Hi,

std::abs used in pivoting when std::norm should be faster? Eigen uses std::norm. The code is run hundreds of millions of times in some applications.

Thanks.

romeric commented 8 months ago

dbf4419 fixes the issue. Thanks for reporting.