Incorporate Frank's suggestions for PR #315
parent
953ff1aa05
commit
1ec15a7345
|
@ -320,21 +320,12 @@ TEST(GaussianBayesTree, shortcut_overlapping_separator)
|
||||||
|
|
||||||
Matrix actualJointJ = joint.augmentedJacobian();
|
Matrix actualJointJ = joint.augmentedJacobian();
|
||||||
|
|
||||||
bool Row0RhsSignsEqual =
|
// PR 315: sign of rows in joint are immaterial
|
||||||
signbit(expectedJointJ(0, 2)) == signbit(actualJointJ(0, 2));
|
if (signbit(expectedJointJ(0, 2)) != signbit(actualJointJ(0, 2)))
|
||||||
|
|
||||||
if (!Row0RhsSignsEqual)
|
|
||||||
{
|
|
||||||
expectedJointJ.row(0) = -expectedJointJ.row(0);
|
expectedJointJ.row(0) = -expectedJointJ.row(0);
|
||||||
}
|
|
||||||
|
|
||||||
bool Row1RhsSignsEqual =
|
if (signbit(expectedJointJ(1, 2)) != signbit(actualJointJ(1, 2)))
|
||||||
signbit(expectedJointJ(1, 2)) == signbit(actualJointJ(1, 2));
|
|
||||||
|
|
||||||
if (!Row1RhsSignsEqual)
|
|
||||||
{
|
|
||||||
expectedJointJ.row(1) = -expectedJointJ.row(1);
|
expectedJointJ.row(1) = -expectedJointJ.row(1);
|
||||||
}
|
|
||||||
|
|
||||||
EXPECT(assert_equal(expectedJointJ, actualJointJ));
|
EXPECT(assert_equal(expectedJointJ, actualJointJ));
|
||||||
}
|
}
|
||||||
|
|
Loading…
Reference in New Issue