-
-
Notifications
You must be signed in to change notification settings - Fork 578
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 blas/base/zswap
#2075
feat: add blas/base/zswap
#2075
Conversation
If applied, this commit will add the package `blas/base/zswap`
Refactoring the code to use |
@performant23 Looks like you are missing the |
Signed-off-by: Athan <kgryte@gmail.com>
/stdlib update-copyright-years |
lib/node_modules/@stdlib/blas/base/zswap/benchmark/fortran/benchmark.length.f
Outdated
Show resolved
Hide resolved
lib/node_modules/@stdlib/blas/base/zswap/benchmark/fortran/benchmark.length.f
Outdated
Show resolved
Hide resolved
lib/node_modules/@stdlib/blas/base/zswap/benchmark/fortran/benchmark.length.f
Show resolved
Hide resolved
Signed-off-by: Athan <kgryte@gmail.com>
lib/node_modules/@stdlib/blas/base/zswap/benchmark/fortran/benchmark.length.f
Outdated
Show resolved
Hide resolved
Signed-off-by: Athan <kgryte@gmail.com>
Signed-off-by: Athan <kgryte@gmail.com>
Signed-off-by: Athan <kgryte@gmail.com>
Signed-off-by: Athan <kgryte@gmail.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, @performant23!
Ah, wait. You need to add the missing |
@performant23 As you are on Windows, you were testing against the C implementation which did not have any issues. However, the Fortran implementation did have bugs, and the Fortran implementation was what was being tested against in CI. The issues should now be addressed. |
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.
This LGTM. Thanks, @performant23! I'll merge once CI passes.
If applied, this commit will add the package
blas/base/zswap
Description
This pull request:
C
andFortran
and obtains Fortran reference from this pageRelated Issues
This pull request:
blas/base/zswap
#2047Questions
No.
Other
No.
Checklist
@stdlib-js/reviewers