Repository navigation
feat: add blas/ext/base/ndarray/dapx - #14424
Conversation
Coverage Report
The above coverage report was generated for the changes in this PR. |
0PrashantYadav0
left a comment
There was a problem hiding this comment.
I think these changes are required as of now.
|
I've applied the suggestions, but the CI is failing with a data type cast error. It seems that passing a plain array to To fix this, should I wrap the input in a Or would you prefer I add |
|
Agreed, var x = array( new Float64Array( [ -2.0, 1.0, 3.0, -5.0, 4.0, 0.0, -1.0, -3.0 ] ) ); makes much more sense. I considered this approach initially but held off because it requires multiple passes. Before we move forward with this change, let's get @kgryte 's review. |
Resolves a part of #14404
Description
This pull request:
ndarrayAPI wrapper for@stdlib/blas/ext/base/dapx(adds a scalar constant to each element in a one-dimensional double-precision floating-point ndarray).@stdlib/blas/ext/basepackages.Related Issues
This pull request has the following related issues:
ndarrayinterface variants for missing modules in@stdlib/blas/ext/base#14404Questions
No.
Other
No.
Checklist
AI Assistance
If you answered "yes" above, how did you use AI assistance?
Disclosure
I used an AI assistant to research existing
ndarraypackages to ensure consistency with canonical example arrays (e.g.,[ -2.0, 1.0, 3.0, -5.0, 4.0, 0.0, -1.0, -3.0 ]for scalar operations) and to help scaffold the tests and benchmarks. All changes were manually verified and aligned with theblas/ext/base/dapxreference tests.@stdlib-js/reviewers