-
-
Notifications
You must be signed in to change notification settings - Fork 908
feat: add C ndarray
interface and refactor implementation for stats/base/dsmeanpw
#4338
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
Coverage Report
The above coverage report was generated for the changes in this PR. |
…to ndarray-dsmeanpw
Signed-off-by: Neeraj Pathak <[email protected]>
/stdlib update-copyright-years |
@Neerajpathak07, slash command not recognized. Please use |
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.
Other than updating to use dssumpw
explicitly, this PR looks good.
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
…s/base/dsmeanpw` PR-URL: stdlib-js#4338 Reviewed-by: Athan Reines <[email protected]> Co-authored-by: stdlib-bot <[email protected]>
…s/base/dsmeanpw` PR-URL: stdlib-js#4338 Reviewed-by: Athan Reines <[email protected]> Co-authored-by: stdlib-bot <[email protected]>
…s/base/dsmeanpw` PR-URL: stdlib-js#4338 Reviewed-by: Athan Reines <[email protected]> Co-authored-by: stdlib-bot <[email protected]>
…s/base/dsmeanpw` PR-URL: stdlib-js#4338 Reviewed-by: Athan Reines <[email protected]> Co-authored-by: stdlib-bot <[email protected]>
Resolves none
Description
This pull request:
ndarray
interface forstats/base/dsmeanpw
Related Issues
This pull request:
Questions
No.
Other
No.
Checklist
@stdlib-js/reviewers