docs: update examples - #14169
Conversation
Propagates fixes from 362ed23 ("chore: clean-up") to sibling packages carrying the same defects: adds missing `f` suffixes to single-precision floating-point literals in the README C code blocks of `blas/base/ssyr` and `blas/ext/base/swxsy`, and fixes JSDoc example indentation in the TypeScript declarations of eighteen `blas/ext/base/ndarray` packages. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01MBunG53uJwaC8GwkyTUAyn
|
CI red on Run affected tests: pre-existing failure on the base branch, not introduced by this PR (the diff here is documentation-only — README C code blocks and Root cause: Fixing the JS/native validation divergence is a runtime change outside this propagation PR's scope; it needs its own fix (align the Generated by Claude Code |
Coverage ReportNo coverage information available. |
develop fixes to sibling packages (2026-08-11)
Description
Propagating fixes merged to
developbetween 2026-08-10 13:50 PDT (44cb452) and 2026-08-10 18:02 PDT (e45106f) to sibling packages carrying the same defects.Pattern: missing
fsuffix on single-precision float literals in C examplesfloat-typed C example code needsf-suffixed literals; unsuffixed literals aredoubleand trigger implicit-narrowing warnings under strict compilers. Source commit fixed this inblas/ext/base/ccopy-within's README andexamples/c/example.c; the same defect was propagated to two further packages, including one spot where thealphaargument diverged from that package's own already-correct example file.362ed235(chore: clean-up)blas/base/ssyr(README Usage and Examples C blocks: 4 array-initializer lines, 3const float alphaarguments)blas/ext/base/swxsy(README Usage and Examples C blocks: 9 array-initializer lines)Pattern:
@exampleobject-literal indentation in.d.tsdocsObject-literal members inside
@exampleblocks must carry a 5-space indent after the leading*(1-space JSDoc gutter + 4-space code indent), not 4.362ed235fixed this in fourblas/ext/base/ndarray/*triupackages but missed the siblingdtriupackage and seventeen otherblas/ext/base/ndarraypackages sharing the samescalar2ndarray( 0, { … })example pattern — 27 lines total. Inslinspace,glinspace, anddlinspace, the third object literal in the same example is already correctly indented, leaving those three files internally inconsistent.362ed235(chore: clean-up)blas/ext/base/ndarray/dtriu(missed by source commit; 1 line)blas/ext/base/ndarray/scopy-within(4 lines)blas/ext/base/ndarray/dcopy-within(4 lines)blas/ext/base/ndarray/slinspace(2 lines)blas/ext/base/ndarray/glinspace(2 lines)blas/ext/base/ndarray/dlinspace(2 lines)blas/ext/base/ndarray/ssorthp(1 line)blas/ext/base/ndarray/ssort(1 line)blas/ext/base/ndarray/scircshift(1 line)blas/ext/base/ndarray/gsorthp(1 line)blas/ext/base/ndarray/gsort(1 line)blas/ext/base/ndarray/gcopy-within(1 line)blas/ext/base/ndarray/gcircshift(1 line)blas/ext/base/ndarray/dsortsh(1 line)blas/ext/base/ndarray/dsortins(1 line)blas/ext/base/ndarray/dsorthp(1 line)blas/ext/base/ndarray/dsort(1 line)blas/ext/base/ndarray/dcircshift(1 line)Related Issues
No.
Questions
No.
Other
Validation
float <name>[] = { … }initializers containing unsuffixed literals in README ```c blocks andexamples/c/*.c(168 lines in 63 files), scoped for this run to the source commit's `blas/` namespace. For the indentation pattern: repo-wide `rg` for asterisk-plus-4-spaces-plus-quote in `docs/types/index.d.ts` JSDoc, scoped to `blas/ext/base/ndarray/` siblings.ccopy-within,ctriu, siblingssyr2/ssprREADMEs, and each package's ownexamples/c/example.c. All 43 lines were confirmed by all agents; zero sites were rejected or flaggedneeds-human.copyWithinbenchmark fixes (e805b48,35f9d71,3fdf0b9,075283f) fixed all affected files at source; themymgf→mgfexample fix (fc38da1) has no analog anywhere (script-verified across all@example/README/repl.txtblocks); the VLA →mallocand Pareto (Type I) parameter-description fixes (ccc3c46) have no remaining sites.Deliberately excluded:
blas/ext/base/{ssort2hp,ssort2sh,ssortins,ssortsh}(same float-literal defect): open refactor PRs feat: add C ndarray API and refactorblas/ext/base/ssort2hp#5102, feat: add C ndarray API and refactorblas/ext/base/ssort2sh#5118, feat: add C ndarray API and refactorblas/ext/base/ssortins#5161, feat: add C ndarray API and refactorblas/ext/base/ssortsh#5165, refactor: updateblas/ext/base/ssortshto follow project conventions #5177, refactor: updateblas/ext/base/ssortinsto follow project conventions #5215 already rewrite these packages.blas/ext/base/ndarray/docs/types/index.d.ts(31 mis-indented lines): open PR feat: updateblas/ext/base/ndarrayTypeScript declarations #14158 touches that exact file; re-check after it lands.math/strided/special,stats/base, andstrided/base— outside the source commit's namespace; thestats/basehits embed0.0/0.0NaN idioms needing a judgment call. Logged for a follow-up round.lib/{index,main}.jsJSDoc (includingctriu/gtriu/striu/ztriuthemselves) plusndarray/{broadcast-scalar-like,from-scalar-like,find,find-last}andstring/{first,last}.d.tsfiles — same defect class, but outside the source commit's file-type and namespace shape. Logged for a follow-up round.blas/ext/base/swxsyREADME documentsX/Y/Wasdouble*though the C signature isfloat*(lines 225-229, 255-261) — a different defect class with no source commit in this window; flagged for a manual fix.Checklist
AI Assistance
If you answered "yes" above, how did you use AI assistance?
Disclosure
This PR was authored by Claude Code on behalf of @Planeshifter as an automated propagation of fixes merged to
developover the prior 24 hours. Candidate source commits were filtered for generalizable patterns, sibling sites located via grep-able pattern signatures, and each proposed patch independently validated by parallel reviewer agents before commits were applied. A maintainer should audit and promote out of draft.🤖 Generated with Claude Code
https://claude.ai/code/session_01MBunG53uJwaC8GwkyTUAyn
Generated by Claude Code