Alec Jacobson

What do we actually use from all this templating?

5

# Current templating ### Dense input ```cpp template … Eigen::MatrixBase & A ``` `A` can 1. ☀️ have either storage order (`ColMajor` or `RowMajor`) 2. ☀️ Can be a `Eigen::Map`...

Maintain texture coordinates during decimation

5

From personal email: > I need to maintain texture coordinates of the result [of igl::decimate], the sample only return vertices and faces. Any advice on how to perhaps post-process the...

Eigen Gotcha exposed by using MatrixBase

8

We use `Eigen::MatrixBase` for input types. This is convenient for a number of reasons: we can place expressions like `A.row(i)` or `A.block(i,j,w,h)` directly in our function arguments. I think (confirm?)...

Clunky initialization when using multiple views

2

#### Describe your issue Is it necessary to append multiple cores during `callback_init`, like the example https://github.com/libigl/libigl/blob/master/tutorial/108_MultipleViews/main.cpp#L18 ? If so, one would have to replace the simple `.launch()` call with...

Creating this pull request on behalf of @jdumas who brings up this issue here https://github.com/libigl/libigl/pull/1597 . There are many functions that use `Eigen::Random` or `rand` or `shuffle` that apparently should...

Leaving this as a note for now... https://github.com/libigl/libigl/blob/c645aac0c5852fad6fabe8c192fdc8675e607263/include/igl/shapeup.h#L11 This header includes a bunch of things that are not needed in the header and should be moved into the cpp.

test_common::run_test_cases only runs until first failure

1

We should either rename more clearly, or, better, find a way to run all tests.

libigl compatible extension of Eigen::Matrix with strict row/column index types

3

@yig and I were discussing how using Eigen matrices for mesh types (list of vertices `V`, list of faces `F`, list of edges `E`, etc.) invites certain index **dangers**. A...

There a bunch of these and maybe later I'll try to enumerate them all. For example in C++ [`igl::boundary`](https://github.com/libigl/libigl/blob/main/include/igl/boundary_loop.h) provides these two overloads: ```cpp template IGL_INLINE void boundary_loop( const Eigen::MatrixBase&...