Added application for performing convex decomposition

[marip8]

Addition per commit message

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (4b4b9e1) 90.91% compared to head (e101cab) 90.91%.

Additional details and impacted files

@@           Coverage Diff           @@
##           master     #968   +/-   ##
=======================================
  Coverage   90.91%   90.91%           
=======================================
  Files         280      280           
  Lines       15855    15855           
=======================================
  Hits        14414    14414           
  Misses       1441     1441           

[rjoomen]

Didn't test it, but LGTM. Essentially this is a variant of TestVHACD, you could consider to use the same parameter range checks (and possibly the same parameter format?).

[Levi-Armstrong]

@marip8 Just need to fix clang-tidy issues and this can be merged.

[marip8]

@marip8 Just need to fix clang-tidy issues and this can be merged.

Looks like all the clang-tidy issues come from VHACD.h. How do we ignore errors from this file?

[rjoomen]

Looks like all the clang-tidy issues come from VHACD.h. How do we ignore errors from this file?

Seems TESSERACT_COMMON_IGNORE_WARNINGS_PUSH does not work for newer clang-tidy. You could add // NOLINTBEGIN just after TESSERACT_COMMON_IGNORE_WARNINGS_PUSH and and // NOLINTEND just before TESSERACT_COMMON_IGNORE_WARNINGS_POP. That solves it for me.

[rjoomen]

It seems adding NOLINTBEGIN and END to the macros does not work. I suggested adding them to VHACD.h, does that work?