Repository navigation
handover: mark the B2 items already measured, correct two assumptions about the workstation - #498
Merged
Merged
Conversation
…ions about this machine The B2 block asked the local agent to discover seven things about the Windows CPU variants. Four of them were measured on that same workstation on 2026-10-09, so each item now says MEASURED or OPEN with the numbers, and the header says so too. Verifying a MEASURED line is still worth the minute it costs -- a different result there is a finding -- but the time belongs to the open ones. Measured, now stated: plain clang 23.1.3 builds all 14 variants with 0 errors; the directory holds 18 DLLs with nothing left in a Release/ subdirectory and jllama-files.txt naming 15 of them; verify-native-deps.py reports 18 libraries and 0 violations. Two assumptions in the draft do not hold on this machine, and both would have cost the local agent time: PATH carries clang 23.1.3, not 20.1.8, and Visual Studio ships no clang here at all (neither 2022\Community nor 18\BuildTools has VC\Tools\Llvm\x64) -- so the "newer versus older clang" choice the prompt set up does not exist; the 20.1.8 of the earlier comparison was unpacked separately. And jllama-extras.txt is ABSENT locally by design: CMake never writes it, merge-native-artifacts.sh derives it in CI from everything jllama-files.txt does not name. Asking "whether it exists" invites a false defect report, so the item now says what its absence means. Two things added that the measurements make useful rather than obvious. The Java suite is marked as the item that decides whether the variant path works at all, with the reason it is open: the measuring session's `mvn compile` dies in an Error Prone / NullAway crash on args/package-info.java, unrelated to the native work, and disabling annotation processing then breaks Lombok. And the variant-selection step carries a caveat before the agent spends time on it -- CLAUDE.md records that ggml logs module decisions at GGML_LOG_DEBUG only, so `-lv 4` may not name the chosen module; listing the DLLs the JVM has mapped is the fallback. The throughput item gets the static-build figures (pp512 391.6, tg128 82.3 t/s with clang and GGML_OPENMP=OFF) as the line a variant build should land near, so the number means something when it arrives. Committed on a branch off main rather than onto claude/hopeful-pascal-9jlbqb: that branch carries an unrelated open PR (#497, BoringSSL), and this file already lives on main, so it reaches the session branch on its next merge without mixing concerns into someone else's review. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01E2h8gXyyE5UeimkL9vQv9G
bernardladenthin
had a problem deploying
to
startgate
October 9, 2026 22:02 — with
GitHub Actions
Error
bernardladenthin
had a problem deploying
to
maven-central
October 9, 2026 22:02 — with
GitHub Actions
Failure
bernardladenthin
had a problem deploying
to
maven-central
October 9, 2026 22:02 — with
GitHub Actions
Failure
|
5 of 6 tasks
This branch had an error being deployed
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.



docs/handover/local-agent-pr-a.mdasked the local agent to discover seven things about the WindowsCPU variants. Four were already measured on that same workstation on 2026-10-09, so each item
now says
MEASUREDorOPENwith the numbers.Verifying a
MEASUREDline is still worth the minute it costs — a different result there is afinding — but the time belongs to the open ones.
Already measured, now stated
Release/,jllama-files.txtnaming 15verify-native-deps.pyTwo assumptions that do not hold on this machine
Both would have cost the local agent time:
PATH" —PATHcarries 23.1.3, and Visual Studio shipsno clang here at all (neither
2022\Communitynor18\BuildToolshasVC\Tools\Llvm\x64). Sothe "newer versus older clang" choice it set up does not exist; the 20.1.8 of the earlier
comparison was unpacked separately. 23.1.3 is the stricter one, and
patches/0017is what makesquants.ccompile there.jllama-extras.txtexists" — it is absent locally by design. CMake neverwrites it;
merge-native-artifacts.shderives it in CI from everythingjllama-files.txtdoes notname, which on Windows is exactly
ggml.dll+ggml-base.dll(no$ORIGINthere, so they must bepre-loaded by full path). Asking neutrally invites a false defect report, so the item now says what
the absence means.
Two additions the measurements make useful
The Java suite is marked as the item that decides whether the variant path works at all, with
the reason it is open: the measuring session's
mvn compiledies in an Error Prone / NullAway crashon
args/package-info.java— unrelated to the native work, and disabling annotation processing thenbreaks Lombok, so there was no quick bypass. CLAUDE.md's warning is quoted at it: a successful load
proves little, since in the earlier Windows measurement every crash came after a load that
reported the right device count.
The variant-selection step gets a caveat before the agent spends time on it: CLAUDE.md records
that ggml logs module decisions at
GGML_LOG_DEBUGonly, so-lv 4may not name the chosen module;listing the DLLs the JVM actually has mapped is the fallback. And the throughput item gets the
static-build figures (pp512 391.6, tg128 82.3 t/s, clang with
GGML_OPENMP=OFF) as the line avariant build should land near — so the number means something when it arrives.
Why this branch
Committed off
mainrather than ontoclaude/hopeful-pascal-9jlbqb: that branch carries anunrelated open PR (#497, BoringSSL), and this file already lives on
main, so the correction reachesthe session branch on its next merge without mixing concerns into someone else's review.
Verified
check-shared-files.pyclean (the file is not shared), buildcheck suite 113 greendocs/handover/**is already covered inREUSE.toml, so no licence header is needed🤖 Generated with Claude Code
https://claude.ai/code/session_01E2h8gXyyE5UeimkL9vQv9G