Files
knewnothing-git cb6f984bbc fix(dfam-check): address maintainer review on #265
Blockers
--------
1. requirements.txt was missing scipy, networkx and lxml. With only
   trimesh/numpy/rtree installed the first measure of any mesh died with a raw
   ModuleNotFoundError from mesh.body_count, and .3mf could not load at all.
   Reproduced in a clean venv, then confirmed fixed there: all four advertised
   formats now load and measure. Only scipy is mirrored into
   requirements-dev.txt; networkx and lxml are already listed under the repo
   tests block, matching how trimesh is handled.

2. Wall thickness is now measured per connected body. Cast against a whole
   assembly, a ray can leave one body, cross a mating clearance and land on
   its neighbour, recording the fit gap as a wall - so a tight-clearance
   assembly reported a wall violation no single part had. _wall_facts splits
   first and measures each body against itself, pooling the results and adding
   a per_body breakdown. Sample budget is shared by surface area with a floor
   so small bodies are still sampled.

Should-fix
----------
3. Each fact family now runs through _safe and degrades to an error field.
   A planar mesh dies in convex_hull; previously that cost the user the whole
   report, including the facts that had computed.

4. New scale block reports bbox_diagonal_mm and units_suspect. A meters-scale
   mesh sits under the 0.1 mm on-plate tolerance, so every down-facing face
   reads as on-plate and the part looks like a flawless print. SKILL.md now
   branches on the measured field instead of asking the agent to eyeball the
   bounding box.

5. Added the skill to the README table and the docs page array.

6. process-limits.md now names which rows have a measured counterpart today
   (walls, self-supporting angle) and which do not (hole diameter, positive
   feature, unsupported bridge), closing the eyeballing loophole.

SKILL.md also gained a rule to attribute an assembly's wall violation to the
body it belongs to.

Verification
------------
- Clean venv built from the skill's own requirements: 26 checks covering
  multi-body clearance, 81k-face performance, a 42-body assembly, planar,
  meters-scale, broken export, all four formats, degenerate mesh.
- Per-body change costs ~150 ms on an 81k-face mesh (7493 -> 7643 ms); the
  bulk of that time is trimesh's pure-Python ray fallback, not the split.
- Real part measurements unchanged: L-bracket still 1200 mm2 unsupported,
  9.0 mm median wall.
- gcode/urdf/dxf suites fail identically with and without this change; those
  are pre-existing.
2026-08-20 22:37:39 +05:30

96 lines
5.2 KiB
Markdown

---
name: dfam-check
description: Measure mesh files against Design for Additive Manufacturing (DfAM) rules and report printability findings per process (FDM, SLS, SLA/DLP, metal PBF, MJF). Use when the user asks whether a part is printable, wants overhang/wall-thickness/support analysis of an `.stl`, `.obj`, `.ply`, or `.3mf` mesh, wants a build-orientation recommendation, or wants DfAM redesign guidance before slicing with `$gcode` or regenerating geometry with `$cad`.
---
# DfAM Check
Provenance: maintained in [earthtojake/text-to-cad](https://github.com/earthtojake/text-to-cad).
Use the installed local skill files as the runtime source of truth; the
repository link is only for provenance and release review.
Use this skill to produce conservative, evidence-backed DfAM reports for mesh
files before slicing or printing. It measures geometry facts locally and
compares them against per-process design limits; it never slices, uploads, or
starts print jobs.
## Geometry Inspection
Use `scripts/dfam_tool.py` in the active project Python environment for all
geometry facts (requires `trimesh`, `numpy`, `rtree`). The tool is fact-only:
it reports measurements and never emits pass/fail or readiness statuses.
Comparisons and verdicts belong to this workflow. Do not estimate wall
thickness, overhang angles, or support volume by eye or from renders when the
tool can measure them.
```bash
python scripts/dfam_tool.py measure part.stl --angle-limit 45
python scripts/dfam_tool.py orientations part.stl --angle-limit 45
```
Set `--angle-limit` to the target process's self-supporting angle from
`references/process-limits.md` before measuring, and re-run when the target
process changes: the aggregate support-area facts are binned against it.
STEP/STP input is boundary-representation CAD, not a mesh. When the `$cad`
skill is installed, export an STL sidecar with it first, then measure the STL
here. Report that remediation instead of attempting raw STEP parsing.
## Workflow
1. Collect print intent: target process, material, layer height, and any
machine or material datasheet the user can provide. If the process is
unknown, measure once with the default 45° limit, then present findings
per candidate process rather than guessing a single verdict.
2. Read `references/process-limits.md` and select the limit column for the
target process. A user-provided machine/material datasheet overrides the
defaults; cite whichever source is used for every comparison.
3. Run `measure` on the exact upload file. Do not inspect only a generator
script, source CAD model, or console summary of the file.
4. Run `orientations` when the process requires supports and the measured
support area is nonzero. Report any candidate that materially reduces
support area, with its build-height tradeoff.
5. Compare each measured fact to the cited limit and report findings with
restrained status labels:
- `✅ pass`: the measured fact satisfies the cited limit.
- `❌ fail`: a measured fact directly violates the cited limit.
- `❓ need more info`: missing process context, unmeasured geometry,
sampling too sparse to trust, or tool limitations.
6. Order findings by severity: watertightness first (blocks slicing for
every process), then wall thickness, then overhangs/supports, then
orientation and cost signals.
## Comparison
Compare only trustworthy pairs of evidence.
- Cite the limit source (process-limits table row, or the user's datasheet
field) and the measured fact (JSON field path) for every finding.
- Treat `p05_mm` below the wall-thickness limit as a violation even when
`min_mm` alone could be a sampling outlier; report both values.
- On an assembly, `wall_thickness` reports `body_count` and a `per_body`
breakdown. Attribute a violation to the body it belongs to; a thin figure
pooled across bodies is not a finding against the part as a whole.
- Do not apply support-angle findings to powder processes (SLS, MJF); the
relevant powder-process check is trapped-volume powder escape, which this
tool does not yet measure — report that as `❓ need more info` when
enclosed cavities are likely.
- Do not silently rescale geometry. `scale.units_suspect` is measured from
the bounding-box diagonal: when it is `true`, the source is probably in
meters or inches, every down-facing face reads as resting on the plate, and
overhang and support figures of 0.0 mean nothing. Report a unit/scale
finding and ask the user to confirm units before comparing anything against
a material limit.
- Support-volume ratios are coarse upper bounds; report them as cost
signals, not hard failures, unless the user has set an explicit budget.
## Redesign Handoff
For every `❌ fail`, include a concrete, plain-language redesign instruction
with target numbers (for example "thicken the wall at [12.4, 3.0, 8.1] from
0.6 mm to ≥1.2 mm" or "chamfer the overhang at [23.3, 10.0, 52.0] to ≥45°").
When the `$cad` skill is installed, offer to apply the redesign instructions
with it and re-measure the regenerated geometry here, repeating until no
`❌ fail` findings remain. When `$cad-viewer` is installed, hand the measured
file path(s) to it so the user can inspect the findings visually.