Skip to content

Commit 39030f0

Browse files
awesomebytesclaude
andcommitted
Docs tone: remove person references from PLAN.md and ARCHITECTURE_V2.md
Project docs speak in neutral project tone; technical content unchanged. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
1 parent b85776e commit 39030f0

2 files changed

Lines changed: 9 additions & 9 deletions

File tree

PLAN.md

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -313,7 +313,7 @@ mechanically, not by discipline.
313313
callable) — documented + round-trip tested. bt_kit refactored onto it.
314314
test-bt 31 / test-bt-frozen 29, all green; frozen path unregressed.
315315

316-
- **Warmup + first-use JIT (2026-07-11, fcee3ec/1cdd216/2459642):** Sam's
316+
- **Warmup + first-use JIT (2026-07-11, fcee3ec/1cdd216/2459642):**
317317
FREEZE.md feedback implemented. `cppyy_kit.first_use()` instruments kit-owned
318318
entry points — one-time, suppressible (RCLCPPYY_JIT_NOTICE=0), LLM-actionable
319319
notice naming the exact warmup() to call; `bt_kit.warmup()` /
@@ -400,7 +400,7 @@ mechanically, not by discipline.
400400
Nav2 plugin injection would reuse this pattern verbatim where an
401401
add/inject API exists. Docs: `docs/control_kit/`.
402402

403-
- **CUDA OpenCV (2026-07-11, 74ba1a1, merged):** Sam's skepticism vindicated
403+
- **CUDA OpenCV (2026-07-11, 74ba1a1, merged):** Initial no-CUDA assumption was wrong
404404
conda-forge genuinely ships none (feedstock sets WITH_CUDA=0; 0/6357 files
405405
with cuda build strings; declined in issues #74/#109), **but Esri's public
406406
channel ships `libopencv 4.13.0 cuda129` (Apache-2.0)** — the exact version
@@ -438,8 +438,8 @@ mechanically, not by discipline.
438438
quick-run lines. **Kit scoreboard: 7 spikes, 7 GOs** (bt, pcl, ompl, nav2,
439439
moveit, control, vision capstone), each with honest boundaries.
440440

441-
- **TF via rclcppyy (2026-07-11, GO; 28854f7, merged):** Sam's assumption
442-
confirmed with source-cited evidence — stock Python tf2_ros feeds the buffer
441+
- **TF via rclcppyy (2026-07-11, GO; 28854f7, merged):** The tf2 Python-overhead
442+
hypothesis confirmed with source-cited evidence — stock Python tf2_ros feeds the buffer
443443
with a Python for-loop per transform (fully-deserialized TFMessage, GIL
444444
held), while C++ TransformListener ingests natively on its own thread.
445445
Measured: ingest CPU 6.7×–14× lower (grows with tf rate: 19.3% → 1.4% at
@@ -475,7 +475,7 @@ mechanically, not by discipline.
475475
bench parity (~963 Hz, 0 dropped — indistinguishable). Recipe 0.2.0 with
476476
real conda deps validated locally. Dev/CI wiring: PYTHONPATH bridge to a
477477
cppyy_kit checkout (a committed file:// channel would break CI portability
478-
— right call). RELEASING.md documents the choreography: (1) Sam authorizes
478+
— right call). RELEASING.md documents the choreography: (1) authorize
479479
prefix.dev for awesomebytes/cppyy_kit, (2) tag suite v0.1.0, (3) swap
480480
bridge → awesomebytes channel here, (4) tag v0.2.0. The kit-suite work now
481481
lives in github.com/awesomebytes/cppyy_kit (docs:

docs/kits/ARCHITECTURE_V2.md

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,7 @@
77
> rclcppyy is now the drop-in accelerator product on top of it (see the repo README
88
> and `RELEASING.md`).
99
10-
**Status: APPROVED DIRECTION (2026-07-11)**Sam approved the architecture with
10+
**Status: APPROVED DIRECTION (2026-07-11)**The architecture was approved with
1111
two refinements, captured in §4: the ROS core becomes **`rclcpp_kit`** (obeying
1212
the same naming rule as every other kit), **`rclcppyy` stays a standalone
1313
product** that depends on it, and the kit suite moves to a **new repo**. §§1–3
@@ -26,7 +26,7 @@ below are the original evaluation; §4 is the operative plan.
2626
| **`@cpp` decorator** (§5.6) | **Adopt** | Natural unification of what we already half-built: `callback()`'s type-hint signature inference + the marshaling patterns (numpy→pointer+size is our §6). Body-never-executed with annotation-driven marshaling is clean, and it composes with the compile cache (each `@cpp` block = one cached `.so`). |
2727
| **`.pyi` stub generation** (§5.1) | **Adopt — revive** | Was on rclcppyy's ORIGINAL roadmap (`scripts/create_stubs.py`, stalled WIP). Belongs at the cppyy_kit level; directly serves the LLM-ergonomics principle (typed corridors help agents too). |
2828
| **async/nogil helpers** (§2.2) | **Adopt, corrected** | The doc's premise ("cppyy calls release the GIL automatically") is **factually wrong** — control_kit *measured* that cppyy holds the GIL on blocking C++ calls (COMMON_PATTERNS §13). But the goal is right: a `nogil()` wrapper (C++-side GIL release shim) + an asyncio `run_in_executor` integration would make blocking C++ calls event-loop-safe. Build it on our corrected foundation. |
29-
| **Layered packaging: base + kit packages** (§5.1/§5.8) | **Adopt — the re-architecture** | Matches Sam's instinct and our reality: cppyy_kit is already ROS-free by content; six kits already follow "depends on the base". See §3. |
29+
| **Layered packaging: base + kit packages** (§5.1/§5.8) | **Adopt — the re-architecture** | Matches the project's design direction and practical reality: cppyy_kit is already ROS-free by content; six kits already follow "depends on the base". See §3. |
3030
| **Fallback contract + `status()` introspection** (§5.2) | **Adopt the pattern, not the target** | We already do capability-detection-with-fallback ad hoc (CUDA auto-detect, frozen-path fallback, gtsam binding fallback). Codify it once in cppyy_kit (capability probe + fallback + introspectable status) so kits stop reinventing it. |
3131

3232
### 1.2 Doesn't make sense for us (skip, with reasons)
@@ -201,12 +201,12 @@ Dependency graph:
201201
ways; `cppyy-kit` → conda-forge submission when stable; community-kit
202202
authoring guide (COMMON_PATTERNS as the manual).
203203

204-
Open decisions for Sam: (a) confirm `cppyy_kit` vs `kitforge` as the project
204+
**Resolved decisions:** (a) `cppyy_kit` confirmed as the project
205205
name; (b) distro-scoping convention `ros-jazzy-<kit>-kit` — recommended for
206206
resolver hygiene alongside robostack; (c) whether the roscon_uk_2025 archive
207207
stays in rclcppyy (recommended — it's product history, not suite material).
208208

209-
### 4.4 Kit anatomy (correction, Sam 2026-07-11: kits are a MIX, not pure Python)
209+
### 4.4 Kit anatomy (correction, 2026-07-11: kits are a MIX, not pure Python)
210210

211211
§3's "pure-Python packages" was packaging shorthand (no ament/colcon needed) and
212212
undersells what a kit is. The kit anatomy:

0 commit comments

Comments
 (0)