Skip to content

docs: propagate fixes from develop across stats, blas, and plot siblings#12267

Merged
kgryte merged 4 commits into
developfrom
philipp/fix-propagation-2026-05-24
May 24, 2026
Merged

docs: propagate fixes from develop across stats, blas, and plot siblings#12267
kgryte merged 4 commits into
developfrom
philipp/fix-propagation-2026-05-24

Conversation

@Planeshifter
Copy link
Copy Markdown
Member

Description

Propagating fixes merged to develop between f8a43f95 (2026-05-24 01:23 -0700) and 40dad68e (2026-05-24 02:54 -0700) to sibling packages with the same underlying defect.

Pattern: propagate parameter-name fixes to umbrella namespace declarations

Commit 47f803ef corrected a stale scale parameter reference (sc) in the entropy JSDoc and per-function TypeScript declaration for @stdlib/stats/base/dists/levy/entropy. The umbrella namespace docs/types/index.d.ts carries a duplicate of that description and requires the same correction at line 73.

Pattern: propagate missing-verb fix in BLAS rank-1 update test descriptions

Test description strings in gger and sger read 'the function the rank 1 operation ...', omitting performs — the same defect fixed in dger. Corrects 16 sites across 6 test files.

Pattern: replace new Array allocations with zeros in SVG plot component onRender re-emit callbacks

Nine sibling SVG plot components share the same onRender re-emit pattern as plot/components/svg/defs, each allocating an argument array via new Array( arguments.length+1 ). Replace each allocation with zeros( arguments.length+1, 'generic' ) and add the corresponding @stdlib/array/zeros require.

  • 40dad68e (chore: fix JavaScript lint errors)
    • @stdlib/plot/components/svg/annotations
    • @stdlib/plot/components/svg/axis
    • @stdlib/plot/components/svg/background
    • @stdlib/plot/components/svg/canvas
    • @stdlib/plot/components/svg/graph
    • @stdlib/plot/components/svg/path
    • @stdlib/plot/components/svg/rug
    • @stdlib/plot/components/svg/symbols
    • @stdlib/plot/components/svg/title

Related Issues

None.

Questions

No.

Other

Validation

Reviewed all 12 commits merged to develop in the 24-hour window. Three source patterns advanced to the patch stage; per-pattern search scope and validation procedure:

  • Search scope: scoped to the source commit's namespace first (stats/base/dists/levy/* for the scale parameter fix; blas/base/* for the test-description grammar fix; plot/components/svg/* for the new Array allocation pattern). Originating packages already fixed by the source commits were excluded.
  • Independent validation: two opus validation agents independently reviewed every candidate site by reading each file in full; both returned confirmed for all 26 sites.
  • An adaptation pass confirmed each @stdlib/array/zeros require insertion point for the nine SVG plot components (placement determined per-file: between inherit and instanceOf where present, otherwise after the last @stdlib/utils/* or @stdlib/assert/* require and before the first local require, mirroring the source commit's placement convention).
  • A sonnet style-consistency pass confirmed JSDoc indentation, single-quoted string form, and require-block ordering match the source-commit canonical form.

Deliberately excluded:

  • 91a6a85e (multi-offset README wording): the source commit was comprehensive — no remaining multi-offset README sites carry the old singular phrasing.
  • f345dd63 (incorrect logEach argument in cgemv example): the defect (logging x where y is the output vector) has no grep-able textual signature distinguishing wrong from correct call sites; would require per-site semantic interpretation.
  • c17ecc0d (separate @example tags in time/day-of-quarter): no remaining time/* packages exhibit the malformed-@example continuation pattern.
  • a2fcfdb2 (ndarray-instance doctest improvement): single-package documentation improvement without a generalizable propagation signature.
  • ac45b111 (refactor: main require alias in stats/base/dists/frechet): refactor commit, not a defect fix; out of scope for this routine.

Checklist

AI Assistance

  • Yes
  • No

If you answered "yes" above, how did you use AI assistance?

  • Code generation (e.g., when writing an implementation or fixing a bug)
  • Test/benchmark generation
  • Documentation (including examples)
  • Research and understanding

Disclosure

This PR was authored by Claude Code on behalf of @Planeshifter as an automated propagation of fixes merged to develop over 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 (two opus validation passes plus a sonnet style-consistency pass) before commits were applied in the primary worktree. A human will audit and promote the PR out of draft.


@stdlib-js/reviewers


Generated by Claude Code

claude added 3 commits May 24, 2026 13:21
Propagates fix from 47f803e ("docs: fix scale parameter reference in
`@stdlib/stats/base/dists/levy/entropy`") to the umbrella declarations
file at `levy/docs/types/index.d.ts`, where the entropy description
still referenced scale `s` instead of the actual `c` parameter.
Propagates fix from 7800c9a ("test: fix grammar of test descriptions in
`blas/base/dger`") to the sibling rank-1 update packages, where the
`tape()` descriptions were missing the verb "performs" between
"function" and "the rank 1 operation".
Propagates fix from 40dad68 ("chore: fix JavaScript lint errors") to
sibling SVG plot components that exhibit the same `new Array( N )`
allocation pattern in their `onRender` re-emit callback, replacing it
with `zeros( N, 'generic' )` to match the lint convention.
Comment thread lib/node_modules/@stdlib/blas/base/sger/test/test.sger.native.js Outdated
Comment thread lib/node_modules/@stdlib/blas/base/sger/test/test.sger.js Outdated
Comment thread lib/node_modules/@stdlib/blas/base/sger/test/test.ndarray.native.js Outdated
Comment thread lib/node_modules/@stdlib/blas/base/gger/test/test.ndarray.js Outdated
Comment thread lib/node_modules/@stdlib/blas/base/sger/test/test.ndarray.js Outdated
Comment thread lib/node_modules/@stdlib/blas/base/sger/test/test.ndarray.js
Comment thread lib/node_modules/@stdlib/blas/base/gger/test/test.main.js Outdated
Co-authored-by: Athan <kgryte@gmail.com>
Signed-off-by: Athan <kgryte@gmail.com>
@kgryte kgryte marked this pull request as ready for review May 24, 2026 19:37
@kgryte kgryte requested a review from a team May 24, 2026 19:37
@stdlib-bot stdlib-bot added the Needs Review A pull request which needs code review. label May 24, 2026
@kgryte
Copy link
Copy Markdown
Member

kgryte commented May 24, 2026

As the CI failure is not due to changes introduced in this PR, I'll go ahead and merge.

@kgryte kgryte added Documentation Improvements, additions, or changes to documentation. and removed Needs Review A pull request which needs code review. labels May 24, 2026
@kgryte kgryte merged commit 3fe5568 into develop May 24, 2026
48 of 49 checks passed
@kgryte kgryte deleted the philipp/fix-propagation-2026-05-24 branch May 24, 2026 19:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Documentation Improvements, additions, or changes to documentation.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants