Skip to content

fix: change big_face default to false in vnf_validate (#1563)#1969

Open
junhinhow wants to merge 1 commit intoBelfrySCAD:masterfrom
junhinhow:fix/vnf-validate-1563
Open

fix: change big_face default to false in vnf_validate (#1563)#1969
junhinhow wants to merge 1 commit intoBelfrySCAD:masterfrom
junhinhow:fix/vnf-validate-1563

Conversation

@junhinhow
Copy link
Copy Markdown

Summary

  • Change big_face default from true to false in vnf_validate() module
  • Large faces (non-triangular, even non-planar) render correctly in both CGAL and Manifold
  • The BIG_FACE warning produced false positives on valid geometry
  • Users can still enable with big_face=true when needed
  • Fix typo in docs: big_false -> big_face

Test plan

  • vnf_validate() no longer warns on valid non-triangular faces by default
  • big_face=true still enables the check
  • No regressions in other validation checks

🤖 Generated with Claude Code

The BIG_FACE check (faces with more than 3 vertices) generated warnings
for valid geometry that intentionally uses quads or larger polygons.

Changes:
- Set big_face=false by default in vnf_validate() module
- Function form already defaulted to false, now module matches
- Fix typo "big_false" -> "big_face" in documentation
- Update description to clarify the parameter's purpose
@junhinhow junhinhow force-pushed the fix/vnf-validate-1563 branch from 404650c to 53d3d90 Compare April 6, 2026 13:49
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.

1 participant