Repository navigation
docs: fix headings - #15454
docs: fix headings#15454
Conversation
The package heading and usage signature both read `gfillGreaterThanEqualEqual`, with a doubled `Equal`. Every other reference in the package -- the require line directly above the signature, `docs/repl.txt`, `docs/types/index.d.ts`, `lib/main.js`, and `package.json` -- uses `gfillGreaterThanEqual`. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01F5HbkjKMwKyrN7mThirjgQ
The namespace registered `dvander`, `gfillGreaterThanEqual`, `gleftPadCircular`, `gleftPadEdge`, `grightPadCircular`, `grightPadEdge`, `gvander`, and `gwhere` on `lib/index.js` without adding the corresponding entries to the namespace declaration, so the eight functions resolved at runtime but not from TypeScript. The other 223 members each carry an import and an interface member; add the same for these eight. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01F5HbkjKMwKyrN7mThirjgQ
Coverage Report
The above coverage report was generated for the changes in this PR. |
blas/ext/base/ndarray|
It is a job timeout, not a benchmark failure: Every benchmark that reported before the cap passed. The ones executing when it died are Same check, same timeout, on No fix exists on Not re-running the job. The failure is a deterministic wall-clock cap, not flake, and it has already reproduced on the base branch — a second hour of runner time would tell us nothing. The real fix belongs in its own PR against Generated by Claude Code |
|
Not clear why so many benchmarks ran for this PR. But, as the CI failure is not due to changes in this PR, I'll go ahead and merge. |
The run was triggered by if [ -n "${{ github.event.pull_request.number }}" ]; then
ancestor_commit=$(git merge-base base.sha head.sha)
files=$(git diff --diff-filter=AM --name-only $ancestor_commit head.sha)
else
files=$(git diff --diff-filter=AM --name-only ${{ github.event.before }} ${{ github.event.after }})
fiOn a push there is no Which is how The Zero benchmark directories — the workflow's So this is general: any Generated by Claude Code |
Follow-up fixes for commits merged to
developbetween 2026-09-22 12:31:02 -0700 and 2026-09-23 04:54:18 -0700.No related issue.
Description
This pull request:
fc9905c8: fix doubledEqualsuffix inlib/node_modules/@stdlib/blas/ext/base/ndarray/gfill-greater-than-equal/README.md— H1 (L21) and usage signature (L39) readgfillGreaterThanEqualEqual; corrected togfillGreaterThanEqualto matchlib/main.jsand the rest of the docs.ec5d60d6,cf2b5045, andf158b2ffeach added new members toblas/ext/base/ndarray's runtime namespace viasetReadOnlyinlib/index.jswithout updatinglib/node_modules/@stdlib/blas/ext/base/ndarray/docs/types/index.d.ts; runtime had 231 members, declarations only 223, sodvander,gfillGreaterThanEqual,gleftPadCircular,gleftPadEdge,grightPadCircular,grightPadEdge,gvander, andgwhereall failTS2339for TS consumers despite existing at runtime. Added the missing imports and<name>: typeof <name>;interface members (with JSDoc/@exampleadapted from each package's own.d.ts) at their alphabetical positions; counts now match at 231/231.Commits are grouped one per package directory:
blas/ext/base/ndarray/gfill-greater-than-equal(docs:) andblas/ext/base/ndarray(fix:).Related Issues
None.
Questions
No.
Other
Review scope. 37 first-parent commits, 196 files,
4e396e96^..f158b2ff. Themes: eight newblas/ext/base/ndarraypackages (gvander,dvander,gwhere,gfill-greater-than-equal,g{left,right}-pad-{edge,circular}) plus their namespace registrations; a float16 dtype rollout acrossarray/base/assert/*andndarray/base/{full,full-by,nans,nans-like}; nine ULP-assertion test migrations; a new C backend forstats/base/ndarray/dnanmeanpw; and lint/tooling work (ESLint flat-config for Markdown code blocks, new_tools/repl-txt/rules/no-redeclare).Validation. Checked for stdlib style-guide compliance (each new package diffed structurally against an established sibling —
blas/ext/base/ndarray/gfill-equalfor the ndarray wrappers,stats/base/ndarray/{dmeanpw,dnanmeanors}for the C addon) and scanned for bugs across the full window diff. Verified independently before committing: the namespace member counts (223 declared vs. 231 registered, the eight missing being exactly those added in this window), and the modified declaration file parsing clean undertsc --noEmitwith no new error classes relative to itsdevelopbaseline.Deliberately excluded. Anything requiring interpretation: subjective suggestions, style preferences not mandated by
docs/style-guides, and issues whose fix would reach outside the window's diff. Three candidates were dropped on that basis —8a518c59migrated onlyellipj/test/test.main.jsto ULP assertions, leavingtest.sncndn.js,test.assign.js, andtest.native.json the oldabs/EPSpattern; none of those three files is in the window's diff, and completing the migration would require deriving new ULP bounds, so it belongs in its own PR. Also dropped:gfill( len, 1.0, xbuf, 3 )ingfill-greater-than-equal/benchmark/benchmark.js, which indexes past thelen-elementxbufand grows the generic array to3*len-2. The measured work is stilllenelements becauseend = lenclips the range, so results are unaffected, and the pre-existing non-ndarrayblas/ext/base/gfill-greater-than-equalbenchmark uses the same pattern — changing only the new package would split the family. Flagging it here rather than fixing it unilaterally.The deliberate numeric reorderings in
fed47c5a(stats/base/dists/hypergeometric/mean,stats/base/dists/chi/skewness), the recalibrated ULP bounds, and thenew Array(W)→zeros(W)change instats/incr/mvmrwere all reviewed and left alone.Checklist
AI Assistance
If you answered "yes" above, how did you use AI assistance?
Disclosure
This PR was written primarily by Claude Code, as part of an automated review of the last 24 hours of commits to
develop. Reviewer agents surfaced the candidate issues; each finding was then re-verified against the tree before any edit, and the two findings that survived filtering are the ones applied here. Opened as a draft for maintainer audit.🤖 Generated with Claude Code
https://claude.ai/code/session_01F5HbkjKMwKyrN7mThirjgQ
Generated by Claude Code