draftwright 0.4.5__tar.gz → 0.4.6__tar.gz

{draftwright-0.4.5 → draftwright-0.4.6}/CHANGELOG.md

@@ -1,6 +1,30 @@
 # Changelog
 
-## Unreleased
+## v0.4.6 — 2026-08-15
+
+### Changed
+
+- **Geometry recognition is supplied by the standalone Apache-2.0
+  [`b123d-recognisers`](https://github.com/pzfreo/b123d-recognisers) package.** The cutover
+  pins the published stable release `v0.1.0` (built from commit
+  `9e622716c14d491729b5191aa6e5d8351c982a51`), removes the
+  duplicate embedded implementation, and keeps only identity-preserving
+  `draftwright.recognition` / `draftwright.score` compatibility re-exports until 0.6.0.
+  Draftwright continues to own the one-result build/lint cache, record→IR conversion, drafting
+  policy, placement, and critique. The package migration is manifest-pinned and golden-backed;
+  its sole normalization makes an exact dominant-axis tie deterministic across operating
+  systems without changing feature policy.
+
+### Fixed
+
+- Hole-callout obstacle checks now account for the rendered leader shaft width and
+  tip-local arrowhead flare instead of treating the leader as a zero-width centreline
+  (#367). The analytical check remains geometry-only and does not over-reserve the
+  arrowhead's clearance along the whole shaft.
+- Exact X/Z slanted flats retain their established lint-clean side-view callout after the
+  recogniser package's cross-platform semantic-axis normalization. The normalized recognition
+  record remains unchanged; `FlatFeature.presentation_axis` carries the downstream drawing-view
+  choice, preventing a leader/silhouette crossing without duplicating recognition.
 
 ## v0.4.5 — 2026-08-15
 

{draftwright-0.4.5 → draftwright-0.4.6}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.5
 Name: draftwright
-Version: 0.4.5
+Version: 0.4.6
 Summary: Automated technical-drawing generation for build123d
 Project-URL: Homepage, https://github.com/pzfreo/draftwright
 Project-URL: Repository, https://github.com/pzfreo/draftwright
@@ -684,6 +684,7 @@ Classifier: Programming Language :: Python :: 3.14
 Classifier: Topic :: Scientific/Engineering
 Requires-Python: <3.15,>=3.10
 Requires-Dist: antlr4-python3-runtime<4.10
+Requires-Dist: b123d-recognisers==0.1.0
 Requires-Dist: build123d-drafting-helpers>=0.14.1
 Requires-Dist: build123d<0.11,>=0.9.0; python_full_version < '3.13'
 Requires-Dist: build123d<0.12,>=0.11; python_full_version >= '3.13'
@@ -967,17 +968,20 @@ or declared features converge on a `PartModel` IR, then planner-fed and
 sanctioned model-routed render intents feed shared placement, projection, and
 export. Coverage lint independently compares recognised geometry with the
 placed drawing, so an upstream omission cannot hide from verification. It
-builds on two libraries:
+builds on three libraries:
 
 ```
 draftwright
+    └── b123d-recognisers          — deterministic geometry recognition
     └── build123d-drafting-helpers  — Dimension, Leader, HoleCallout, …
     └── build123d                   — CAD kernel
 ```
 
-It owns feature recognition (`recognition/`) and linting (`linting/`); annotation
-primitives (`Dimension`, `Leader`, etc.) live in `build123d-drafting-helpers` and can
-be used independently. The compiler is largely converged in production — turned
+Geometry recognition lives in the Apache-2.0 `b123d-recognisers` package. Draftwright owns
+the per-build recognition cache, record→IR conversion, drafting policy, and linting
+(`linting/`); annotation primitives (`Dimension`, `Leader`, etc.) live in
+`build123d-drafting-helpers` and can be used independently. The compiler is largely converged
+in production — turned
 dims/lengths, centre marks, envelope, slots, holes (callouts/locations/grouping),
 the section A–A trigger, the prismatic step-ladder + rotational furniture, and PMI/GD&T
 are all on the IR. See

{draftwright-0.4.5 → draftwright-0.4.6}/README.md

@@ -268,17 +268,20 @@ or declared features converge on a `PartModel` IR, then planner-fed and
 sanctioned model-routed render intents feed shared placement, projection, and
 export. Coverage lint independently compares recognised geometry with the
 placed drawing, so an upstream omission cannot hide from verification. It
-builds on two libraries:
+builds on three libraries:
 
 ```
 draftwright
+    └── b123d-recognisers          — deterministic geometry recognition
     └── build123d-drafting-helpers  — Dimension, Leader, HoleCallout, …
     └── build123d                   — CAD kernel
 ```
 
-It owns feature recognition (`recognition/`) and linting (`linting/`); annotation
-primitives (`Dimension`, `Leader`, etc.) live in `build123d-drafting-helpers` and can
-be used independently. The compiler is largely converged in production — turned
+Geometry recognition lives in the Apache-2.0 `b123d-recognisers` package. Draftwright owns
+the per-build recognition cache, record→IR conversion, drafting policy, and linting
+(`linting/`); annotation primitives (`Dimension`, `Leader`, etc.) live in
+`build123d-drafting-helpers` and can be used independently. The compiler is largely converged
+in production — turned
 dims/lengths, centre marks, envelope, slots, holes (callouts/locations/grouping),
 the section A–A trigger, the prismatic step-ladder + rotational furniture, and PMI/GD&T
 are all on the IR. See

{draftwright-0.4.5 → draftwright-0.4.6}/docs/adr/README.md

@@ -25,15 +25,15 @@ architecture** table; open retired or superseded records only for design history
 | [0004](0004-compose-then-pack-view-blocks.md) | Compose-then-pack: views as blocks carrying their annotation footprint | Compose each view with its annotation footprint, then pack fixed-topology blocks. | Accepted | `test_layout.py`, `test_layout_cleanliness.py`, `test_refactor_golden.py` |
 | [0005](0005-pipeline-architecture-and-state-ownership.md) | Compiler-pipeline module boundaries and single-owner build state | Give compiler stages explicit module homes and build-time state explicit owners. | Accepted; implemented; compatibility aliases tracked by #720 | `test_import_boundaries.py`, `test_drawing_encapsulation.py`, `test_registry.py` |
 | [0006](0006-deterministic-layout-via-bundled-fonts.md) | Deterministic cross-platform layout via bundled, path-pinned fonts | Pin bundled font files so text measurement and layout are cross-platform deterministic. | Accepted | `test_refactor_golden.py`, `test_layout_cleanliness.py` |
-| [0007](0007-own-recognition-and-linting.md) | draftwright owns feature recognition and linting; helpers becomes the rendering library | draftwright owns recognition and linting; helpers remains a rendering library. | Accepted | `test_import_boundaries.py`, `test_recognition.py`, `test_linting.py` |
+| [0007](0007-own-recognition-and-linting.md) | draftwright owns feature recognition and linting; helpers becomes the rendering library | Draftwright owns linting, recognition lifecycle, IR conversion, and drafting policy; `b123d-recognisers` owns geometry recognition; helpers remains the rendering library. | Accepted; extraction deployed by Amendment 2 | `test_import_boundaries.py`, `test_external_recognition_boundary.py`, `test_linting.py` |
 | [0010](0010-annotation-provenance-seam.md) | Annotation provenance: record intent → annotation once, at the render seam | Record annotation provenance once at the render/add seam. | Accepted; landed | `test_render_seam.py`, `test_registry.py` |
 | [0011](0011-ir-as-public-input.md) | The IR as a public input: declare features, don't only detect them | Accept the feature IR as public input through declarations and the `Sheet` façade. | Accepted; core landed; #62/#462/#495 remain | `test_declare.py`, `test_object_aspects.py`, `test_sheet_gdt.py` |
 | [0012](0012-edits-as-pinned-priority-candidates-in-the-global-solve.md) | User annotation edits are pinned, priority-ranked corridor candidates | Drain recorded semantic edits through corridor placement with pin and priority. | Accepted; partially landed; full recomposition/parity remains #426/#707 (#661 detail views landed) | `test_make_drawing.py`, `test_sheet_emit.py` |
-| [0013](0013-uniform-recognition-and-shared-package.md) | A uniform recogniser/feature contract (with `b123d-recognisers` as its deferred shared deployment) | Enforce a uniform geometry-only recogniser contract internally; defer package extraction until there is a second consumer. | Accepted; Phase 1 in progress; extraction deferred | `test_recogniser_contract.py`, `test_import_boundaries.py` |
+| [0013](0013-uniform-recognition-and-shared-package.md) | A uniform recogniser/feature contract and shared `b123d-recognisers` deployment | Enforce the geometry-only contract in the shared package and consume it through Draftwright's record→IR boundary. | Accepted; Phases 1–2 deployed | `test_external_recognition_boundary.py`, `test_detect_registry.py`, package semantic goldens |
 | [0014](0014-collect-then-solve-annotation-placement.md) | Collect-then-solve annotation placement (as built) | Collect, select, assign, and deterministically solve annotations per corridor before rendering. | Accepted; supersedes 0009 | `test_carve_free_position_callers.py`, `test_strip_layout.py`, `test_layout_property.py`, `test_import_boundaries.py` |
 | [0015](0015-part-drawing-compiler-as-built.md) | The part-drawing compiler, as built | Use one detected-or-declared feature IR and planner-fed dimension groups as the compiler waist. | Accepted; supersedes 0008 | `test_part_model.py`, `test_detect_once.py`, `test_import_boundaries.py` |
 | [0016](0016-declared-dimensioning-intent.md) | Declared dimensioning intent: capture what to measure, let the engine place it | Declare which measurements matter as scale-independent intent routed through the planner and corridor solve; never hardcode dimension geometry. | Accepted; epic #867 complete; phase 6 landed (#940) | `test_compiled_plan_boundary.py`, `test_label_provenance.py`, `test_sheet_emit.py`, `test_add_dimension.py` |
-| [0017](0017-recognition-inventory-correspondence-and-measurement-provenance.md) | One recognition result per run; correspondence is evidence-gated | Produce one explicit recognition result owned by `BuildState`; require vertical-slice evidence before generalising correspondence, identity, requirements, outcomes, or reconciliation. | Accepted; ownership phase landed, extensions gated by #1018 | `test_recognition_manifest.py`, `test_recognition_result.py`, `test_declared_recognition_gate.py` |
+| [0017](0017-recognition-inventory-correspondence-and-measurement-provenance.md) | One recognition result per run; correspondence is evidence-gated | Produce one external immutable recognition result held by Draftwright's per-run cache; require vertical-slice evidence before generalising correspondence, identity, requirements, outcomes, or reconciliation. | Accepted; external cache ownership clarified, extensions gated by #1018 | `test_external_recognition_boundary.py`, `test_recognition_manifest.py`, `test_declared_recognition_gate.py` |
 | [0018](0018-requirement-driven-view-planning-and-editable-sheet-layout.md) | Requirement-driven view planning and editable sheet layout | Use one `ViewSpec` vocabulary and planner with distinct authored `ViewConstraints` and immutable `ResolvedViewPlan`; jointly validate views, typography, convention, scale, paper and layout against requirement survival. If accepted, supersedes ADR 0004's fixed-topology assumption while retaining compose-then-pack. | Proposed; tracked by #1130 | Required guards are listed in the ADR |
 
 ## Historical records

{draftwright-0.4.5 → draftwright-0.4.6}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "hatchling.build"
 
 [project]
 name = "draftwright"
-version = "0.4.5"
+version = "0.4.6"
 description = "Automated technical-drawing generation for build123d"
 readme = "README.md"
 license = { file = "LICENSE" }
@@ -18,6 +18,7 @@ dependencies = [
     "build123d>=0.9.0,<0.11 ; python_full_version < '3.13'",
     "build123d>=0.11,<0.12 ; python_full_version >= '3.13'",
     "build123d-drafting-helpers>=0.14.1",
+    "b123d-recognisers==0.1.0",
     "numpy>=1.24",
     # PNG raster export (SVG→PDF→PNG). Both permissively licensed (Pillow HPND; pypdfium2
     # Apache-2.0 wrapping Google PDFium BSD-3) and ship pre-built wheels with NO native system
@@ -163,6 +164,7 @@ dev = [
     "pytest-timeout>=2",
     "pytest-xdist>=3",
     "ruff>=0.4",
+    "tomli>=2; python_version < '3.11'",
 ]
 
 [tool.codespell]

{draftwright-0.4.5 → draftwright-0.4.6}/src/draftwright/_core.py

@@ -25,8 +25,9 @@ from types import SimpleNamespace
 from typing import TYPE_CHECKING, Literal
 
 if TYPE_CHECKING:
+    from b123d_recognisers import RecognitionResult, TurnedProfile
+
     from draftwright.compose import StripDepths
-    from draftwright.recognition import RecognitionResult, TurnedProfile
 
 from build123d import Align, BoundBox, Compound, Edge, Location, Mode, Shape, Text, Vector
 from build123d_drafting.helpers import (

{draftwright-0.4.5 → draftwright-0.4.6}/src/draftwright/_geometry.py

@@ -327,6 +327,94 @@ def _segment_clips_box(p, q, box, pad=0.0) -> bool:
     return _segment_clip_extent(p, q, box, pad=pad) is not None
 
 
+def _convex_polygon_overlaps_box(points, box) -> bool:
+    """Whether a convex page-plane polygon has positive-area overlap with an AABB.
+
+    This is the small separating-axis primitive used by
+    :func:`_leader_ink_crosses_box`.  Keeping it here makes the rendered-ink model pure
+    rectangle/vector maths: placement does not build temporary OCC geometry merely to ask
+    whether a candidate is clear (ADRs 0004/0014).
+    """
+    corners = (
+        (box[0], box[1]),
+        (box[2], box[1]),
+        (box[2], box[3]),
+        (box[0], box[3]),
+    )
+    axes = [(1.0, 0.0), (0.0, 1.0)]
+    axes.extend(
+        (-(second[1] - first[1]), second[0] - first[0])
+        for first, second in zip(points, (*points[1:], points[0]), strict=True)
+    )
+    for ax, ay in axes:
+        if abs(ax) + abs(ay) <= 1e-12:
+            continue
+        polygon_projection = [x * ax + y * ay for x, y in points]
+        box_projection = [x * ax + y * ay for x, y in corners]
+        if max(polygon_projection) <= min(box_projection) or max(box_projection) <= min(
+            polygon_projection
+        ):
+            return False
+    return True
+
+
+def _leader_ink_crosses_box(
+    tip,
+    elbow,
+    box,
+    *,
+    arrow_length: float,
+    line_width: float,
+) -> bool:
+    """Whether a rendered leader's tip-to-elbow ink overlaps *box*.
+
+    A leader is not a zero-width segment.  Its shaft is a swept rectangle of
+    ``line_width`` and each supported arrowhead style is contained by the triangle from
+    the tip to a base ``arrow_length`` along the shaft, with the helper's
+    ``arrow_length / 3`` lateral flare.  Testing those two local convex footprints keeps
+    the arrow clearance near the tip; inflating the entire shaft by the arrow size would
+    reject genuinely clear obstacles near the elbow (#367).
+
+    Touching boundaries are clear, matching :func:`_boxes_overlap` and the placement-side
+    semantics of :func:`_segment_crosses_box`.
+    """
+    dx, dy = float(elbow[0]) - float(tip[0]), float(elbow[1]) - float(tip[1])
+    length = math.hypot(dx, dy)
+    half_line = max(0.0, float(line_width)) / 2.0
+    head_length = max(0.0, float(arrow_length))
+
+    if length <= 1e-12:
+        half = max(half_line, head_length / 3.0)
+        point_box = (tip[0] - half, tip[1] - half, tip[0] + half, tip[1] + half)
+        return _boxes_overlap(point_box, box)
+
+    ux, uy = dx / length, dy / length
+    vx, vy = -uy, ux
+
+    if half_line > 0.0:
+        shaft = (
+            (tip[0] + vx * half_line, tip[1] + vy * half_line),
+            (elbow[0] + vx * half_line, elbow[1] + vy * half_line),
+            (elbow[0] - vx * half_line, elbow[1] - vy * half_line),
+            (tip[0] - vx * half_line, tip[1] - vy * half_line),
+        )
+        if _convex_polygon_overlaps_box(shaft, box):
+            return True
+
+    if head_length > 0.0:
+        half_head = head_length / 3.0
+        base_x, base_y = tip[0] + ux * head_length, tip[1] + uy * head_length
+        arrow = (
+            (tip[0], tip[1]),
+            (base_x + vx * half_head, base_y + vy * half_head),
+            (base_x - vx * half_head, base_y - vy * half_head),
+        )
+        if _convex_polygon_overlaps_box(arrow, box):
+            return True
+
+    return False
+
+
 def _segments_cross_or_overlap(a1, a2, b1, b2) -> bool:
     """Whether two page-plane segments cross or overlap beyond a shared endpoint.
 

{draftwright-0.4.5 → draftwright-0.4.6}/src/draftwright/analysis.py

@@ -17,6 +17,14 @@ from collections.abc import Callable
 from dataclasses import dataclass, replace
 from typing import cast
 
+from b123d_recognisers import (
+    RecognitionResult,
+    TurnedProfile,
+    TurnedStep,
+    analyse_cylinders,
+    build_recognition_result,
+    full_cylinders,
+)
 from build123d import Compound, Shape
 from build123d_drafting.helpers import draft_preset
 from OCP.IFSelect import IFSelect_ReturnStatus
@@ -47,14 +55,6 @@ from draftwright.compose import (
 from draftwright.model import build_part_model
 from draftwright.model.ir import Datum, PartModel, StepFeature, StepLevelFeature
 from draftwright.model.planner import plan_dimensions
-from draftwright.recognition import (
-    RecognitionResult,
-    TurnedProfile,
-    TurnedStep,
-    analyse_cylinders,
-    build_recognition_result,
-    full_cylinders,
-)
 
 _log = logging.getLogger(__name__)
 

{draftwright-0.4.5 → draftwright-0.4.6}/src/draftwright/annotations/from_model.py

@@ -1881,6 +1881,7 @@ def render_flats(dwg, plan, a, *, ctx, only=None) -> int:
         collapse.setdefault(
             (
                 g.facts.axis,
+                g.facts.presentation_axis,
                 g.facts.axis_direction,
                 g.facts.axis_line,
                 g.facts.stock_span,
@@ -1889,7 +1890,7 @@ def render_flats(dwg, plan, a, *, ctx, only=None) -> int:
             [],
         ).append((g, pd))
     jobs = []
-    for gi, ((axis, _direction, _line, _span, _across), members) in enumerate(
+    for gi, ((_axis, presentation_axis, _direction, _line, _span, _across), members) in enumerate(
         sorted(collapse.items())
     ):
         if only is not None:
@@ -1920,7 +1921,7 @@ def render_flats(dwg, plan, a, *, ctx, only=None) -> int:
         )
         jobs.append(
             (
-                f"m_flat_{axis}{gi}",
+                f"m_flat_{presentation_axis}{gi}",
                 view,
                 vb,
                 _flat_label(members[0][1].value_text, _tol_suffix(tol, draft)),

{draftwright-0.4.5 → draftwright-0.4.6}/src/draftwright/annotations/holes.py

@@ -33,7 +33,7 @@ from draftwright._core import (
     _iso_bbox,
     _log,
 )
-from draftwright._geometry import plane_axes
+from draftwright._geometry import _leader_ink_crosses_box, plane_axes
 from draftwright.annotations._common import (
     CROSSABLE_TYPES,
     CorridorCandidate,
@@ -42,7 +42,6 @@ from draftwright.annotations._common import (
     _geom_box,
     _hole_location_coverage_fact,
     _same_location_ordinate,
-    _segment_crosses_box,
     _with_hole_center_coverage,
     _with_hole_location_coverage,
     box_within_page_and_clear,
@@ -1885,15 +1884,25 @@ def _needs_section(feat: HoleFeature | PatternFeature):
     return bore.cbore is not None or bore.spotface is not None or not bore.through
 
 
-def _leader_hits(leader, tip, elbow, side, obstacles):
+def _leader_hits(leader, tip, elbow, side, obstacles, draft):
     """True when *leader*'s rendered footprint truly overlaps any *obstacles* box — split into
-    the diagonal tip→elbow SHAFT (checked precisely via `_segment_crosses_box`, since its AABB
-    over-claims the empty triangle it doesn't occupy — #305) and the elbow→label shelf+text
-    (axis-aligned, so the coarse AABB check there is already exact). Promoted (#638; pure)."""
+    the diagonal tip→elbow SHAFT + ARROWHEAD (checked against their analytical rendered ink,
+    since the composite AABB over-claims the empty triangle it doesn't occupy — #305/#367) and
+    the elbow→label shelf+text (axis-aligned, so the coarse AABB check there is already exact).
+    Promoted (#638; pure)."""
     full_box = _geom_box(leader)
     if full_box is None or not _box_hits(full_box, obstacles):
         return False  # fast reject: nowhere near any obstacle
-    if any(_segment_crosses_box(tip, elbow, o) for o in obstacles):
+    if any(
+        _leader_ink_crosses_box(
+            tip,
+            elbow,
+            obstacle,
+            arrow_length=draft.arrow_length,
+            line_width=draft.line_width,
+        )
+        for obstacle in obstacles
+    ):
         return True
     label_box = (
         (elbow[0], full_box[1], full_box[2], full_box[3])
@@ -2433,7 +2442,7 @@ def _place_queue(
             continue
         y = final_y[tid]
         leader, tip, elbow = _build_leader_at(s, edge, side, y, to_page, elbow_dx, draft, a.SCALE)
-        if _leader_hits(leader, tip, elbow, side, occupied):
+        if _leader_hits(leader, tip, elbow, side, occupied, draft):
             crossing.append((s, y, leader))
         else:
             placed.append((s, y, leader))

{draftwright-0.4.5 → draftwright-0.4.6}/src/draftwright/drawing.py

@@ -20,7 +20,7 @@ from dataclasses import field as dataclasses_field
 from typing import TYPE_CHECKING, Any, NamedTuple
 
 if TYPE_CHECKING:
-    from draftwright.recognition import RecognitionResult
+    from b123d_recognisers import RecognitionResult
 
 # PEP 702 @deprecated. A `sys.version_info` guard (not try/except) so the type checker,
 # which targets the 3.10 floor, resolves the backport branch instead of `warnings.deprecated`
@@ -30,6 +30,7 @@ if sys.version_info >= (3, 13):
 else:
     from typing_extensions import deprecated
 
+from b123d_recognisers import analyse_cylinders
 from build123d import (
     Color,
     ExportSVG,
@@ -98,7 +99,7 @@ from draftwright.linting.quality import quality_components
 from draftwright.projection import (
     project_view_geometry,
 )
-from draftwright.recognition import analyse_cylinders, build_recognition_result
+from draftwright.recognition_cache import RecognitionCache
 from draftwright.registry import AnnotationRegistry
 from draftwright.repair import repair_drawing
 
@@ -320,7 +321,7 @@ class BuildState:
     """
 
     analysis: Analysis | None = None
-    recognition: RecognitionResult | None = None
+    recognition_cache: RecognitionCache = dataclasses_field(default_factory=RecognitionCache)
     part_model: object | None = None
     view_edge_cache: dict = dataclasses_field(default_factory=dict)
     ann_box_cache: dict = dataclasses_field(default_factory=dict)
@@ -335,6 +336,16 @@ class BuildState:
     #: suppression rule produced four issue reports before anyone found the rule (#997).
     omissions: tuple = ()
 
+    @property
+    def recognition(self) -> RecognitionResult | None:
+        """The immutable result held by Draftwright's consumer-owned lifecycle cache."""
+
+        return self.recognition_cache.result
+
+    @recognition.setter
+    def recognition(self, value: RecognitionResult | None) -> None:
+        self.recognition_cache.result = value
+
     def clear_geometry_caches(self) -> None:
         """The one invalidation seam (finalize rollback): view edges + annotation
         boxes together — a rolled-back drawing must re-measure everything."""
@@ -353,9 +364,7 @@ class BuildState:
         On a detected build ``recognition`` is already filled by the builder, so this returns
         it and recognises nothing.
         """
-        if self.recognition is None:
-            self.recognition = build_recognition_result(part)
-        return self.recognition
+        return self.recognition_cache.ensure(part)
 
 
 class Drawing:

{draftwright-0.4.5 → draftwright-0.4.6}/src/draftwright/linting/channel_coverage.py

@@ -5,8 +5,9 @@ from __future__ import annotations
 from dataclasses import dataclass
 from typing import Literal
 
+from b123d_recognisers import RecognitionResult, has_multi_axis_plates
+
 from draftwright.linting.issues import LintIssue
-from draftwright.recognition import RecognitionResult, has_multi_axis_plates
 
 ChannelRequirementState = Literal["placed", "suppressed", "dropped", "missing", "unverifiable"]
 

{draftwright-0.4.5 → draftwright-0.4.6}/src/draftwright/linting/coverage.py

@@ -24,20 +24,7 @@ from collections.abc import Iterable
 from math import atan2, pi, tau
 from typing import Literal
 
-from build123d import GeomType
-from build123d_drafting.helpers import CenterMark, Dimension, TitleBlock
-
-from draftwright._core import (
-    _END_ON,
-    HoleRef,
-    _annotation_diameter_sources,
-    _axis_letter,
-    _fmt,
-    _xyz,
-)
-from draftwright.linting.issues import LintIssue
-from draftwright.linting.profiled_bore_coverage import profiled_bore_key
-from draftwright.recognition import (
+from b123d_recognisers import (
     RecognitionResult,
     TurnedProfile,
     analyse_cylinders,
@@ -51,11 +38,24 @@ from draftwright.recognition import (
     recognise_rectangular_pads,
     recognise_turned_steps,
 )
-from draftwright.recognition.profiled_bores import (
+from b123d_recognisers.profiled_bores import (
     double_d_bores_from_openings,
     double_d_profile,
     principal_boundary_plane,
 )
+from build123d import GeomType
+from build123d_drafting.helpers import CenterMark, Dimension, TitleBlock
+
+from draftwright._core import (
+    _END_ON,
+    HoleRef,
+    _annotation_diameter_sources,
+    _axis_letter,
+    _fmt,
+    _xyz,
+)
+from draftwright.linting.issues import LintIssue
+from draftwright.linting.profiled_bore_coverage import profiled_bore_key
 
 _UNSET = object()  # sentinel: distinguishes "not supplied" from a valid prof=None
 

{draftwright-0.4.5 → draftwright-0.4.6}/src/draftwright/linting/flat_coverage.py

@@ -11,9 +11,10 @@ from collections import defaultdict
 from dataclasses import dataclass
 from typing import Literal
 
+from b123d_recognisers import RecognitionResult
+
 from draftwright._geometry import _canonical_axis_direction
 from draftwright.linting.issues import LintIssue
-from draftwright.recognition import RecognitionResult
 
 FlatRequirementState = Literal["placed", "suppressed", "dropped", "missing", "unverifiable"]
 

{draftwright-0.4.5 → draftwright-0.4.6}/src/draftwright/linting/hole_coverage.py

@@ -12,9 +12,10 @@ from dataclasses import dataclass
 from math import hypot
 from typing import Literal
 
+from b123d_recognisers import HoleSpec, RecognitionResult, countersink_matches_hole
+
 from draftwright._geometry import _is_principal_axis
 from draftwright.linting.issues import LintIssue
-from draftwright.recognition import HoleSpec, RecognitionResult, countersink_matches_hole
 
 HoleRequirementState = Literal["placed", "suppressed", "dropped", "missing", "unverifiable"]
 HoleSourceKind = Literal["hole", "hole_pattern"]

{draftwright-0.4.5 → draftwright-0.4.6}/src/draftwright/linting/polygonal_stock_coverage.py

@@ -5,8 +5,9 @@ from __future__ import annotations
 from dataclasses import dataclass
 from typing import Literal
 
+from b123d_recognisers import RecognitionResult
+
 from draftwright.linting.issues import LintIssue
-from draftwright.recognition import RecognitionResult
 
 PolygonalStockState = Literal["placed", "suppressed", "dropped", "missing", "unverifiable"]
 

{draftwright-0.4.5 → draftwright-0.4.6}/src/draftwright/linting/profiled_bore_coverage.py

@@ -11,9 +11,10 @@ from collections import Counter
 from math import hypot, isfinite
 from typing import Literal
 
+from b123d_recognisers import RecognitionResult
+
 from draftwright._geometry import _fmt
 from draftwright.linting.issues import LintIssue
-from draftwright.recognition import RecognitionResult
 
 
 def _axis_letter(value) -> str:

{draftwright-0.4.5 → draftwright-0.4.6}/src/draftwright/linting/slot_coverage.py

@@ -12,8 +12,9 @@ from collections import defaultdict
 from dataclasses import dataclass
 from typing import Literal
 
+from b123d_recognisers import RecognitionResult
+
 from draftwright.linting.issues import LintIssue
-from draftwright.recognition import RecognitionResult
 
 SlotRequirementState = Literal["placed", "suppressed", "dropped", "missing", "unverifiable"]
 SlotSourceKind = Literal["slot", "slot_pattern"]

{draftwright-0.4.5 → draftwright-0.4.6}/src/draftwright/model/compiled.py

@@ -272,7 +272,14 @@ _FACTS: dict[str, tuple[str, ...]] = {
     # These are STRUCTURE, not measurements: together they say which piece of stock a flat
     # belongs to, so the renderer can tell one double-D's two faces from independent aligned
     # or slanted regions (#1013/#1036). The drawing prints none of them.
-    "flat": ("frame", "axis", "axis_line", "stock_span", "axis_direction"),
+    "flat": (
+        "frame",
+        "axis",
+        "presentation_axis",
+        "axis_line",
+        "stock_span",
+        "axis_direction",
+    ),
     "groove": ("frame", "axis"),
     "plate": ("frame", "axis"),
     # Raw AP242 PMI is the documented non-generated exception: its source-authored label

{draftwright-0.4.5 → draftwright-0.4.6}/src/draftwright/model/declare.py

@@ -290,7 +290,7 @@ def double_d_bore(
     with exactly that boundary; arbitrary line/arc profiles and blind profiles fail closed.
     """
     if obj is not None:
-        from draftwright.recognition.profiled_bores import read_double_d_tool
+        from b123d_recognisers.profiled_bores import read_double_d_tool
 
         r_axis, r_major, r_af, r_at, r_depth, r_direction = read_double_d_tool(obj)
         axis = r_axis if axis is None else axis
@@ -329,12 +329,11 @@ def read_countersink(cone) -> tuple[float, float]:
     """``(major_diameter, included_angle°)`` of a countersink **cone** tool — the larger rim ⌀
     and the full cone angle, read off its conical **face** (not a removed edge, #576 lesson).
     The cone geometry is the recogniser's own
-    :func:`~draftwright.recognition.countersinks.cone_rims` (#704), so a declared
+    :func:`~b123d_recognisers.countersinks.cone_rims` (#704), so a declared
     countersink reads identically to a detected one by construction."""
+    from b123d_recognisers import cone_rims
     from build123d import GeomType
 
-    from draftwright.recognition import cone_rims
-
     faces = cone.faces().filter_by(GeomType.CONE)
     if not faces:
         raise ValueError("countersink(cone=...) needs a conical tool (a build123d Cone)")
@@ -506,10 +505,10 @@ def _read_chamfer_face(face) -> tuple[str, float, float, Point]:
     """Read a chamfer off its **oblique planar bevel face**: the axis (the edge the chamfer
     runs along), the two legs (the face's in-plane extents), and a point **on the bevel** (the
     face centre — not the removed sharp corner). The classification + leg geometry is the
-    recogniser's own :func:`~draftwright.recognition.chamfers.classify_bevel` (#704), so a
+    recogniser's own :func:`~b123d_recognisers.chamfers.classify_bevel` (#704), so a
     declared chamfer reads identically to a detected one by construction; only the
     user-facing error messages live here."""
-    from draftwright.recognition import BevelReject, classify_bevel
+    from b123d_recognisers import BevelReject, classify_bevel
 
     try:
         edge_i, _nv, _span, hi, lo = classify_bevel(face)
@@ -597,7 +596,7 @@ def _read_fillet_face(face) -> tuple[str, float, Point]:
     # Anchor on the curved radius surface itself — the recogniser's own fillet_anchor
     # (#622 lesson: never the bbox centre), so a declared fillet's leader tip is
     # identical to the detected one's by construction (#704).
-    from draftwright.recognition import fillet_anchor
+    from b123d_recognisers import fillet_anchor
 
     p = fillet_anchor(s)
     return (
@@ -722,7 +721,7 @@ def _read_groove_face(face) -> tuple[str, float, float, Point]:
     span = ((bb.min.X, bb.max.X), (bb.min.Y, bb.max.Y), (bb.min.Z, bb.max.Z))[axis_i]
     # The leader tip is the recogniser's own floor_face_anchor (#704), so a declared
     # groove anchors identically to a detected one by construction.
-    from draftwright.recognition import floor_face_anchor
+    from b123d_recognisers import floor_face_anchor
 
     c = floor_face_anchor(face)
     return (
@@ -874,7 +873,7 @@ def _read_step_levels(
     mirroring ``model/detect.py``), the ``datum`` (bbox min corner the positions measure from)
     and ``at`` (the frame anchor — bbox centre X/Y at ``base``, matching ``detect.py`` so an
     object round-trips to the same IR)."""
-    from draftwright.recognition import (
+    from b123d_recognisers import (
         project_step_shoulders,
         recognise_risers,
         step_level_records,