diff options
author | Antonio Sanchez <cantonios@google.com> | 2020-09-30 08:21:20 -0700 |
---|---|---|
committer | Antonio Sanchez <cantonios@google.com> | 2020-09-30 08:42:03 -0700 |
commit | d5a0d894915c19199590b32149834351b60e2fd1 (patch) | |
tree | 514a506ba166870a94664d1d60f107bc5051c74c /Eigen/src/Core/arch/SSE | |
parent | 30960d485ec7e45b095d3ad206b2dbcc8bc835ba (diff) |
Fix alignedbox 32-bit precision test failure.
The current `test/geo_alignedbox` tests fail on 32-bit arm due to small floating-point errors.
In particular, the following is not guaranteed to hold:
```
IsometryTransform identity = IsometryTransform::Identity();
BoxType transformedC;
transformedC.extend(c.transformed(identity));
VERIFY(transformedC.contains(c));
```
since `c.transformed(identity)` is ever-so-slightly different from `c`. Instead, we replace this test with one that checks an identity transform is within floating-point precision of `c`.
Also updated the condition on `AlignedBox::transform(...)` to only accept `Affine`, `AffineCompact`, and `Isometry` modes explicitly. Otherwise, invalid combinations of modes would also incorrectly pass the assertion.
Diffstat (limited to 'Eigen/src/Core/arch/SSE')
0 files changed, 0 insertions, 0 deletions