Skip to content

Commit 24cd22d

Browse files
committed
refactor: call into generalized utility
--- 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: na - task: lint_repl_help status: na - 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: 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: na - task: lint_license_headers status: passed ---
1 parent 622fb73 commit 24cd22d

File tree

5 files changed

+11
-70
lines changed

5 files changed

+11
-70
lines changed

lib/node_modules/@stdlib/ndarray/base/ternary-tiling-block-size/README.md

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -83,7 +83,7 @@ The function supports the following arguments:
8383
```javascript
8484
var dtypes = require( '@stdlib/ndarray/dtypes' );
8585
var cartesianPower = require( '@stdlib/array/base/cartesian-power' );
86-
var promotionRules = require( '@stdlib/ndarray/promotion-rules' );
86+
var promoteDataTypes = require( '@stdlib/ndarray/base/promote-dtypes' );
8787
var ternaryBlockSize = require( '@stdlib/ndarray/base/ternary-tiling-block-size' );
8888

8989
// Generate a list of input ndarray dtype triplets:
@@ -95,8 +95,8 @@ var b;
9595
var i;
9696
console.log( 'block_size, xdtype, ydtype, zdtype, wdtype' );
9797
for ( i = 0; i < dt.length; i++ ) {
98-
t = promotionRules.apply( null, dt[ i ] );
99-
dt[ i ].push( ( t === -1 ) ? 'generic' : t );
98+
t = promoteDataTypes( dt[ i ] );
99+
dt[ i ].push( ( t === null ) ? 'generic' : t );
100100
b = ternaryBlockSize.apply( null, dt[ i ] );
101101
console.log( '%d, %s, %s, %s, %s', b, dt[i][0], dt[i][1], dt[i][2], dt[i][3] );
102102
}

lib/node_modules/@stdlib/ndarray/base/ternary-tiling-block-size/examples/index.js

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

2121
var dtypes = require( '@stdlib/ndarray/dtypes' );
2222
var cartesianPower = require( '@stdlib/array/base/cartesian-power' );
23-
var promotionRules = require( '@stdlib/ndarray/promotion-rules' );
23+
var promoteDataTypes = require( '@stdlib/ndarray/base/promote-dtypes' );
2424
var ternaryBlockSize = require( './../lib' );
2525

2626
// Generate a list of input ndarray dtype triplets:
@@ -32,8 +32,8 @@ var b;
3232
var i;
3333
console.log( 'block_size, xdtype, ydtype, zdtype, wdtype' );
3434
for ( i = 0; i < dt.length; i++ ) {
35-
t = promotionRules.apply( null, dt[ i ] );
36-
dt[ i ].push( ( t === -1 ) ? 'generic' : t );
35+
t = promoteDataTypes( dt[ i ] );
36+
dt[ i ].push( ( t === null ) ? 'generic' : t );
3737
b = ternaryBlockSize.apply( null, dt[ i ] );
3838
console.log( '%d, %s, %s, %s, %s', b, dt[i][0], dt[i][1], dt[i][2], dt[i][3] );
3939
}

lib/node_modules/@stdlib/ndarray/base/ternary-tiling-block-size/lib/defaults.js

Lines changed: 0 additions & 34 deletions
This file was deleted.

lib/node_modules/@stdlib/ndarray/base/ternary-tiling-block-size/lib/main.js

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

2121
// MODULES //
2222

23-
var bytesPerElement = require( '@stdlib/ndarray/base/bytes-per-element' );
24-
var defaults = require( './defaults.js' );
23+
var blockSize = require( '@stdlib/ndarray/base/tiling-block-size' );
2524

2625

2726
// MAIN //
@@ -40,31 +39,7 @@ var defaults = require( './defaults.js' );
4039
* // returns <number>
4140
*/
4241
function ternaryBlockSize( dtypeX, dtypeY, dtypeZ, dtypeW ) {
43-
var nbx;
44-
var nby;
45-
var nbz;
46-
var nbw;
47-
var max;
48-
49-
nbx = bytesPerElement( dtypeX );
50-
nby = bytesPerElement( dtypeY );
51-
nbz = bytesPerElement( dtypeZ );
52-
nbw = bytesPerElement( dtypeW );
53-
if ( nbx === null || nby === null || nbz === null || nbw === null ) { // e.g., "generic" arrays
54-
return defaults.BLOCK_SIZE_IN_ELEMENTS;
55-
}
56-
// Find the largest element size among all four arrays...
57-
max = nbx;
58-
if ( nby > max ) {
59-
max = nby;
60-
}
61-
if ( nbz > max ) {
62-
max = nbz;
63-
}
64-
if ( nbw > max ) {
65-
max = nbw;
66-
}
67-
return ( defaults.BLOCK_SIZE_IN_BYTES/max )|0; // asm type annotation
42+
return blockSize( [ dtypeX, dtypeY, dtypeZ, dtypeW ] );
6843
}
6944

7045

lib/node_modules/@stdlib/ndarray/base/ternary-tiling-block-size/test/test.js

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -24,7 +24,7 @@ var tape = require( 'tape' );
2424
var isPositiveInteger = require( '@stdlib/assert/is-positive-integer' ).isPrimitive;
2525
var dtypes = require( '@stdlib/ndarray/dtypes' );
2626
var cartesianPower = require( '@stdlib/array/base/cartesian-power' );
27-
var promotionRules = require( '@stdlib/ndarray/promotion-rules' );
27+
var promoteDataTypes = require( '@stdlib/ndarray/base/promote-dtypes' );
2828
var ternaryBlockSize = require( './../lib' );
2929

3030

@@ -48,8 +48,8 @@ tape( 'the function returns a block size', function test( t ) {
4848
dt = cartesianPower( dtypes(), 3 );
4949

5050
for ( i = 0; i < dt.length; i++ ) {
51-
pt = promotionRules.apply( null, dt[ i ] );
52-
dt[ i ].push( ( pt === -1 ) ? 'generic' : pt );
51+
pt = promoteDataTypes( dt[ i ] );
52+
dt[ i ].push( ( pt === null ) ? 'generic' : pt );
5353
v = ternaryBlockSize( dt[ i ][ 0 ], dt[ i ][ 1 ], dt[ i ][ 2 ], dt[ i ][ 3 ] ); // eslint-disable-line max-len
5454
t.strictEqual( isPositiveInteger( v ), true, 'returns a positive integer when provided ('+dt[ i ].join( ', ' )+')' );
5555
}

0 commit comments

Comments
 (0)