feat: add `blas/ext/base/ndarray/dapxsum` by piyxsh31 · Pull Request #9287 · stdlib-js/stdlib

piyxsh31 · 2025-12-20T19:22:37Z

type: pre_commit_static_analysis_report
description: Results of running static analysis checks when committing changes. report:

task: lint_filenames status: passed
task: lint_editorconfig status: passed
task: lint_markdown status: passed
task: lint_package_json status: passed
task: lint_repl_help status: passed
task: lint_javascript_src status: passed
task: lint_javascript_cli status: na
task: lint_javascript_examples status: passed
task: lint_javascript_tests status: passed
task: lint_javascript_benchmarks status: passed
task: lint_python status: na
task: lint_r status: na
task: lint_c_src status: na
task: lint_c_examples status: na
task: lint_c_benchmarks status: na
task: lint_c_tests_fixtures status: na
task: lint_shell status: na
task: lint_typescript_declarations status: passed
task: lint_typescript_tests status: passed
task: lint_license_headers status: passed ---

Description

What is the purpose of this pull request?

This pull request:

add blas/ext/base/ndarray/dapxsum

Related Issues

Does this pull request have any related issues?

No.

Questions

Any questions for reviewers of this pull request?

No.

Other

Any other information relevant to this pull request? This may include screenshots, references, and/or implementation notes.

No.

Checklist

Please ensure the following tasks are completed before submitting this pull request.

Read, understood, and followed the contributing guidelines.

AI Assistance

When authoring the changes proposed in this PR, did you use any kind of 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

If you answered "yes" to using AI assistance, please provide a short disclosure indicating how you used AI assistance. This helps reviewers determine how much scrutiny to apply when reviewing your contribution. Example disclosures: "This PR was written primarily by Claude Code." or "I consulted ChatGPT to understand the codebase, but the proposed changes were fully authored manually by myself.".

@stdlib-js/reviewers

--- type: pre_commit_static_analysis_report description: Results of running static analysis checks when committing changes. report: - task: lint_filenames status: passed - task: lint_editorconfig status: passed - task: lint_markdown status: passed - task: lint_package_json status: passed - task: lint_repl_help status: passed - task: lint_javascript_src status: passed - task: lint_javascript_cli status: na - task: lint_javascript_examples status: passed - task: lint_javascript_tests status: passed - task: lint_javascript_benchmarks status: passed - task: lint_python status: na - task: lint_r status: na - task: lint_c_src status: na - task: lint_c_examples status: na - task: lint_c_benchmarks status: na - task: lint_c_tests_fixtures status: na - task: lint_shell status: na - task: lint_typescript_declarations status: passed - task: lint_typescript_tests status: passed - task: lint_license_headers status: passed ---

stdlib-bot · 2025-12-20T19:24:11Z

Coverage Report

Package	Statements	Branches	Functions	Lines
blas/ext/base/ndarray/dapxsum	$\color{green}116/116$ $\color{green}+0.00%$	$\color{green}3/3$ $\color{green}+0.00%$	$\color{green}1/1$ $\color{green}+0.00%$	$\color{green}116/116$ $\color{green}+0.00%$

The above coverage report was generated for the changes in this PR.

Signed-off-by: piyxsh31 <agrawalpiyush415@gmail.com>

kgryte · 2025-12-20T21:35:53Z

lib/node_modules/@stdlib/blas/ext/base/ndarray/dapxsum/README.md

+
+# dapxsum
+
+> Add a scalar constant to each one-dimensional double-precision floating-point ndarray element and compute the sum.


Suggested change

> Add a scalar constant to each one-dimensional double-precision floating-point ndarray element and compute the sum.

> Add a scalar constant to each element of a one-dimensional double-precision floating-point ndarray and compute the sum.

Let's go ahead and make this change to the description here and elsewhere in this PR. I know that you are basing it off of blas/ext/base/dapxsum, but I think the suggested phrasing is a touch better.

kgryte · 2025-12-20T21:36:48Z

lib/node_modules/@stdlib/blas/ext/base/ndarray/dapxsum/docs/types/index.d.ts

+/**
+* Adds a scalar constant to each one-dimensional double-precision floating-point ndarray element and computes the sum.
+*
+* @param arrays - array-like object containing a one-dimensional input ndarray and a zero-dimensional ndarray containing a scalar constant.


Suggested change

* @param arrays - array-like object containing a one-dimensional input ndarray and a zero-dimensional ndarray containing a scalar constant.

* @param arrays - array-like object containing a one-dimensional input ndarray and a zero-dimensional ndarray containing a scalar constant

kgryte · 2025-12-20T21:36:57Z

lib/node_modules/@stdlib/blas/ext/base/ndarray/dapxsum/docs/types/index.d.ts

+* Adds a scalar constant to each one-dimensional double-precision floating-point ndarray element and computes the sum.
+*
+* @param arrays - array-like object containing a one-dimensional input ndarray and a zero-dimensional ndarray containing a scalar constant.
+* @returns computed sum


Suggested change

* @returns computed sum

* @returns sum

kgryte · 2025-12-20T21:37:42Z

lib/node_modules/@stdlib/blas/ext/base/ndarray/dapxsum/lib/main.js

+/**
+* Adds a scalar constant to each one-dimensional double-precision floating-point ndarray element and computes the sum.
+*
+* @param {ArrayLikeObject<Object>} arrays - array-like object containing a one-dimensional input ndarray and a zero-dimensional ndarray containing a scalar constant.


Suggested change

* @param {ArrayLikeObject<Object>} arrays - array-like object containing a one-dimensional input ndarray and a zero-dimensional ndarray containing a scalar constant.

* @param {ArrayLikeObject<Object>} arrays - array-like object containing a one-dimensional input ndarray and a zero-dimensional ndarray containing a scalar constant

kgryte · 2025-12-20T21:37:51Z

lib/node_modules/@stdlib/blas/ext/base/ndarray/dapxsum/lib/main.js

+* Adds a scalar constant to each one-dimensional double-precision floating-point ndarray element and computes the sum.
+*
+* @param {ArrayLikeObject<Object>} arrays - array-like object containing a one-dimensional input ndarray and a zero-dimensional ndarray containing a scalar constant.
+* @returns {number} computed sum


Suggested change

* @returns {number} computed sum

* @returns {number} sum

kgryte · 2025-12-20T21:38:14Z

lib/node_modules/@stdlib/blas/ext/base/ndarray/dapxsum/package.json

+{
+  "name": "@stdlib/blas/ext/base/ndarray/dapxsum",
+  "version": "0.0.0",
+  "description": "Adds a scalar constant to each one-dimensional double-precision floating-point ndarray element and computes the sum.",


Suggested change

"description": "Adds a scalar constant to each one-dimensional double-precision floating-point ndarray element and computes the sum.",

"description": "Add a scalar constant to each element of a one-dimensional double-precision floating-point ndarray and compute the sum.",

kgryte · 2025-12-20T21:39:38Z

lib/node_modules/@stdlib/blas/ext/base/ndarray/dapxsum/test/test.js

+	t.end();
+});
+
+tape( 'the function adds a scalar constant to each element and computes the sum', function test( t ) {


The tests here are a little too spartan. See other similar base/ndarray packages. You should include tests for non-unit strides, a non-zero offset, negative strides, and what happens when provided an empty ndarray.

kgryte · 2025-12-20T21:39:59Z

lib/node_modules/@stdlib/blas/ext/base/ndarray/dapxsum/test/test.js

+	v = dapxsum( [ x, alpha ] );
+
+	t.strictEqual( v, 16.0, 'returns expected sum' );
+	t.deepEqual( getData( x ), new Float64Array( [ 1.0, -2.0, 2.0 ] ), 'does not mutate input ndarray' );


Suggested change

t.deepEqual( getData( x ), new Float64Array( [ 1.0, -2.0, 2.0 ] ), 'does not mutate input ndarray' );

t.deepEqual( getData( x ), new Float64Array( [ 1.0, -2.0, 2.0 ] ), 'returns expected value' );

kgryte · 2025-12-20T21:40:08Z

lib/node_modules/@stdlib/blas/ext/base/ndarray/dapxsum/test/test.js

+
+	v = dapxsum( [ x, alpha ] );
+
+	t.strictEqual( v, 16.0, 'returns expected sum' );


Suggested change

t.strictEqual( v, 16.0, 'returns expected sum' );

t.strictEqual( v, 16.0, 'returns expected value' );

kgryte

Overall, this is looking good. Needs some minor clean-up and additional tests.

--- type: pre_commit_static_analysis_report description: Results of running static analysis checks when committing changes. report: - task: lint_filenames status: passed - task: lint_editorconfig status: passed - task: lint_markdown status: passed - task: lint_package_json status: passed - task: lint_repl_help status: passed - task: lint_javascript_src status: passed - task: lint_javascript_cli status: na - task: lint_javascript_examples status: na - task: lint_javascript_tests status: passed - task: lint_javascript_benchmarks status: na - task: lint_python status: na - task: lint_r status: na - task: lint_c_src status: na - task: lint_c_examples status: na - task: lint_c_benchmarks status: na - task: lint_c_tests_fixtures status: na - task: lint_shell status: na - task: lint_typescript_declarations status: passed - task: lint_typescript_tests status: passed - task: lint_license_headers status: passed ---

piyxsh31 · 2025-12-21T06:36:09Z

@kgryte
I’ve addressed all the suggested cleanups and added additional tests covering non-unit strides, non-zero offsets, negative strides, and empty ndarrays.

Whenever you have time, could you please take another look? Thanks!

Planeshifter

A few small documentation edits. Thanks for your PR, which should be ready to go otherwise!

lib/node_modules/@stdlib/blas/ext/base/ndarray/dapxsum/README.md

lib/node_modules/@stdlib/blas/ext/base/ndarray/dapxsum/docs/repl.txt

lib/node_modules/@stdlib/blas/ext/base/ndarray/dapxsum/docs/types/index.d.ts

lib/node_modules/@stdlib/blas/ext/base/ndarray/dapxsum/lib/main.js

Co-authored-by: Philipp Burckhardt <pburckhardt@outlook.com> Signed-off-by: piyxsh31 <agrawalpiyush415@gmail.com>

…pl.txt Co-authored-by: Philipp Burckhardt <pburckhardt@outlook.com> Signed-off-by: piyxsh31 <agrawalpiyush415@gmail.com>

…pes/index.d.ts Co-authored-by: Philipp Burckhardt <pburckhardt@outlook.com> Signed-off-by: piyxsh31 <agrawalpiyush415@gmail.com>

…n.js Co-authored-by: Philipp Burckhardt <pburckhardt@outlook.com> Signed-off-by: piyxsh31 <agrawalpiyush415@gmail.com>

piyxsh31 · 2026-02-09T03:32:24Z

@Planeshifter
I’ve addressed all the suggested documentation edits

Whenever you have time, could you please take another look? Thanks!

stdlib-bot added BLAS Issue or pull request related to Basic Linear Algebra Subprograms (BLAS). Needs Review A pull request which needs code review. labels Dec 20, 2025

remove lint errors

0347df4

Signed-off-by: piyxsh31 <agrawalpiyush415@gmail.com>

kgryte added Feature Issue or pull request for adding a new feature. and removed Needs Review A pull request which needs code review. labels Dec 20, 2025

kgryte reviewed Dec 20, 2025

View reviewed changes

kgryte requested changes Dec 20, 2025

View reviewed changes

kgryte added the Needs Changes Pull request which needs changes before being merged. label Dec 20, 2025

Planeshifter requested changes Jan 10, 2026

View reviewed changes

piyxsh31 and others added 5 commits February 9, 2026 08:59

Update lib/node_modules/@stdlib/blas/ext/base/ndarray/dapxsum/README.md

55a2589

Co-authored-by: Philipp Burckhardt <pburckhardt@outlook.com> Signed-off-by: piyxsh31 <agrawalpiyush415@gmail.com>

Update lib/node_modules/@stdlib/blas/ext/base/ndarray/dapxsum/docs/re…

7d6878f

…pl.txt Co-authored-by: Philipp Burckhardt <pburckhardt@outlook.com> Signed-off-by: piyxsh31 <agrawalpiyush415@gmail.com>

Update lib/node_modules/@stdlib/blas/ext/base/ndarray/dapxsum/docs/re…

2e487fe

…pl.txt Co-authored-by: Philipp Burckhardt <pburckhardt@outlook.com> Signed-off-by: piyxsh31 <agrawalpiyush415@gmail.com>

Update lib/node_modules/@stdlib/blas/ext/base/ndarray/dapxsum/docs/ty…

87cf6c6

…pes/index.d.ts Co-authored-by: Philipp Burckhardt <pburckhardt@outlook.com> Signed-off-by: piyxsh31 <agrawalpiyush415@gmail.com>

Update lib/node_modules/@stdlib/blas/ext/base/ndarray/dapxsum/lib/mai…

e54af9c

…n.js Co-authored-by: Philipp Burckhardt <pburckhardt@outlook.com> Signed-off-by: piyxsh31 <agrawalpiyush415@gmail.com>


		# dapxsum

		> Add a scalar constant to each one-dimensional double-precision floating-point ndarray element and compute the sum.

	> Add a scalar constant to each one-dimensional double-precision floating-point ndarray element and compute the sum.
	> Add a scalar constant to each element of a one-dimensional double-precision floating-point ndarray and compute the sum.

	* @param arrays - array-like object containing a one-dimensional input ndarray and a zero-dimensional ndarray containing a scalar constant.
	* @param arrays - array-like object containing a one-dimensional input ndarray and a zero-dimensional ndarray containing a scalar constant

	* @param {ArrayLikeObject<Object>} arrays - array-like object containing a one-dimensional input ndarray and a zero-dimensional ndarray containing a scalar constant.
	* @param {ArrayLikeObject<Object>} arrays - array-like object containing a one-dimensional input ndarray and a zero-dimensional ndarray containing a scalar constant

	"description": "Adds a scalar constant to each one-dimensional double-precision floating-point ndarray element and computes the sum.",
	"description": "Add a scalar constant to each element of a one-dimensional double-precision floating-point ndarray and compute the sum.",

	t.deepEqual( getData( x ), new Float64Array( [ 1.0, -2.0, 2.0 ] ), 'does not mutate input ndarray' );
	t.deepEqual( getData( x ), new Float64Array( [ 1.0, -2.0, 2.0 ] ), 'returns expected value' );


		v = dapxsum( [ x, alpha ] );

		t.strictEqual( v, 16.0, 'returns expected sum' );

Uh oh!

Comments

Conversation

piyxsh31 commented Dec 20, 2025

Description

Related Issues

Questions

Other

Checklist

AI Assistance

Disclosure

Uh oh!

stdlib-bot commented Dec 20, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Coverage Report

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

kgryte left a comment

Choose a reason for hiding this comment

Uh oh!

piyxsh31 commented Dec 21, 2025

Uh oh!

Planeshifter left a comment • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

piyxsh31 commented Feb 9, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants

stdlib-bot commented Dec 20, 2025 •

edited

Loading

Planeshifter left a comment •

edited

Loading

piyxsh31 commented Feb 9, 2026 •

edited

Loading