-
-
Notifications
You must be signed in to change notification settings - Fork 836
refactor: update blas/ext/base/dnannsumkbn2
to follow current project…
#2063
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
blas/ext/base/dnannsumkbn2
to follow current project…
lib/node_modules/@stdlib/blas/ext/base/dnannsumkbn2/docs/repl.txt
Outdated
Show resolved
Hide resolved
lib/node_modules/@stdlib/blas/ext/base/dnannsumkbn2/lib/dnannsumkbn2.native.js
Outdated
Show resolved
Hide resolved
Signed-off-by: Philipp Burckhardt <[email protected]>
@kgryte @Planeshifter I am getting errors in running tests and constantly getting error: 29 tests complete (38ms) Running test: lib/node_modules/@stdlib/blas/ext/base/dnannsumkbn2/test/test.dnannsumkbn2.native.js RangeError: invalid argument. Fourth argument has insufficient elements based on the associated stride and the number of indexed elements. can u help me with this? |
lib/node_modules/@stdlib/blas/ext/base/dnannsumkbn2/src/addon.c
Outdated
Show resolved
Hide resolved
lib/node_modules/@stdlib/blas/ext/base/dnannsumkbn2/src/addon.c
Outdated
Show resolved
Hide resolved
lib/node_modules/@stdlib/blas/ext/base/dnannsumkbn2/lib/ndarray.native.js
Outdated
Show resolved
Hide resolved
@kgryte @Planeshifter I'm delighted to inform you that I've successfully resolved the error we encountered earlier. I want to extend my heartfelt gratitude to @kgryte for your invaluable guidance throughout this process. I'm now ready to proceed with the review. Thank you once again for your assistance. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you, @vr-varad!
Description
refactor: update blas/ext/base/dnannsumkbn2 to follow current project conventions
Related Issues
This pull request:
blas/ext/base/dnannsumkbn2
to follow current project conventions #1480Questions
No.
Other
No.
Checklist
@stdlib-js/reviewers