Skip to content

Commit 6f75d45

Browse files
AuenKrPlaneshifter
andauthored
refactor: update blas/ext/base/dsapxsum to follow current project conventions
PR-URL: #1888 Closes: #1489 --------- Signed-off-by: Philipp Burckhardt <[email protected]> Co-authored-by: Philipp Burckhardt <[email protected]> Reviewed-by: Philipp Burckhardt <[email protected]>
1 parent aebb788 commit 6f75d45

21 files changed

+204
-319
lines changed

Diff for: lib/node_modules/@stdlib/blas/ext/base/dsapxsum/README.md

+10-26
Original file line numberDiff line numberDiff line change
@@ -44,9 +44,8 @@ Adds a constant to each single-precision floating-point strided array element an
4444
var Float32Array = require( '@stdlib/array/float32' );
4545

4646
var x = new Float32Array( [ 1.0, -2.0, 2.0 ] );
47-
var N = x.length;
4847

49-
var v = dsapxsum( N, 5.0, x, 1 );
48+
var v = dsapxsum( 3, 5.0, x, 1 );
5049
// returns 16.0
5150
```
5251

@@ -56,16 +55,14 @@ The function has the following parameters:
5655
- **x**: input [`Float32Array`][@stdlib/array/float32].
5756
- **stride**: index increment for `x`.
5857

59-
The `N` and `stride` parameters determine which elements in `x` are accessed at runtime. For example, to access every other element in `x`,
58+
The `N` and `stride` parameters determine which elements in the strided array are accessed at runtime. For example, to access every other element in the strided array,
6059

6160
```javascript
6261
var Float32Array = require( '@stdlib/array/float32' );
63-
var floor = require( '@stdlib/math/base/special/floor' );
6462

6563
var x = new Float32Array( [ 1.0, 2.0, 2.0, -7.0, -2.0, 3.0, 4.0, 2.0 ] );
66-
var N = floor( x.length / 2 );
6764

68-
var v = dsapxsum( N, 5.0, x, 2 );
65+
var v = dsapxsum( 4, 5.0, x, 2 );
6966
// returns 25.0
7067
```
7168

@@ -75,14 +72,11 @@ Note that indexing is relative to the first index. To introduce an offset, use [
7572

7673
```javascript
7774
var Float32Array = require( '@stdlib/array/float32' );
78-
var floor = require( '@stdlib/math/base/special/floor' );
7975

8076
var x0 = new Float32Array( [ 2.0, 1.0, 2.0, -2.0, -2.0, 2.0, 3.0, 4.0 ] );
8177
var x1 = new Float32Array( x0.buffer, x0.BYTES_PER_ELEMENT*1 ); // start at 2nd element
8278

83-
var N = floor( x0.length / 2 );
84-
85-
var v = dsapxsum( N, 5.0, x1, 2 );
79+
var v = dsapxsum( 4, 5.0, x1, 2 );
8680
// returns 25.0
8781
```
8882

@@ -94,26 +88,23 @@ Adds a constant to each single-precision floating-point strided array element an
9488
var Float32Array = require( '@stdlib/array/float32' );
9589

9690
var x = new Float32Array( [ 1.0, -2.0, 2.0 ] );
97-
var N = x.length;
9891

99-
var v = dsapxsum.ndarray( N, 5.0, x, 1, 0 );
92+
var v = dsapxsum.ndarray( 3, 5.0, x, 1, 0 );
10093
// returns 16.0
10194
```
10295

10396
The function has the following additional parameters:
10497

10598
- **offset**: starting index for `x`.
10699

107-
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 access 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 access every other value in the strided array starting from the second value
108101

109102
```javascript
110103
var Float32Array = require( '@stdlib/array/float32' );
111-
var floor = require( '@stdlib/math/base/special/floor' );
112104

113105
var x = new Float32Array( [ 2.0, 1.0, 2.0, -2.0, -2.0, 2.0, 3.0, 4.0 ] );
114-
var N = floor( x.length / 2 );
115106

116-
var v = dsapxsum.ndarray( N, 5.0, x, 2, 1 );
107+
var v = dsapxsum.ndarray( 4, 5.0, x, 2, 1 );
117108
// returns 25.0
118109
```
119110

@@ -139,18 +130,11 @@ var v = dsapxsum.ndarray( N, 5.0, x, 2, 1 );
139130
<!-- eslint no-undef: "error" -->
140131

141132
```javascript
142-
var randu = require( '@stdlib/random/base/randu' );
143-
var round = require( '@stdlib/math/base/special/round' );
144-
var Float32Array = require( '@stdlib/array/float32' );
133+
var discreteUniform = require( '@stdlib/random/base/discrete-uniform' ).factory;
134+
var filledarrayBy = require( '@stdlib/array/filled-by' );
145135
var dsapxsum = require( '@stdlib/blas/ext/base/dsapxsum' );
146136

147-
var x;
148-
var i;
149-
150-
x = new Float32Array( 10 );
151-
for ( i = 0; i < x.length; i++ ) {
152-
x[ i ] = round( randu()*100.0 );
153-
}
137+
var x = filledarrayBy( 10, 'float32', discreteUniform( 0, 100 ) );
154138
console.log( x );
155139

156140
var v = dsapxsum( x.length, 5.0, x, 1 );

Diff for: lib/node_modules/@stdlib/blas/ext/base/dsapxsum/benchmark/benchmark.js

+8-9
Original file line numberDiff line numberDiff line change
@@ -21,14 +21,19 @@
2121
// MODULES //
2222

2323
var bench = require( '@stdlib/bench' );
24-
var randu = require( '@stdlib/random/base/randu' );
24+
var uniform = require( '@stdlib/random/base/uniform' ).factory;
25+
var filledarrayBy = require( '@stdlib/array/filled-by' );
2526
var isnan = require( '@stdlib/math/base/assert/is-nan' );
2627
var pow = require( '@stdlib/math/base/special/pow' );
27-
var Float32Array = require( '@stdlib/array/float32' );
2828
var pkg = require( './../package.json' ).name;
2929
var dsapxsum = require( './../lib/dsapxsum.js' );
3030

3131

32+
// VARIABLES //
33+
34+
var rand = uniform( -10.0, 10.0 );
35+
36+
3237
// FUNCTIONS //
3338

3439
/**
@@ -39,13 +44,7 @@ var dsapxsum = require( './../lib/dsapxsum.js' );
3944
* @returns {Function} benchmark function
4045
*/
4146
function createBenchmark( len ) {
42-
var x;
43-
var i;
44-
45-
x = new Float32Array( len );
46-
for ( i = 0; i < x.length; i++ ) {
47-
x[ i ] = ( randu()*20.0 ) - 10.0;
48-
}
47+
var x = filledarrayBy( len, 'float32', rand );
4948
return benchmark;
5049

5150
function benchmark( b ) {

Diff for: lib/node_modules/@stdlib/blas/ext/base/dsapxsum/benchmark/benchmark.native.js

+4-9
Original file line numberDiff line numberDiff line change
@@ -22,10 +22,10 @@
2222

2323
var resolve = require( 'path' ).resolve;
2424
var bench = require( '@stdlib/bench' );
25-
var randu = require( '@stdlib/random/base/randu' );
25+
var uniform = require( '@stdlib/random/base/uniform' ).factory;
26+
var filledarrayBy = require( '@stdlib/array/filled-by' );
2627
var isnan = require( '@stdlib/math/base/assert/is-nan' );
2728
var pow = require( '@stdlib/math/base/special/pow' );
28-
var Float32Array = require( '@stdlib/array/float32' );
2929
var tryRequire = require( '@stdlib/utils/try-require' );
3030
var pkg = require( './../package.json' ).name;
3131

@@ -36,6 +36,7 @@ var dsapxsum = tryRequire( resolve( __dirname, './../lib/dsapxsum.native.js' ) )
3636
var opts = {
3737
'skip': ( dsapxsum instanceof Error )
3838
};
39+
var rand = uniform( -10.0, 10.0 );
3940

4041

4142
// FUNCTIONS //
@@ -48,13 +49,7 @@ var opts = {
4849
* @returns {Function} benchmark function
4950
*/
5051
function createBenchmark( len ) {
51-
var x;
52-
var i;
53-
54-
x = new Float32Array( len );
55-
for ( i = 0; i < x.length; i++ ) {
56-
x[ i ] = ( randu()*20.0 ) - 10.0;
57-
}
52+
var x = filledarrayBy( len, 'float32', rand );
5853
return benchmark;
5954

6055
function benchmark( b ) {

Diff for: lib/node_modules/@stdlib/blas/ext/base/dsapxsum/benchmark/benchmark.ndarray.js

+8-9
Original file line numberDiff line numberDiff line change
@@ -21,14 +21,19 @@
2121
// MODULES //
2222

2323
var bench = require( '@stdlib/bench' );
24-
var randu = require( '@stdlib/random/base/randu' );
24+
var uniform = require( '@stdlib/random/base/uniform' ).factory;
25+
var filledarrayBy = require( '@stdlib/array/filled-by' );
2526
var isnan = require( '@stdlib/math/base/assert/is-nan' );
2627
var pow = require( '@stdlib/math/base/special/pow' );
27-
var Float32Array = require( '@stdlib/array/float32' );
2828
var pkg = require( './../package.json' ).name;
2929
var dsapxsum = require( './../lib/ndarray.js' );
3030

3131

32+
// VARIABLES //
33+
34+
var rand = uniform( -10.0, 10.0 );
35+
36+
3237
// FUNCTIONS //
3338

3439
/**
@@ -39,13 +44,7 @@ var dsapxsum = require( './../lib/ndarray.js' );
3944
* @returns {Function} benchmark function
4045
*/
4146
function createBenchmark( len ) {
42-
var x;
43-
var i;
44-
45-
x = new Float32Array( len );
46-
for ( i = 0; i < x.length; i++ ) {
47-
x[ i ] = ( randu()*20.0 ) - 10.0;
48-
}
47+
var x = filledarrayBy( len, 'float32', rand );
4948
return benchmark;
5049

5150
function benchmark( b ) {

Diff for: lib/node_modules/@stdlib/blas/ext/base/dsapxsum/benchmark/benchmark.ndarray.native.js

+4-9
Original file line numberDiff line numberDiff line change
@@ -22,10 +22,10 @@
2222

2323
var resolve = require( 'path' ).resolve;
2424
var bench = require( '@stdlib/bench' );
25-
var randu = require( '@stdlib/random/base/randu' );
25+
var uniform = require( '@stdlib/random/base/uniform' ).factory;
26+
var filledarrayBy = require( '@stdlib/array/filled-by' );
2627
var isnan = require( '@stdlib/math/base/assert/is-nan' );
2728
var pow = require( '@stdlib/math/base/special/pow' );
28-
var Float32Array = require( '@stdlib/array/float32' );
2929
var tryRequire = require( '@stdlib/utils/try-require' );
3030
var pkg = require( './../package.json' ).name;
3131

@@ -36,6 +36,7 @@ var dsapxsum = tryRequire( resolve( __dirname, './../lib/ndarray.native.js' ) );
3636
var opts = {
3737
'skip': ( dsapxsum instanceof Error )
3838
};
39+
var rand = uniform( -10.0, 10.0 );
3940

4041

4142
// FUNCTIONS //
@@ -48,13 +49,7 @@ var opts = {
4849
* @returns {Function} benchmark function
4950
*/
5051
function createBenchmark( len ) {
51-
var x;
52-
var i;
53-
54-
x = new Float32Array( len );
55-
for ( i = 0; i < x.length; i++ ) {
56-
x[ i ] = ( randu()*20.0 ) - 10.0;
57-
}
52+
var x = filledarrayBy( len, 'float32', rand );
5853
return benchmark;
5954

6055
function benchmark( b ) {

Diff for: lib/node_modules/@stdlib/blas/ext/base/dsapxsum/docs/repl.txt

+7-11
Original file line numberDiff line numberDiff line change
@@ -4,8 +4,8 @@
44
element and computes the sum using extended accumulation and returning an
55
extended precision result.
66

7-
The `N` and `stride` parameters determine which elements in `x` are accessed
8-
at runtime.
7+
The `N` and `stride` parameters determine which elements in the strided
8+
array are accessed at runtime.
99

1010
Indexing is relative to the first index. To introduce an offset, use a typed
1111
array view.
@@ -40,19 +40,16 @@
4040

4141
// Using `N` and `stride` parameters:
4242
> x = new {{alias:@stdlib/array/float32}}( [ -2.0, 1.0, 1.0, -5.0, 2.0, -1.0 ] );
43-
> var N = {{alias:@stdlib/math/base/special/floor}}( x.length / 2 );
44-
> var stride = 2;
45-
> {{alias}}( N, 5.0, x, stride )
43+
> {{alias}}( 3, 5.0, x, 2 )
4644
16.0
4745

4846
// Using view offsets:
4947
> var x0 = new {{alias:@stdlib/array/float32}}( [ 1.0, -2.0, 3.0, 2.0, 5.0, -1.0 ] );
5048
> var x1 = new {{alias:@stdlib/array/float32}}( x0.buffer, x0.BYTES_PER_ELEMENT*1 );
51-
> N = {{alias:@stdlib/math/base/special/floor}}( x0.length / 2 );
52-
> stride = 2;
53-
> {{alias}}( N, 5.0, x1, stride )
49+
> {{alias}}( 3, 5.0, x1, 2 )
5450
14.0
5551

52+
5653
{{alias}}.ndarray( N, alpha, x, stride, offset )
5754
Adds a constant to each single-precision floating-point strided array
5855
element and computes the sum using extended accumulation and alternative
@@ -92,9 +89,8 @@
9289
16.0
9390

9491
// Using offset parameter:
95-
> var x = new {{alias:@stdlib/array/float32}}( [ 1.0, -2.0, 3.0, 2.0, 5.0, -1.0 ] );
96-
> var N = {{alias:@stdlib/math/base/special/floor}}( x.length / 2 );
97-
> {{alias}}.ndarray( N, 5.0, x, 2, 1 )
92+
> x = new {{alias:@stdlib/array/float32}}( [ 1.0, -2.0, 3.0, 2.0, 5.0, -1.0 ] );
93+
> {{alias}}.ndarray( 3, 5.0, x, 2, 1 )
9894
14.0
9995

10096
See Also

Diff for: lib/node_modules/@stdlib/blas/ext/base/dsapxsum/examples/index.js

+3-10
Original file line numberDiff line numberDiff line change
@@ -18,18 +18,11 @@
1818

1919
'use strict';
2020

21-
var randu = require( '@stdlib/random/base/randu' );
22-
var round = require( '@stdlib/math/base/special/round' );
23-
var Float32Array = require( '@stdlib/array/float32' );
21+
var discreteUniform = require( '@stdlib/random/base/discrete-uniform' ).factory;
22+
var filledarrayBy = require( '@stdlib/array/filled-by' );
2423
var dsapxsum = require( './../lib' );
2524

26-
var x;
27-
var i;
28-
29-
x = new Float32Array( 10 );
30-
for ( i = 0; i < x.length; i++ ) {
31-
x[ i ] = round( randu()*100.0 );
32-
}
25+
var x = filledarrayBy( 10, 'float32', discreteUniform( 0, 100 ) );
3326
console.log( x );
3427

3528
var v = dsapxsum( x.length, 5.0, x, 1 );

Diff for: lib/node_modules/@stdlib/blas/ext/base/dsapxsum/include.gypi

+1-1
Original file line numberDiff line numberDiff line change
@@ -36,7 +36,7 @@
3636

3737
# Source files:
3838
'src_files': [
39-
'<(src_dir)/addon.cpp',
39+
'<(src_dir)/addon.c',
4040
'<!@(node -e "var arr = require(\'@stdlib/utils/library-manifest\')(\'./manifest.json\',{},{\'basedir\':process.cwd(),\'paths\':\'posix\'}).src; for ( var i = 0; i < arr.length; i++ ) { console.log( arr[ i ] ); }")',
4141
],
4242

Diff for: lib/node_modules/@stdlib/blas/ext/base/dsapxsum/lib/dsapxsum.js

+1-2
Original file line numberDiff line numberDiff line change
@@ -38,9 +38,8 @@ var dsapxsumpw = require( '@stdlib/blas/ext/base/dsapxsumpw' );
3838
* var Float32Array = require( '@stdlib/array/float32' );
3939
*
4040
* var x = new Float32Array( [ 1.0, -2.0, 2.0 ] );
41-
* var N = x.length;
4241
*
43-
* var v = dsapxsum( N, 5.0, x, 1 );
42+
* var v = dsapxsum( 3, 5.0, x, 1 );
4443
* // returns 16.0
4544
*/
4645
function dsapxsum( N, alpha, x, stride ) {

Diff for: lib/node_modules/@stdlib/blas/ext/base/dsapxsum/lib/dsapxsum.native.js

+1-2
Original file line numberDiff line numberDiff line change
@@ -38,9 +38,8 @@ var addon = require( './../src/addon.node' );
3838
* var Float32Array = require( '@stdlib/array/float32' );
3939
*
4040
* var x = new Float32Array( [ 1.0, -2.0, 2.0 ] );
41-
* var N = x.length;
4241
*
43-
* var v = dsapxsum( N, 5.0, x, 1 );
42+
* var v = dsapxsum( 3, 5.0, x, 1 );
4443
* // returns 16.0
4544
*/
4645
function dsapxsum( N, alpha, x, stride ) {

Diff for: lib/node_modules/@stdlib/blas/ext/base/dsapxsum/lib/index.js

+2-5
Original file line numberDiff line numberDiff line change
@@ -28,20 +28,17 @@
2828
* var dsapxsum = require( '@stdlib/blas/ext/base/dsapxsum' );
2929
*
3030
* var x = new Float32Array( [ 1.0, -2.0, 2.0 ] );
31-
* var N = x.length;
3231
*
33-
* var v = dsapxsum( N, 5.0, x, 1 );
32+
* var v = dsapxsum( 3, 5.0, x, 1 );
3433
* // returns 16.0
3534
*
3635
* @example
3736
* var Float32Array = require( '@stdlib/array/float32' );
38-
* var floor = require( '@stdlib/math/base/special/floor' );
3937
* var dsapxsum = require( '@stdlib/blas/ext/base/dsapxsum' );
4038
*
4139
* var x = new Float32Array( [ 2.0, 1.0, 2.0, -2.0, -2.0, 2.0, 3.0, 4.0 ] );
42-
* var N = floor( x.length / 2 );
4340
*
44-
* var v = dsapxsum.ndarray( N, 5.0, x, 2, 1 );
41+
* var v = dsapxsum.ndarray( 4, 5.0, x, 2, 1 );
4542
* // returns 25.0
4643
*/
4744

Diff for: lib/node_modules/@stdlib/blas/ext/base/dsapxsum/lib/ndarray.js

+1-3
Original file line numberDiff line numberDiff line change
@@ -37,12 +37,10 @@ var dsapxsumpw = require( '@stdlib/blas/ext/base/dsapxsumpw' ).ndarray;
3737
*
3838
* @example
3939
* var Float32Array = require( '@stdlib/array/float32' );
40-
* var floor = require( '@stdlib/math/base/special/floor' );
4140
*
4241
* var x = new Float32Array( [ 2.0, 1.0, 2.0, -2.0, -2.0, 2.0, 3.0, 4.0 ] );
43-
* var N = floor( x.length / 2 );
4442
*
45-
* var v = dsapxsum( N, 5.0, x, 2, 1 );
43+
* var v = dsapxsum( 4, 5.0, x, 2, 1 );
4644
* // returns 25.0
4745
*/
4846
function dsapxsum( N, alpha, x, stride, offset ) {

0 commit comments

Comments
 (0)