search

gazebosim / gz-common

An audio-visual library supports processing audio and video files, a graphics library can load a variety 3D mesh file formats into a generic in-memory representation, and the core library of Gazebo Common contains functionality that spans Base64 encoding/decoding to thread pools.
https://gazebosim.org
Apache License 2.0
12 stars 38 forks source link

Flaky Test: GTSMeshUtils.DelaunayTriangulation #50

Open osrf-migration opened 5 years ago

osrf-migration commented 5 years ago

Original report (archived issue) by Michael Carroll (Bitbucket: Michael Carroll, GitHub: mjcarroll).

Found on Windows7 AMD64 CI Builds:

Error Message
unknown file
SEH exception with code 0xc0000005 thrown in the test body.
Stacktrace
unknown file
SEH exception with code 0xc0000005 thrown in the test body.
osrf-migration commented 5 years ago

Original comment by Steve Peters (Bitbucket: Steven Peters, GitHub: scpeters).

I would call this more than flaky, since it always fails our CI

osrf-migration commented 5 years ago

Original comment by Steve Peters (Bitbucket: Steven Peters, GitHub: scpeters).

I ran this test through the debugger with devenv /debugexe UNIT_GTSMeshUtils_TEST and it said the following:

Exception thrown at 0x00007FFE31322ACB (ignition-common3-graphics.dll) in
 UNIT_GTSMeshUtils_TEST.exe: 0xC0000005: Access violation writing location 0x00007FFE313E1D50.

ignition-common3-graphics.dll!ignition::common::GTSMeshUtils::DelaunayTriangulation(const std::vector<ignition::math::v6::Vector2<double>,std::allocator<ignition::math::v6::Vector2<double> > > & _vertices, const std::vector<ignition::math::v6::Vector2<int>,std::allocator<ignition::math::v6::Vector2<int> > > & _edges) Line 191
  at C:\Users\scpet\Desktop\ws\ignition\src\ign-common\graphics\src\GTSMeshUtils.cc(191)

Line 191 of GTSMeshUtils.cc attempts to set the variable gts_allow_floating_vertices to true. That is a variable defined in gts.h:

That header file appears to set some DLL macros, so I don't know if the problem is with the vcpkg build of gts or something we are doing.

osrf-migration commented 5 years ago

Original comment by Steve Peters (Bitbucket: Steven Peters, GitHub: scpeters).

test disabled for windows in pr #205