Context
HierarchicalLSCM::setPinnedVertices() is a public instance API method but has no dedicated test. The InstanceAPILevelRatio test exercises setLevelRatio/setMinCoarseVertices but not explicit pin selection.
Suggested fix
Add a test that:
- Selects two specific boundary vertices with known positions
- Calls
hlscm.setPinnedVertices(p0, p1) and hlscm.compute(mesh)
- Verifies those vertices land at UV (0,0) and (d,0)/(0,d) as expected by the pin placement logic
Related
Found during code review of PR #44.
Context
HierarchicalLSCM::setPinnedVertices()is a public instance API method but has no dedicated test. TheInstanceAPILevelRatiotest exercisessetLevelRatio/setMinCoarseVerticesbut not explicit pin selection.Suggested fix
Add a test that:
hlscm.setPinnedVertices(p0, p1)andhlscm.compute(mesh)Related
Found during code review of PR #44.