Skip to content

Bump libigl to v2.4.0#130

Merged
alecjacobson merged 7 commits intomasterfrom
alecjacobson/bump-libigl-v2.4.0
Feb 28, 2022
Merged

Bump libigl to v2.4.0#130
alecjacobson merged 7 commits intomasterfrom
alecjacobson/bump-libigl-v2.4.0

Conversation

@alecjacobson
Copy link
Copy Markdown
Contributor

libigl-python-bindings previously grabbed libigl from a v2.2.* commit from Dec 2020.

This PR bumps libigl to the recently released v2.4.0. In order to make this compile, a few additional changes are needed:

  • update CMakeLists.txt to remove old-style options and remove all igl targets except igl::core
  • rm predicates
  • slight modification of setup.py so that it's possible to compile on my m1 mac
  • use FetchContent to grab libigl rather than custom cmake function
  • adjust flip_avoiding_line_search's API to match the bug-fixed one in libigl v2.4.0
  • rm obsolete functions point_in_poly and polygon_mesh_to_triangle_mesh.cpp
  • rm triangulation and predicate functions

I propose to merge this to get igl::core bindings for v2.4.0 in place. We could even create a corresponding release here.

Then, I plan to flesh out bindings for missing functions in the other modules of igl (predicates, etc.). We could do a minor release on this repo once those are in place.

@nodtem66
Copy link
Copy Markdown
Contributor

@alecjacobson
The builds are successful, but the tests failed.
I think some TestBasic tests should've been removed

FAILED tests/test_basic.py::TestBasic::test_delaunay_triangulation
FAILED tests/test_basic.py::TestBasic::test_incircle
FAILED tests/test_basic.py::TestBasic::test_insphere
FAILED tests/test_basic.py::TestBasic::test_is_delaunay
FAILED tests/test_basic.py::TestBasic::test_lexicographic_triangulation
FAILED tests/test_basic.py::TestBasic::test_orient2d
FAILED tests/test_basic.py::TestBasic::test_orient3d
FAILED tests/test_basic.py::TestBasic::test_point_in_poly
FAILED tests/test_basic.py::TestBasic::test_polygon_mesh_to_triangle_mesh
FAILED tests/test_basic.py::TestBasic::test_polygon_mesh_to_triangle_mesh_from_list

Alec Jacobson added 2 commits February 26, 2022 17:30
@alecjacobson alecjacobson merged commit 6bf30bc into master Feb 28, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants