Skip to content
Snippets Groups Projects
Commit 1234ae99 authored by Tadej Novak's avatar Tadej Novak
Browse files

Merge branch 'partialy_rev_58980' into 'master'

partially revert a change from MR58980 as it seems to create a small cpu regression

See merge request atlas/athena!59047
parents 6d780a1b b6e2f9c4
No related branches found
No related tags found
2 merge requests!59383cppcheck in trigger code: Prefer prefix ++/-- operators for non-primitive types.,!59047partially revert a change from MR58980 as it seems to create a small cpu regression
......@@ -64,8 +64,9 @@ inline bool saneCovarianceDiagonal(const AmgSymMatrix(N) & mat) {
//// length of the vector is still within the Geneva area, i.e. 10 km
template <int N>
inline bool saneVector(const AmgVector(N) & vec) {
if (vec.hasNaN() || !vec.allFinite()) {
return false;
for (int i = 0; i < N; ++i) {
if (std::isnan(vec[i]) || std::isinf(vec[i]))
return false;
}
constexpr double max_length2 = 1.e16;
return vec.dot(vec) < max_length2;
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment