mirror/glm
1
0
Fork 0
mirror of https://github.com/g-truc/glm.git synced 2024-09-20 00:12:17 +00:00
Code Issues Packages Projects Releases Wiki Activity

Avoid fp comparison of glm::sign return value by introducing a sign comparison helper function

Browse Source
This commit is contained in:
SGrottel 2021-06-05 16:43:47 +02:00
parent 5dbba35275
commit 104c1214ee
1 changed files with 10 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

14
test/gtx/gtx_pca.cpp
View File

@ -52,6 +52,12 @@ bool matrixEpsilonEqual(glm::mat<D, D, T, Q> const& a, glm::mat<D, D, T, Q> cons
return true;
}
template<typename T>
GLM_INLINE bool sameSign(T const& a, T const& b)
{
return ((a >= 0) && (b >= 0)) || ((a < 0) && (b < 0));
}
template<typename T>
T failReport(T line)
{
@ -518,7 +524,7 @@ int testEigenvectors(T epsilon)
{
vec act = glm::normalize(eigenvectors[i]);
vec exp = glm::normalize(_1aga::expectedEigenvectors<D>()[i]);
if (glm::sign(act[0]) != glm::sign(exp[0])) exp = -exp;
if (!sameSign(act[0], exp[0])) exp = -exp;
if (!vectorEpsilonEqual(act, exp, epsilon))
return failReport(__LINE__);
}
@ -631,13 +637,13 @@ int rndTest(unsigned int randomEngineSeed)
return failReport(__LINE__);
glm::sortEigenvalues(evals, evecs);
if (glm::sign(evecs[0][0]) != glm::sign(x[0])) evecs[0] = -evecs[0];
if (!sameSign(evecs[0][0], x[0])) evecs[0] = -evecs[0];
if(!vectorEpsilonEqual(x, evecs[0], myEpsilon<double>()))
return failReport(__LINE__);
if (glm::sign(evecs[2][0]) != glm::sign(y[0])) evecs[2] = -evecs[2];
if (!sameSign(evecs[2][0], y[0])) evecs[2] = -evecs[2];
if (!vectorEpsilonEqual(y, evecs[2], myEpsilon<double>()))
return failReport(__LINE__);
if (glm::sign(evecs[1][0]) != glm::sign(z[0])) evecs[1] = -evecs[1];
if (!sameSign(evecs[1][0], z[0])) evecs[1] = -evecs[1];
if (!vectorEpsilonEqual(z, evecs[1], myEpsilon<double>()))
return failReport(__LINE__);