docs: update descriptions#13481
Merged
Merged
Conversation
Append the "using alternative indexing semantics" qualifier to the lib/ndarray.js JSDoc summary so it matches the phrase already carried by the package's README, repl.txt, and TypeScript declarations for the ndarray API. Convention followed by 13 sibling packages within @stdlib/lapack/base.
Append the "using alternative indexing semantics" qualifier to the lib/ndarray.js JSDoc summary so it matches the phrase already carried by the package's README, repl.txt, and TypeScript declarations for the ndarray API. Convention followed by sibling package dlaset within the same-shape group.
Append the "and alternative indexing semantics" qualifier to the lib/ndarray.js JSDoc summary so it matches the phrase already carried by the package's README, repl.txt, and TypeScript declarations for the ndarray API. Convention followed by 13 sibling packages within @stdlib/lapack/base.
Append the "using alternative indexing semantics" qualifier to the lib/ndarray.js JSDoc summary so it matches the phrase already carried by the package's README, repl.txt, and TypeScript declarations for the ndarray API. Convention followed by 13 sibling packages within @stdlib/lapack/base.
Append the "using alternative indexing semantics" qualifier to the lib/ndarray.js JSDoc summary so it matches the phrase already carried by the package's README, repl.txt, and TypeScript declarations for the ndarray API. Convention followed by 13 sibling packages within @stdlib/lapack/base.
Append the "using alternative indexing semantics" qualifier to the lib/ndarray.js JSDoc summary so it matches the phrase already carried by the package's README, repl.txt, and TypeScript declarations for the ndarray API. Convention followed by 13 sibling packages within @stdlib/lapack/base.
Append the "using alternative indexing semantics" qualifier to the lib/ndarray.js JSDoc summary so it matches the phrase already carried by the package's README, repl.txt, and TypeScript declarations for the ndarray API. Convention followed by 13 sibling packages within @stdlib/lapack/base.
Append the "and alternative indexing semantics" qualifier to the lib/ndarray.js JSDoc summary so it matches the phrase already carried by the package's README, repl.txt, and TypeScript declarations for the ndarray API. Convention followed by 13 sibling packages within @stdlib/lapack/base.
Append the "and alternative indexing semantics" qualifier to the lib/ndarray.js JSDoc summary so it matches the phrase already carried by the package's README, repl.txt, and TypeScript declarations for the ndarray API. Convention followed by 13 sibling packages within @stdlib/lapack/base.
Append the "using alternative indexing semantics" qualifier to the lib/ndarray.js JSDoc summary so it matches the phrase already carried by the package's README, repl.txt, and TypeScript declarations for the ndarray API. Convention followed by sibling package clacgv within the same-shape group.
Append the "using alternative indexing semantics" qualifier to the lib/ndarray.js JSDoc summary so it matches the phrase already carried by the package's README, repl.txt, and TypeScript declarations for the ndarray API. Convention followed by sibling package dlaset within the same-shape group.
Append the "and alternative indexing semantics" qualifier to the lib/ndarray.js JSDoc summary so it matches the phrase already carried by the package's README, repl.txt, and TypeScript declarations for the ndarray API. Convention followed by 13 sibling packages within @stdlib/lapack/base.
Append the "using alternative indexing semantics" qualifier to the lib/ndarray.js JSDoc summary so it matches the phrase already carried by the package's README, repl.txt, and TypeScript declarations for the ndarray API. Convention followed by sibling package crot within the same-shape group.
Contributor
Coverage Report
The above coverage report was generated for the changes in this PR. |
kgryte
marked this pull request as ready for review
July 15, 2026 08:49
kgryte
approved these changes
Jul 15, 2026
lapack/base
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.
Description
This pull request:
lib/ndarray.jsJSDoc summary line in 13 packages under@stdlib/lapack/basewith the qualifier phrase ("using alternative indexing semantics" or "and alternative indexing semantics") that is already carried by each package's ownREADME.md,docs/repl.txt, anddocs/types/index.d.ts. In each case the ndarray JSDoc summary had been a copy of the strided (main) API summary, erasing the distinction that the other three doc surfaces already draw.Namespace summary
Analyzed 32 packages (excluding the
sharedcommon-types module) under@stdlib/lapack/base. Structural features (file tree,package.jsonshape, README section list,manifest.jsonshape, test/benchmark/example naming) and semantic features (public signatures, validation prologues, error construction, JSDoc shape, dependencies) were extracted per package. Features with a clear majority (≥75% conformance): file tree,package.jsonkeys,directories,engines,os, README section set, error construction viaformat(100%), TypeScript declarations, and per-file JSDoc@throws↔throwsymmetry. Features without a clear majority (excluded from drift analysis):package.jsonextras (__stdlib__,browser,gypfile, present only for packages with native bindings), presence ofC APIsREADME section (justified absent for pure-JS complex-arithmetic packages), presence ofbenchmark/benchmark.ndarray.js/lib/ndarray.js/test/test.ndarray.js(justified absent for scalar-input routines).Per outlier package
lapack/base/clacgvlib/ndarray.jssummary omitted the "using alternative indexing semantics" qualifier that the package's ownREADME.md,docs/repl.txt, anddocs/types/index.d.tsalready carry for the.ndarrayAPI. Appended the qualifier so the four in-package doc surfaces agree.lapack/base/clasetSame drift as sibling
dlaset(which carries the qualifier).lib/ndarray.jssummary now matches the package's own README (line 84), repl.txt, and TS declarations for the.ndarrayoverload.lapack/base/claswpBecause the summary already contains a "using pivot indices stored in
IPIV" clause, the ndarray-form qualifier is appended as "and alternative indexing semantics", matching the phrasing already used in the package's README (line 113), repl.txt, and TS declarations.lapack/base/crotThe strided (main) API summary was reused verbatim for the
.ndarrayvariant. Appended "using alternative indexing semantics" to restore the distinction, aligning with the phrasing the package's other doc surfaces already carry.lapack/base/dladivThe
.ndarrayJSDoc was a copy of the strided API summary. Appended "using alternative indexing semantics" to match the package's README (line 59), repl.txt, and TS declarations for the.ndarrayoverload.lapack/base/dlarfgit log -S "alternative indexing semantics"shows the qualifier was propagated through README, repl, and TS but never landed inlib/ndarray.js. Appended it here so all four doc surfaces agree.lapack/base/dlarf1fSibling of
dlarfwith an identical docs skeleton; same drift, same fix.lib/ndarray.jsJSDoc summary now matches README, repl, and TS.lapack/base/dlaswpAppended "and alternative indexing semantics" (the qualifier variant used when the summary already contains a "using" clause), matching the phrasing the package's own README, repl.txt, and TS declarations already use for the
.ndarrayAPI.lapack/base/slaswpMirror of
dlaswpat single precision; same drift, same fix. Ndarray JSDoc summary now matches the phrasing in the package's other doc surfaces.lapack/base/zlacgvMirror of
clacgvat double precision. Appended "using alternative indexing semantics" so the.ndarrayJSDoc summary matches the package's own README and repl.txt.lapack/base/zlasetSame drift as sibling
dlaset(which carries the qualifier). Ndarray JSDoc summary now matches the package's own README (line 84), repl.txt, and TS declarations.lapack/base/zlaswpMirror of
dlaswpat double-precision complex; same drift, same fix. Ndarray JSDoc summary now matches the phrasing already present in the package's other doc surfaces.lapack/base/zrotMirror of
crotat double precision. The.ndarrayJSDoc reused the strided API summary verbatim; appended "using alternative indexing semantics" to restore the distinction, matching the phrasing in the package's README, repl.txt, and TS declarations.Validation
sharedmembers, deriving majority patterns per feature (≥75% conformance threshold).format()-based error construction; per-file@throws↔throwsymmetry holds across 24/25 files.confirmed-driftfor all 13 (no intentional deviation); cross-reference confirmed no test or example under any of the 13 packages references the JSDoc summary text (safe to change); structural-review confirmed the fix direction (append to the 13 lacking JSDoc summaries; do not remove elsewhere).xerblaexports a directory path, not a routine;disnan/dlaisnan/dlamch/dlapy2/dlapy3are scalar-input functions with no.ndarrayvariant; pure-JS complex-arithmetic packages legitimately lackC APIssections); one-line metadata tweaks that failed the materiality gate (missing routine-name keyword indge-trans,sge-trans,xerblapackage.json); thezrotparameter naming drift (strideX/strideYvs the sibling conventionstrideZX/strideZY) because a coherent fix would require touchingdocs/repl.txt, which is out of scope for this pass.Related Issues
No.
Questions
No.
Other
Two orthogonal drift signals were flagged during review but not corrected in this PR (either because they require the same-group companion change or because they involve a generator-owned file):
docs/types/index.d.tsinclacgvandzlacgvis also missing the "using alternative indexing semantics" qualifier on the.ndarrayoverload description; anddladiv's TS declaration uses "with" instead of "using".clacgv/README.mdandzlacgv/README.mddescribe the input as "single-precision/double-precision floating-point vector" (missing "complex"), while the JSDoc, TS, and repl.txt in the same packages say "complex floating-point vector".Checklist
AI Assistance
If you answered "yes" above, how did you use AI assistance?
Disclosure
This PR was authored with AI assistance in the drift-detection loop: automated structural and semantic feature extraction across the 32 packages of
@stdlib/lapack/base, majority computation per feature, and three-agent adversarial validation of the 13 outliers surfaced. The final diff — 13 one-line JSDoc summary edits, no code changes — was reviewed against each package's README,docs/repl.txt, anddocs/types/index.d.tsbefore commit.@stdlib-js/reviewers
Generated by Claude Code