Which chord-to-arc tau was Brep2Shape-250k tokenized with? Paper says 0.995, shipped BRT call sites default to 0.7
Nobody has claimed this yet.
- Dominant language
- Python
- Stars
- 11
- Forks
- 3
- PR merge metrics
- No merged PRs in 30d
Description
Question
Appendix D of the paper derives the chord-to-arc ratio criterion for the quadtree boundary decomposition and states "we empirically set tau = 0.995". However, the tokenization code this pipeline builds on (BRT, process/solid_to_triangles2.py) calls splitRectangle(face, rect, crvs, max_split=5) at both call sites without passing tol, and splitRectangle defaults to tol=0.7 (process/triangles3.py), which it forwards to chordErrorCheckInRect. The function default of chordErrorCheckInRect itself is 0.995, but it is shadowed by the caller default, so the shipped pipeline effectively runs at tau = 0.7 with max_split = 5.
Was Brep2Shape-250k tokenized with tau = 0.995 (per the paper) or with the shipped call-site default of 0.7?
Why it matters
tau controls how finely boundary-crossing quadtree cells are subdivided before the Bezier triangle fit. At 0.7 the trim curve can bend substantially inside a kept cell, so boundary triangles cover the wrong parametric region more often and the fitted control points land off the true surface on curved-boundary faces. This changes every face token on trimmed faces, so it affects both the pretraining loss floor and the comparability of reproductions.
While reproducing the paper we tokenized with the shipped defaults (0.7 / max_split 5) and matched your Table 1 statistics closely (for example MFCAD++: 29.6 mean faces / 154.8 mean edges vs the paper's 29.91 / 157.36), which suggests the released corpus may also have been built at 0.7. A one-line clarification (or passing tol=0.995 explicitly at the call sites, matching the paper) would remove the ambiguity for anyone reproducing the pipeline.
Happy to send the one-line PR to BRT or add a note to the README here, whichever you prefer.
Contributor guide
No contributing guide indexed for this repository
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start with both splitRectangle call sites in process/solid_to_triangles2.py, then read the tol default and forwarding logic in process/triangles3.py. Trace the value into chordErrorCheckInRect and compare it with the paper's Appendix D statement. Done means the Brep2Shape-250k tau is confirmed and the paper, shipped defaults, and any README or call-site clarification agree.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- python
- Domain
- computer-graphics
- Issue type
- Documentation
- Difficulty
- 2/5
- Estimated time
- 1-3 hours
- Activity status
- Quiet
- Clarity
- Clearly specified
- Newbie friendliness
- 62/100