Skip to content
This repository was archived by the owner on Aug 20, 2026. It is now read-only.

chore(deps): bump teeny submodule to main (dispatch_values, scan_, subsample, slice_along) - #54

Merged
balbasty merged 1 commit into
teenyfrom
claude/bump-teeny-pin-main
Aug 1, 2026
Merged

chore(deps): bump teeny submodule to main (dispatch_values, scan_, subsample, slice_along)#54
balbasty merged 1 commit into
teenyfrom
claude/bump-teeny-pin-main

Conversation

@balbasty

Copy link
Copy Markdown
Collaborator

Agent:claude-fastfields-to-teeny

Closesfastfields/fastfields-lib#106. Prerequisite for #50 (the regulariser tap-table engine). Part of the teeny-refactor umbrella (fastfields-lib#21).

What changed

Two lines:

  • external/teeny: dcd591c5c46bd4 (teeny main tip, +125 commits).
  • .gitmodules: branch = claude/fastfields-teeny-refactor-js42idbranch = main.

The old pin was the tip of teeny's stale claude/fastfields-teeny-refactor-js42id
branch, which has not moved since the refactor began — git ls-remote confirms it
still points at exactly dcd591c. main is what this project tracks now, so
.gitmodules should say so.

What it brings in

Verified against teeny's actual git log dcd591c..main, not a summary:

landingteeny PR
dispatch_values — product-form runtime→static dispatch#465
forward+backward scan_ sweep idiom docs#466
scan_/scan sequential fold along one axis#254
unfold<Axis>(size, step) sliding/strided window view#256
subsample<Axes...>(k, starts...) coloured-lattice sugar#258
peel_zip<Axes...>(a,b[,c]) lock-step iteration#327
index_select<Axis>(idx) gather#326
in-place maximum_/minimum_#325
sqdist/dist wrappers#324
host-side atomic fetch_add_/sub_#257
static-unrolled dot/sqdist fast path#255

Plus a large amount of unrelated hardening (signedness-aware offset decode in
dot/sqdist/peel_zip, the -DTNY_NO_NEGATIVE_INDEX CI variant, the
out-of-place and in-place __restrict__ linear fast paths).

Breaking changes and downstream impact

Exactly two commits in the range are marked breaking:

  • 3c4ec01 refactor!: rename take_along to slice_along (teeny#423)
  • 8458067 refactor(iterate)!: move scan_/scan's axis<A>{} tag to trailing (teeny#348)

The scan_ tag move cannot break us: scan_ did not exist at dcd591c, so there
are no call sites to break.

For the rename, I grepped every fastfields repo (fastfields-kernels,
-cpu-impl, -cpu-lib, -cuda-impl, -cuda-lib, -lib, and the Python repos)
for take_along:

$ grep -rn "take_along" fastfields-* --include=*.h --include=*.hpp \
--include=*.cpp --include=*.cu --include=*.md
(no matches)

Zero call sites. The only hits anywhere for the #254–259 family of names are
two prose mentions of scan_/subsample/index_select in
TEENIFICATION_REVIEW.md (the document that requested those features), which
are descriptive text, not code. So no downstream fixups ride along in this PR.

Verification

make -C fastfields-cpu-lib clean test, full rebuild, both compilers, before and
after the bump. Check counts are byte-identical across all four runs:

suitebefore (clang++/g++)after (clang++/g++)
test_distance23522352
test_distance_mesh46224622
test_distance_spline704704
test_posdef50925092
test_pushpull326326
test_reg_field465465
test_reg_flow854854
test_reg_op186186
test_resize41304130
test_restrict191191
test_splinc45774577

failures: 0 / PASSED on every line, exit=0 on every run. 23,499 checks total,
no coverage gained or lost — which is the expected shape for a pin bump that adds
new API surface without changing any API we already call.

CUDA is compile/link-only in this environment (no nvcc here, no GPU in CI), so the
cuda-impl/cuda-lib path is unexercised by this PR beyond the fact that it
touches no CUDA source.


Generated by Claude Code

The pin sat at dcd591c, the tip of teeny's stale
`claude/fastfields-teeny-refactor-js42id` integration branch, which is
125 commits behind teeny's `main`. That is now the branch this project
tracks, so point `.gitmodules` at `main` too rather than at a branch that
has not moved since the refactor started.
Brings in, among 125 commits:
- dispatch_values, product-form runtime->static dispatch (teeny#465)
- the forward+backward scan_ sweep idiom docs (teeny#466)
- scan_/scan (teeny#254), unfold (#256), subsample (#258), peel_zip,
index_select, in-place maximum_/minimum_, sqdist/dist, host-side
atomic fetch_add_/sub_ (#257)
- the breaking rename take_along -> slice_along (teeny#423)
The rename is the only breaking change that could reach us; a tree-wide
grep for `take_along` across all fastfields repos finds no call sites
(only two prose mentions of unrelated names in TEENIFICATION_REVIEW.md),
so no downstream fixups are needed.
Verified: `make -C fastfields-cpu-lib clean test` with CXX=clang++ and
CXX=g++, before and after the bump. All 11 suites PASS with byte-identical
check counts (2352/4622/704/5092/326/465/854/186/4130/191/4577).
@claude

claudeBot commented Aug 1, 2026

Copy link
Copy Markdown

Agent:claude-fastfields-to-teeny

About the red ❌ on test-via-cpu-lib

That check is not reporting on this diff. The workflow hardcodes ref: main when it checks out the cpu-lib harness, then pins only the nested kernels submodule to the commit under test — so any PR based on the teeny branch is built by the pre-teeny-refactor cpu-lib and cpu-impl, which cannot compile teeny-branch kernels at all.

Running the workflow's own recipe against teeny's tip e579add (which predates this PR) fails identically:

./impl/kernels/distance/spline.h:3:10: fatal error: 'teeny/teeny.h' file not found

Filed as fastfields/fastfields-lib#104, deliberately not fixed here (unrelated concern, own branch).

The gate that was actually run

make -C fastfields-cpu-lib clean test against the dev-tree checkout, on both compilers, before and after:

test_distance 2352 | test_distance_mesh 4622 | test_distance_spline 704
test_posdef 5092 | test_pushpull 326 | test_reg_field 465
test_reg_flow 854 | test_reg_op 186 | test_resize 4130
test_restrict 191 | test_splinc 4577

failures: 0, PASSED on every line, exit=0 — 23,499 checks, identical counts to the pre-change baseline.


Generated by Claude Code

@balbasty
balbasty merged commit dce4bd6 into teenyAug 1, 2026
1 check failed
@balbasty
balbasty deleted the claude/bump-teeny-pin-main branch August 1, 2026 13:41
Sign up for freeto subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

@balbasty@claude