Skip to content

Commit 0e2e776

Browse files
committed
chore: apply review changes
1 parent 4e57266 commit 0e2e776

File tree

10 files changed

+62
-42
lines changed

10 files changed

+62
-42
lines changed

lib/node_modules/@stdlib/blas/ext/base/dnansumkbn2/README.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -97,7 +97,7 @@ The function has the following additional parameters:
9797

9898
- **offsetX**: starting index for `x`.
9999

100-
While [`typed array`][mdn-typed-array] views mandate a view offset based on the underlying buffer, the offset parameter supports indexing semantics based on a starting index. For example, to calculate the sum of every other value in `x` starting from the second value:
100+
While [`typed array`][mdn-typed-array] views mandate a view offset based on the underlying buffer, the offset parameter supports indexing semantics based on a starting index. For example, to calculate the sum of every other element starting from the second element:
101101

102102
```javascript
103103
var Float64Array = require( '@stdlib/array/float64' );

lib/node_modules/@stdlib/blas/ext/base/dnansumkbn2/benchmark/benchmark.js

Lines changed: 14 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -32,6 +32,19 @@ var dnansumkbn2 = require( './../lib/dnansumkbn2.js' );
3232

3333
// FUNCTIONS //
3434

35+
/**
36+
* Returns a random number.
37+
*
38+
* @private
39+
* @returns {number} random number
40+
*/
41+
function rand() {
42+
if ( bernoulli( 0.7 ) > 0 ) {
43+
return uniform( -100.0, 100.0 );
44+
}
45+
return NaN;
46+
}
47+
3548
/**
3649
* Creates a benchmark function.
3750
*
@@ -40,16 +53,9 @@ var dnansumkbn2 = require( './../lib/dnansumkbn2.js' );
4053
* @returns {Function} benchmark function
4154
*/
4255
function createBenchmark( len ) {
43-
var x = filledarrayBy( len, 'float64', clbk );
56+
var x = filledarrayBy( len, 'float64', rand );
4457
return benchmark;
4558

46-
function clbk() {
47-
if ( bernoulli( 0.7 ) > 0 ) {
48-
return uniform( -10.0, 10.0 );
49-
}
50-
return NaN;
51-
}
52-
5359
function benchmark( b ) {
5460
var v;
5561
var i;

lib/node_modules/@stdlib/blas/ext/base/dnansumkbn2/benchmark/benchmark.native.js

Lines changed: 14 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -41,6 +41,19 @@ var opts = {
4141

4242
// FUNCTIONS //
4343

44+
/**
45+
* Returns a random number.
46+
*
47+
* @private
48+
* @returns {number} random number
49+
*/
50+
function rand() {
51+
if ( bernoulli( 0.7 ) > 0 ) {
52+
return uniform( -100.0, 100.0 );
53+
}
54+
return NaN;
55+
}
56+
4457
/**
4558
* Creates a benchmark function.
4659
*
@@ -49,16 +62,9 @@ var opts = {
4962
* @returns {Function} benchmark function
5063
*/
5164
function createBenchmark( len ) {
52-
var x = filledarrayBy( len, 'float64', clbk );
65+
var x = filledarrayBy( len, 'float64', rand );
5366
return benchmark;
5467

55-
function clbk() {
56-
if ( bernoulli( 0.7 ) > 0 ) {
57-
return uniform( -10.0, 10.0 );
58-
}
59-
return NaN;
60-
}
61-
6268
function benchmark( b ) {
6369
var v;
6470
var i;

lib/node_modules/@stdlib/blas/ext/base/dnansumkbn2/benchmark/benchmark.ndarray.js

Lines changed: 14 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -32,6 +32,19 @@ var dnansumkbn2 = require( './../lib/ndarray.js' );
3232

3333
// FUNCTIONS //
3434

35+
/**
36+
* Returns a random number.
37+
*
38+
* @private
39+
* @returns {number} random number
40+
*/
41+
function rand() {
42+
if ( bernoulli( 0.7 ) > 0 ) {
43+
return uniform( -100.0, 100.0 );
44+
}
45+
return NaN;
46+
}
47+
3548
/**
3649
* Creates a benchmark function.
3750
*
@@ -40,16 +53,9 @@ var dnansumkbn2 = require( './../lib/ndarray.js' );
4053
* @returns {Function} benchmark function
4154
*/
4255
function createBenchmark( len ) {
43-
var x = filledarrayBy( len, 'float64', clbk );
56+
var x = filledarrayBy( len, 'float64', rand );
4457
return benchmark;
4558

46-
function clbk() {
47-
if ( bernoulli( 0.7 ) > 0 ) {
48-
return uniform( -10.0, 10.0 );
49-
}
50-
return NaN;
51-
}
52-
5359
function benchmark( b ) {
5460
var v;
5561
var i;

lib/node_modules/@stdlib/blas/ext/base/dnansumkbn2/benchmark/benchmark.ndarray.native.js

Lines changed: 14 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -41,6 +41,19 @@ var opts = {
4141

4242
// FUNCTIONS //
4343

44+
/**
45+
* Returns a random number.
46+
*
47+
* @private
48+
* @returns {number} random number
49+
*/
50+
function rand() {
51+
if ( bernoulli( 0.7 ) > 0 ) {
52+
return uniform( -100.0, 100.0 );
53+
}
54+
return NaN;
55+
}
56+
4457
/**
4558
* Creates a benchmark function.
4659
*
@@ -49,16 +62,9 @@ var opts = {
4962
* @returns {Function} benchmark function
5063
*/
5164
function createBenchmark( len ) {
52-
var x = filledarrayBy( len, 'float64', clbk );
65+
var x = filledarrayBy( len, 'float64', rand );
5366
return benchmark;
5467

55-
function clbk() {
56-
if ( bernoulli( 0.7 ) > 0 ) {
57-
return uniform( -10.0, 10.0 );
58-
}
59-
return NaN;
60-
}
61-
6268
function benchmark( b ) {
6369
var v;
6470
var i;

lib/node_modules/@stdlib/blas/ext/base/dnansumkbn2/lib/dnansumkbn2.js

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -46,9 +46,8 @@ var ndarray = require( './ndarray.js' );
4646
* var Float64Array = require( '@stdlib/array/float64' );
4747
*
4848
* var x = new Float64Array( [ 1.0, -2.0, NaN, 2.0 ] );
49-
* var N = x.length;
5049
*
51-
* var v = dnansumkbn2( N, x, 1 );
50+
* var v = dnansumkbn2( x.length, x, 1 );
5251
* // returns 1.0
5352
*/
5453
function dnansumkbn2( N, x, strideX ) {

lib/node_modules/@stdlib/blas/ext/base/dnansumkbn2/lib/dnansumkbn2.native.js

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -37,9 +37,8 @@ var addon = require( './../src/addon.node' );
3737
* var Float64Array = require( '@stdlib/array/float64' );
3838
*
3939
* var x = new Float64Array( [ 1.0, -2.0, NaN, 2.0 ] );
40-
* var N = x.length;
4140
*
42-
* var v = dnansumkbn2( N, x, 1 );
41+
* var v = dnansumkbn2( x.length, x, 1 );
4342
* // returns 1.0
4443
*/
4544
function dnansumkbn2( N, x, strideX ) {

lib/node_modules/@stdlib/blas/ext/base/dnansumkbn2/lib/index.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -38,7 +38,7 @@
3838
*
3939
* var x = new Float64Array( [ 2.0, 1.0, 2.0, -2.0, -2.0, 2.0, 3.0, 4.0, NaN, NaN ] );
4040
*
41-
* var v = dnansumkbn2.ndarray( 5, x, 2, 1 );
41+
* var v = dnansumkbn2.ndarray( x.length, x, 2, 1 );
4242
* // returns 5.0
4343
*/
4444

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

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -71,8 +71,7 @@ function dnansumkbn2( N, x, strideX, offsetX ) {
7171
if ( isnan( x[ ix ] ) ) {
7272
return sum;
7373
}
74-
sum = x[ ix ] * N;
75-
return sum;
74+
return x[ ix ] * N;
7675
}
7776
ccs = 0.0; // second order correction term for lost low order bits
7877
cs = 0.0; // first order correction term for lost low order bits

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

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -80,8 +80,7 @@ double API_SUFFIX(stdlib_strided_dnansumkbn2_ndarray)( const CBLAS_INT N, const
8080
if ( stdlib_base_is_nan( X[ ix ] ) ) {
8181
return sum;
8282
}
83-
sum = X[ ix ] * N;
84-
return sum;
83+
return X[ ix ] * N;
8584
}
8685
ccs = 0.0; // second order correction term for lost lower order bits
8786
cs = 0.0; // first order correction term for lost low order bits

0 commit comments

Comments
 (0)