Now I would like to focus on updating the Minimize Stretch operator so it uses SLIM behind the scenes. Thus no time for updating documentation.
Great news! Thank you all for testing and reviewing this PR :)
After a closer look at the code: it turns out Eigen excepts a column-major format of input. It means that the given arrays are initialized in the following layout:
`[ all x coordinates, 0 .. n…
Can we finish this review in the nearest feature? I think there is not much work left here.
Using a linearized form is safer because we have full control over the data layout. The proper layout is essential when the arrays are passed to initialize Eigen Map objects.
There are a lot of places in the code which assumes that the input mesh does not contain degenerate triangles (because that's an assumption of the original SLIM implementation in lib IGL). In…
There are a lot of places in the code which assumes that the input mesh does not contain degenerate triangles (because that's an assumption of the original SLIM implementation in lib IGL). In…
There are a lot of places in the code which assumes that the input mesh does not contain degenerate triangles (because that's an assumption of the original SLIM implementation in lib IGL). In…
Yes, it was an arbitrary number chosen by Aurel. My understanding is it should be 'not too big, not very small'. I think giving access to this setting to the user is not justified as it is the…
In my opinion we should leave it as is for now and gather feedback from users - it's hard to predict whether this feature will be widely at all. My it's a module owner who should make the final…
I think it makes most sense the mesh can define it's UV importance group, similar to
vertex_group_active_index.
Sounds like an interesting idea but in which place in the UI could the user…
This name was originally proposed by @brecht , check here: blender/blender#114545 (comment)
Thank you for your input. I will introduce fixes after Campbell takes a look at the code (unfortunately I am not able to mark him in this message due to some reason)