Skip to content

Commit f95e706

Browse files
rxbryanPranavchikuPlaneshifter
authored
refactor: update blas/ext/base/scusumpw to follow current project conventions
PR-URL: #1929 Closes: #1520 --------- Signed-off-by: Pranav <85227306+Pranavchiku@users.noreply.github.com> Signed-off-by: Philipp Burckhardt <pburckhardt@outlook.com> Co-authored-by: Pranav <85227306+Pranavchiku@users.noreply.github.com> Co-authored-by: Philipp Burckhardt <pburckhardt@outlook.com> Reviewed-by: Pranav Goswami <goswami.4@iitj.ac.in> Reviewed-by: Philipp Burckhardt <pburckhardt@outlook.com>
1 parent 548f903 commit f95e706

File tree

17 files changed

+198
-320
lines changed

17 files changed

+198
-320
lines changed

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

Lines changed: 8 additions & 23 deletions
Original file line numberDiff line numberDiff line change
@@ -65,18 +65,15 @@ The function has the following parameters:
6565
- **y**: output [`Float32Array`][@stdlib/array/float32].
6666
- **strideY**: index increment for `y`.
6767

68-
The `N` and `stride` parameters determine which elements in `x` and `y` are accessed at runtime. For example, to compute the cumulative sum of every other element in `x`,
68+
The `N` and `stride` parameters determine which elements in the strided arrays are accessed at runtime. For example, to compute the cumulative sum of every other element in `x`,
6969

7070
```javascript
7171
var Float32Array = require( '@stdlib/array/float32' );
72-
var floor = require( '@stdlib/math/base/special/floor' );
7372

7473
var x = new Float32Array( [ 1.0, 2.0, 2.0, -7.0, -2.0, 3.0, 4.0, 2.0 ] );
7574
var y = new Float32Array( x.length );
7675

77-
var N = floor( x.length / 2 );
78-
79-
var v = scusumpw( N, 0.0, x, 2, y, 1 );
76+
var v = scusumpw( 4, 0.0, x, 2, y, 1 );
8077
// y => <Float32Array>[ 1.0, 3.0, 1.0, 5.0, 0.0, 0.0, 0.0, 0.0 ]
8178
```
8279

@@ -86,7 +83,6 @@ Note that indexing is relative to the first index. To introduce an offset, use [
8683

8784
```javascript
8885
var Float32Array = require( '@stdlib/array/float32' );
89-
var floor = require( '@stdlib/math/base/special/floor' );
9086

9187
// Initial arrays...
9288
var x0 = new Float32Array( [ 2.0, 1.0, 2.0, -2.0, -2.0, 2.0, 3.0, 4.0 ] );
@@ -96,9 +92,7 @@ var y0 = new Float32Array( x0.length );
9692
var x1 = new Float32Array( x0.buffer, x0.BYTES_PER_ELEMENT*1 ); // start at 2nd element
9793
var y1 = new Float32Array( y0.buffer, y0.BYTES_PER_ELEMENT*3 ); // start at 4th element
9894

99-
var N = floor( x0.length / 2 );
100-
101-
scusumpw( N, 0.0, x1, -2, y1, 1 );
95+
scusumpw( 4, 0.0, x1, -2, y1, 1 );
10296
// y0 => <Float32Array>[ 0.0, 0.0, 0.0, 4.0, 6.0, 4.0, 5.0, 0.0 ]
10397
```
10498

@@ -125,14 +119,11 @@ While [`typed array`][mdn-typed-array] views mandate a view offset based on the
125119

126120
```javascript
127121
var Float32Array = require( '@stdlib/array/float32' );
128-
var floor = require( '@stdlib/math/base/special/floor' );
129122

130123
var x = new Float32Array( [ 2.0, 1.0, 2.0, -2.0, -2.0, 2.0, 3.0, 4.0 ] );
131124
var y = new Float32Array( x.length );
132125

133-
var N = floor( x.length / 2 );
134-
135-
scusumpw.ndarray( N, 0.0, x, 2, 1, y, -1, y.length-1 );
126+
scusumpw.ndarray( 4, 0.0, x, 2, 1, y, -1, y.length-1 );
136127
// y => <Float32Array>[ 0.0, 0.0, 0.0, 0.0, 5.0, 1.0, -1.0, 1.0 ]
137128
```
138129

@@ -158,20 +149,14 @@ scusumpw.ndarray( N, 0.0, x, 2, 1, y, -1, y.length-1 );
158149
<!-- eslint no-undef: "error" -->
159150

160151
```javascript
161-
var randu = require( '@stdlib/random/base/randu' );
162-
var round = require( '@stdlib/math/base/special/round' );
152+
var discreteUniform = require( '@stdlib/random/base/discrete-uniform' ).factory;
153+
var filledarrayBy = require( '@stdlib/array/filled-by' );
163154
var Float32Array = require( '@stdlib/array/float32' );
164155
var scusumpw = require( '@stdlib/blas/ext/base/scusumpw' );
165156

166-
var y;
167-
var x;
168-
var i;
157+
var x = filledarrayBy( 10, 'float32', discreteUniform( 0, 100 ) );
158+
var y = new Float32Array( x.length );
169159

170-
x = new Float32Array( 10 );
171-
y = new Float32Array( x.length );
172-
for ( i = 0; i < x.length; i++ ) {
173-
x[ i ] = round( randu()*100.0 );
174-
}
175160
console.log( x );
176161
console.log( y );
177162

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

Lines changed: 9 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -21,14 +21,20 @@
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 isnanf = require( '@stdlib/math/base/assert/is-nanf' );
2627
var pow = require( '@stdlib/math/base/special/pow' );
2728
var Float32Array = require( '@stdlib/array/float32' );
2829
var pkg = require( './../package.json' ).name;
2930
var scusumpw = require( './../lib/scusumpw.js' );
3031

3132

33+
// VARIABLES //
34+
35+
var rand = uniform( -10.0, 10.0 );
36+
37+
3238
// FUNCTIONS //
3339

3440
/**
@@ -39,15 +45,9 @@ var scusumpw = require( './../lib/scusumpw.js' );
3945
* @returns {Function} benchmark function
4046
*/
4147
function createBenchmark( len ) {
42-
var y;
43-
var x;
44-
var i;
48+
var y = new Float32Array( len );
49+
var x = filledarrayBy( len, 'float32', rand );
4550

46-
x = new Float32Array( len );
47-
y = new Float32Array( len );
48-
for ( i = 0; i < x.length; i++ ) {
49-
x[ i ] = ( randu()*20.0 ) - 10.0;
50-
}
5151
return benchmark;
5252

5353
function benchmark( b ) {

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

Lines changed: 5 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -22,7 +22,8 @@
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 isnanf = require( '@stdlib/math/base/assert/is-nanf' );
2728
var pow = require( '@stdlib/math/base/special/pow' );
2829
var Float32Array = require( '@stdlib/array/float32' );
@@ -36,6 +37,7 @@ var scusumpw = tryRequire( resolve( __dirname, './../lib/scusumpw.native.js' ) )
3637
var opts = {
3738
'skip': ( scusumpw instanceof Error )
3839
};
40+
var rand = uniform( -10.0, 10.0 );
3941

4042

4143
// FUNCTIONS //
@@ -48,15 +50,9 @@ var opts = {
4850
* @returns {Function} benchmark function
4951
*/
5052
function createBenchmark( len ) {
51-
var x;
52-
var y;
53-
var i;
53+
var x = filledarrayBy( len, 'float32', rand );
54+
var y = new Float32Array( len );
5455

55-
x = new Float32Array( len );
56-
y = new Float32Array( len );
57-
for ( i = 0; i < x.length; i++ ) {
58-
x[ i ] = ( randu()*20.0 ) - 10.0;
59-
}
6056
return benchmark;
6157

6258
function benchmark( b ) {

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

Lines changed: 9 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -21,14 +21,20 @@
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 isnanf = require( '@stdlib/math/base/assert/is-nanf' );
2627
var pow = require( '@stdlib/math/base/special/pow' );
2728
var Float32Array = require( '@stdlib/array/float32' );
2829
var pkg = require( './../package.json' ).name;
2930
var scusumpw = require( './../lib/ndarray.js' );
3031

3132

33+
// VARIABLES //
34+
35+
var rand = uniform( -10.0, 10.0 );
36+
37+
3238
// FUNCTIONS //
3339

3440
/**
@@ -39,15 +45,9 @@ var scusumpw = require( './../lib/ndarray.js' );
3945
* @returns {Function} benchmark function
4046
*/
4147
function createBenchmark( len ) {
42-
var x;
43-
var y;
44-
var i;
48+
var x = filledarrayBy( len, 'float32', rand );
49+
var y = new Float32Array( len );
4550

46-
x = new Float32Array( len );
47-
y = new Float32Array( len );
48-
for ( i = 0; i < x.length; i++ ) {
49-
x[ i ] = ( randu()*20.0 ) - 10.0;
50-
}
5151
return benchmark;
5252

5353
function benchmark( b ) {

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

Lines changed: 5 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -22,7 +22,8 @@
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 isnanf = require( '@stdlib/math/base/assert/is-nanf' );
2728
var pow = require( '@stdlib/math/base/special/pow' );
2829
var Float32Array = require( '@stdlib/array/float32' );
@@ -36,6 +37,7 @@ var scusumpw = tryRequire( resolve( __dirname, './../lib/ndarray.native.js' ) );
3637
var opts = {
3738
'skip': ( scusumpw instanceof Error )
3839
};
40+
var rand = uniform( -10.0, 10.0 );
3941

4042

4143
// FUNCTIONS //
@@ -48,15 +50,9 @@ var opts = {
4850
* @returns {Function} benchmark function
4951
*/
5052
function createBenchmark( len ) {
51-
var x;
52-
var y;
53-
var i;
53+
var x = filledarrayBy( len, 'float32', rand );
54+
var y = new Float32Array( len );
5455

55-
x = new Float32Array( len );
56-
y = new Float32Array( len );
57-
for ( i = 0; i < x.length; i++ ) {
58-
x[ i ] = ( randu()*20.0 ) - 10.0;
59-
}
6056
return benchmark;
6157

6258
function benchmark( b ) {

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

Lines changed: 7 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -3,8 +3,8 @@
33
Computes the cumulative sum of single-precision floating-point strided array
44
elements using pairwise summation.
55

6-
The `N` and `stride` parameters determine which elements in `x` and `y` are
7-
accessed at runtime.
6+
The `N` and stride parameters determine which elements in the strided arrays
7+
are accessed at runtime.
88

99
Indexing is relative to the first index. To introduce an offset, use a typed
1010
array view.
@@ -44,24 +44,23 @@
4444
> {{alias}}( x.length, 0.0, x, 1, y, 1 )
4545
<Float32Array>[ 1.0, -1.0, 1.0 ]
4646

47-
// Using `N` and `stride` parameters:
47+
// Using `N` and stride parameters:
4848
> x = new {{alias:@stdlib/array/float32}}( [ -2.0, 1.0, 1.0, -5.0, 2.0, -1.0 ] );
4949
> y = new {{alias:@stdlib/array/float32}}( x.length );
50-
> var N = {{alias:@stdlib/math/base/special/floor}}( x.length / 2 );
51-
> {{alias}}( N, 0.0, x, 2, y, 2 )
50+
> {{alias}}( 3, 0.0, x, 2, y, 2 )
5251
<Float32Array>[ -2.0, 0.0, -1.0, 0.0, 1.0, 0.0 ]
5352

5453
// Using view offsets:
5554
> var x0 = new {{alias:@stdlib/array/float32}}( [ 1.0, -2.0, 3.0, 2.0, 5.0, -1.0 ] );
5655
> var y0 = new {{alias:@stdlib/array/float32}}( x0.length );
5756
> var x1 = new {{alias:@stdlib/array/float32}}( x0.buffer, x0.BYTES_PER_ELEMENT*1 );
5857
> var y1 = new {{alias:@stdlib/array/float32}}( y0.buffer, y0.BYTES_PER_ELEMENT*3 );
59-
> N = {{alias:@stdlib/math/base/special/floor}}( x0.length / 2 );
60-
> {{alias}}( N, 0.0, x1, 2, y1, 1 )
58+
> {{alias}}( 3, 0.0, x1, 2, y1, 1 )
6159
<Float32Array>[ -2.0, 0.0, -1.0 ]
6260
> y0
6361
<Float32Array>[ 0.0, 0.0, 0.0, -2.0, 0.0, -1.0 ]
6462

63+
6564
{{alias}}.ndarray( N, sum, x, strideX, offsetX, y, strideY, offsetY )
6665
Computes the cumulative sum of single-precision floating-point strided array
6766
elements using pairwise summation and alternative indexing semantics.
@@ -112,8 +111,7 @@
112111
// Advanced indexing:
113112
> x = new {{alias:@stdlib/array/float32}}( [ 1.0, -2.0, 3.0, 2.0, 5.0, -1.0 ] );
114113
> y = new {{alias:@stdlib/array/float32}}( x.length );
115-
> var N = {{alias:@stdlib/math/base/special/floor}}( x.length / 2 );
116-
> {{alias}}.ndarray( N, 0.0, x, 2, 1, y, -1, y.length-1 )
114+
> {{alias}}.ndarray( 3, 0.0, x, 2, 1, y, -1, y.length-1 )
117115
<Float32Array>[ 0.0, 0.0, 0.0, -1.0, 0.0, -2.0 ]
118116

119117
See Also

lib/node_modules/@stdlib/blas/ext/base/scusumpw/examples/index.js

Lines changed: 4 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -18,20 +18,14 @@
1818

1919
'use strict';
2020

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

26-
var y;
27-
var x;
28-
var i;
26+
var x = filledarrayBy( 10, 'float32', discreteUniform( 0, 100 ) );
27+
var y = new Float32Array( x.length );
2928

30-
x = new Float32Array( 10 );
31-
y = new Float32Array( x.length );
32-
for ( i = 0; i < x.length; i++ ) {
33-
x[ i ] = round( randu()*100.0 );
34-
}
3529
console.log( x );
3630
console.log( y );
3731

lib/node_modules/@stdlib/blas/ext/base/scusumpw/include.gypi

Lines changed: 1 addition & 1 deletion
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

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

Lines changed: 9 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -20,7 +20,8 @@
2020

2121
// MODULES //
2222

23-
var Float32Array = require( '@stdlib/array/float32' );
23+
var minViewBufferIndex = require( '@stdlib/strided/base/min-view-buffer-index' );
24+
var offsetView = require( '@stdlib/strided/base/offset-view' );
2425
var addon = require( './scusumpw.native.js' );
2526

2627

@@ -53,14 +54,13 @@ var addon = require( './scusumpw.native.js' );
5354
function scusumpw( N, sum, x, strideX, offsetX, y, strideY, offsetY ) {
5455
var viewX;
5556
var viewY;
56-
if ( strideX < 0 ) {
57-
offsetX += (N-1) * strideX;
58-
}
59-
if ( strideY < 0 ) {
60-
offsetY += (N-1) * strideY;
61-
}
62-
viewX = new Float32Array( x.buffer, x.byteOffset+(x.BYTES_PER_ELEMENT*offsetX), x.length-offsetX ); // eslint-disable-line max-len
63-
viewY = new Float32Array( y.buffer, y.byteOffset+(y.BYTES_PER_ELEMENT*offsetY), y.length-offsetY ); // eslint-disable-line max-len
57+
58+
offsetX = minViewBufferIndex( N, strideX, offsetX );
59+
offsetY = minViewBufferIndex( N, strideY, offsetY );
60+
61+
viewX = offsetView( x, offsetX );
62+
viewY = offsetView( y, offsetY );
63+
6464
addon( N, sum, viewX, strideX, viewY, strideY );
6565
return y;
6666
}

0 commit comments

Comments
 (0)