DLR-SC / tigl

The TiGL Geometry Library to process aircraft geometries in pre-design.
https://dlr-sc.github.io/tigl/
Apache License 2.0
241 stars 61 forks source link

Tests failed 2, passed 506 #514

Closed cfsengineering closed 6 years ago

cfsengineering commented 6 years ago

When I run the unit tests without making any change to the code, 2 tests failed, the others are fine and the tiglviewer is working.

The tests are:

Machine configuration: OS : centos 7 Used libs: oce-0.17, Qt5

The tests output:

/home/cfse/Malo/tigl/cmake-build-debug/tests/unittests/TiGL-unittests --gtest_filter=* --gtest_color=no /home/cfse/Malo/tigl/tests/unittests/testOccIssues.cpp:52: FailureValue of: extrema.NbExtrema() Actual: 1 Expected: 2 /home/cfse/Malo/tigl/tests/unittests/testOccIssues.cpp:56: FailureThe difference between c1->FirstParameter() and p1.Distance(p2) is 0.68237463714649649, which exceeds 1e-4, where c1->FirstParameter() evaluates to 0, p1.Distance(p2) evaluates to 0.68237463714649649, and 1e-4 evaluates to 0.0001. /home/cfse/Malo/tigl/tests/unittests/testOccIssues.cpp:60: FailureThe difference between 0.215758486 and u2 is 0.020546618337591915, which exceeds 1e-6, where 0.215758486 evaluates to 0.215758486, u2 evaluates to 0.19521186766240808, and 1e-6 evaluates to 9.9999999999999995e-07. unknown file: Failure FailureUnknown C++ exception thrown in the test body.

Time wingGetPoint [us]: 90Time fuselageGetPoint [us]: 0Time PointTranslator [us]: 23Triangularization time [ms]: 30 Number of Polygons/Vertices: 2434/1370Triangularization time [ms]: 1530 Number of Polygons/Vertices: 23176/11838Number of Polygons/Vertices: 13962/7368Average time: 0.000000 [ms]Runtime getIsOnTop [us]: 1800Elapsed time per projection [us]: 6.5unknown file: Failure FailureC++ exception with description "Curves do not intersect each other" thrown in the test body.

rainman110 commented 6 years ago

OCE 0.17 contains a bug. Out test cases check, whether the OpenCASCADE bug is present or not.

TiGL relies on the bugfix. Therefore we provide patched OCE binaries in our releases.

To fix this issue, either

cfsengineering commented 6 years ago

Hi, Thank you,

Yes a replace my OCE with the binaries from https://build.opensuse.org/package/show/science:dlr/OCE.

Then I run cmake with the TIGL_OCE_COONS_PATCHED=ON and it work fine. cmake -DTIGL_BUILD_TESTS=ON -DTIGL_OCE_COONS_PATCHED=ON