Repository navigation
feat: add blas/ext/base/ndarray/sfill - #9094
Amansingh0807 wants to merge 6 commits into
Conversation
Coverage Report
The above coverage report was generated for the changes in this PR. |
|
Thanks @kgryte. I have applied all the requested changes. The review really gave me a much clearer understanding of the architectural patterns and the 'why' behind them. I have updated this PR and propagated the same standards to all my related PRs (eg : sapx, gapx and etc.) as well. Ready for review. P.S. I am aware of RFC #8647 regarding string interpolation. For now, I stuck to the dsorthp pattern (concatenation), but let me know if you would prefer I migrate these benchmarks to use @stdlib/string/format in this PR and related ones. |
|
@Amansingh0807 I suggest going ahead and using |
|
Acknowledged, thanks @kgryte. I’ve migrated the benchmarks to use |
|
Ref: #15307 |
|
Thank you for working on this pull request. However, we cannot accept your contribution as the issue this pull request seeks to resolve has already been addressed in a different pull request or commit. Thank you again for your interest in stdlib, and we look forward to reviewing your future contributions. |
type: pre_commit_static_analysis_report
description: Results of running static analysis checks when committing changes.
report:
Resolves None.
Description
This pull request:
blas/ext/base/ndarray/sfillRelated Issues
This pull request has the following related issues:
Questions
No.
Other
No.
Checklist
AI Assistance
If you answered "yes" above, how did you use AI assistance?
Disclosure
@stdlib-js/reviewers