fix: update implementation to preserve signed zeros

---
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: na
  - task: lint_package_json
    status: na
  - task: lint_repl_help
    status: na
  - 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: passed
  - 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: na
  - task: lint_typescript_tests
    status: na
  - task: lint_license_headers
    status: passed
---

Author: kgryte

lib/node_modules/@stdlib/blas/ext/base/dnannsumkbn/lib/ndarray.js

@@ -57,33 +57,67 @@ var abs = require( '@stdlib/math/base/special/abs' );
 */
 function dnannsumkbn( N, x, strideX, offsetX, out, strideOut, offsetOut ) {
 	var sum;
+	var flg;
 	var ix;
 	var v;
 	var t;
 	var c;
 	var n;
 	var i;
 
-	sum = 0.0;
 	if ( N <= 0 ) {
-		out[ offsetOut ] = sum;
+		out[ offsetOut ] = 0.0;
 		out[ offsetOut+strideOut ] = 0;
 		return out;
 	}
 	ix = offsetX;
 	if ( strideX === 0 ) {
 		if ( isnan( x[ ix ] ) ) {
-			out[ offsetOut ] = sum;
+			out[ offsetOut ] = 0.0;
 			out[ offsetOut+strideOut ] = 0;
 			return out;
 		}
 		out[ offsetOut ] = x[ ix ] * N;
 		out[ offsetOut+strideOut ] = N;
 		return out;
 	}
-	c = 0.0;
-	n = 0;
+	// Find the first non-NaN element...
 	for ( i = 0; i < N; i++ ) {
+		v = x[ ix ];
+		if ( isnan( v ) === false ) {
+			break;
+		}
+		ix += strideX;
+	}
+	if ( i === N ) {
+		out[ offsetOut ] = 0.0;
+		out[ offsetOut+strideOut ] = 0;
+		return out;
+	}
+	n = 1;
+	sum = v;
+	ix += strideX;
+	i += 1;
+
+	// In order to preserve the sign of zero which can be lost during compensated summation below, find the first non-zero element...
+	if ( sum === 0.0 ) {
+		for ( ; i < N; i++ ) {
+			v = x[ ix ];
+			if ( isnan( v ) === false ) {
+				if ( v !== 0.0 ) {
+					flg = true;
+					break;
+				}
+				sum += v;
+				n += 1;
+			}
+			ix += strideX;
+		}
+	} else {
+		flg = true;
+	}
+	c = 0.0;
+	for ( ; i < N; i++ ) {
 		v = x[ ix ];
 		if ( isnan( v ) === false ) {
 			t = sum + v;
@@ -97,7 +131,7 @@ function dnannsumkbn( N, x, strideX, offsetX, out, strideOut, offsetOut ) {
 		}
 		ix += strideX;
 	}
-	out[ offsetOut ] = sum + c;
+	out[ offsetOut ] = ( flg ) ? sum+c : sum;
 	out[ offsetOut+strideOut ] = n;
 	return out;
 }

lib/node_modules/@stdlib/blas/ext/base/dnannsumkbn/src/main.c

@@ -69,22 +69,56 @@ double API_SUFFIX(stdlib_strided_dnannsumkbn_ndarray)( const CBLAS_INT N, const
 	double v;
 	double t;
 	double c;
+	int flg;
 
-	sum = 0.0;
 	*n = 0;
 	if ( N <= 0 ) {
-		return sum;
+		return 0.0;
 	}
 	ix = offsetX;
 	if ( strideX == 0 ) {
 		if ( stdlib_base_is_nan( X[ ix ] ) ) {
-			return sum;
+			return 0.0;
 		}
 		*n += N;
 		return X[ ix ] * N;
 	}
-	c = 0.0;
+	// Find the first non-NaN element...
 	for ( i = 0; i < N; i++ ) {
+		v = X[ ix ];
+		if ( !stdlib_base_is_nan( v ) ) {
+			break;
+		}
+		ix += strideX;
+	}
+	if ( i == N ) {
+		return 0.0;
+	}
+	*n += 1;
+	sum = v;
+	ix += strideX;
+	i += 1;
+	flg = 0;
+
+	// In order to preserve the sign of zero which can be lost during compensated summation below, find the first non-zero element...
+	if ( sum == 0.0 ) {
+		for ( ; i < N; i++ ) {
+			v = X[ ix ];
+			if ( !stdlib_base_is_nan( v ) ) {
+				if ( v != 0.0 ) {
+					flg = 1;
+					break;
+				}
+				sum += v;
+				*n += 1;
+			}
+			ix += strideX;
+		}
+	} else {
+		flg = 1;
+	}
+	c = 0.0;
+	for ( ; i < N; i++ ) {
 		v = X[ ix ];
 		if ( !stdlib_base_is_nan( v ) ) {
 			t = sum + v;
@@ -98,5 +132,5 @@ double API_SUFFIX(stdlib_strided_dnannsumkbn_ndarray)( const CBLAS_INT N, const
 		}
 		ix += strideX;
 	}
-	return sum + c;
+	return ( flg == 1 ) ? sum+c : sum;
 }

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

@@ -22,6 +22,7 @@
 
 var tape = require( 'tape' );
 var Float64Array = require( '@stdlib/array/float64' );
+var isSameFloat64Array = require( '@stdlib/assert/is-same-float64array' );
 var dnannsumkbn = require( './../lib/ndarray.js' );
 
 
@@ -119,6 +120,24 @@ tape( 'the function calculates the sum of strided array elements (ignoring NaN v
 	t.end();
 });
 
+tape( 'the function preserves the sign of zero', function test( t ) {
+	var expected;
+	var out;
+	var x;
+	var v;
+
+	x = new Float64Array( [ -0.0, -0.0, -0.0, -0.0, -0.0 ] );
+
+	out = new Float64Array( 2 );
+	v = dnannsumkbn( x.length, x, 1, 0, out, 1, 0 );
+
+	expected = new Float64Array( [ -0.0, 5.0 ] );
+	t.strictEqual( v, out, 'returns expected value' );
+	t.strictEqual( isSameFloat64Array( v, expected ), true, 'returns expected value' );
+
+	t.end();
+});
+
 tape( 'if provided an `N` parameter less than or equal to `0`, the function returns `0.0`', function test( t ) {
 	var expected;
 	var out;

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

@@ -23,6 +23,7 @@
 var resolve = require( 'path' ).resolve;
 var tape = require( 'tape' );
 var Float64Array = require( '@stdlib/array/float64' );
+var isSameFloat64Array = require( '@stdlib/assert/is-same-float64array' );
 var tryRequire = require( '@stdlib/utils/try-require' );
 
 
@@ -128,6 +129,24 @@ tape( 'the function calculates the sum of strided array elements (ignoring NaN v
 	t.end();
 });
 
+tape( 'the function preserves the sign of zero', opts, function test( t ) {
+	var expected;
+	var out;
+	var x;
+	var v;
+
+	x = new Float64Array( [ -0.0, -0.0, -0.0, -0.0, -0.0 ] );
+
+	out = new Float64Array( 2 );
+	v = dnannsumkbn( x.length, x, 1, 0, out, 1, 0 );
+
+	expected = new Float64Array( [ -0.0, 5.0 ] );
+	t.strictEqual( v, out, 'returns expected value' );
+	t.strictEqual( isSameFloat64Array( v, expected ), true, 'returns expected value' );
+
+	t.end();
+});
+
 tape( 'if provided an `N` parameter less than or equal to `0`, the function returns `0.0`', opts, function test( t ) {
 	var expected;
 	var out;