-
-
Notifications
You must be signed in to change notification settings - Fork 447
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
feat: add lapack/base/dlacpy
#2548
Conversation
@Pranavchiku Given the recent change in approach for the |
I marked it as draft, I am refactoring implementation. |
Applied changes, I think this PR can have a review. |
lib/node_modules/@stdlib/lapack/base/dlacpy/benchmark/benchmark.ndarray.js
Show resolved
Hide resolved
Signed-off-by: Pranav <85227306+Pranavchiku@users.noreply.github.com>
Signed-off-by: Pranav <85227306+Pranavchiku@users.noreply.github.com>
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.
LGTM. Thanks for working on this, @Pranavchiku.
Note that I had to make a number of clean-up changes. And also, wrt testing, two things:
- Testing for approximate equality doesn't make sense for this package.
- It isn't enough to test on wide matrices, but not long matrices. We need to test on both to ensure that loop limits work appropriately.
PR-URL: stdlib-js#2548 Co-authored-by: Athan Reines <kgryte@gmail.com> Reviewed-by: Athan Reines <kgryte@gmail.com> Signed-off-by: Pranav Goswami <goswami.4@iitj.ac.in>
Description
This pull request adds JS implementation for
lapack/blas/dlacpy
Related Issues
No.
Questions
No.
Other
No.
Checklist
@stdlib-js/reviewers