Skip to content

feat: add C ndarray interface and refactor implementation for stats/base/dnanstdev #4252

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

Closed

Conversation

Neerajpathak07
Copy link
Member

Resolves none.

Description

What is the purpose of this pull request?

This pull request:

  • adds C ndarray interface for stats/base/dnanstdev
  • refactor JavaScript implementation
  • refactor examples, tests, docs and benchmarks in manifest.json

Related Issues

Does this pull request have any related issues?

This pull request:

  • resolves none

Questions

Any questions for reviewers of this pull request?

No.

Other

Any other information relevant to this pull request? This may include screenshots, references, and/or implementation notes.

No.

Checklist

Please ensure the following tasks are completed before submitting this pull request.


@stdlib-js/reviewers

@stdlib-bot stdlib-bot added the Statistics Issue or pull request related to statistical functionality. label Dec 26, 2024
@0PrashantYadav0
Copy link
Member

0PrashantYadav0 commented Feb 18, 2025

@Neerajpathak07 Are you still working on this if not please close this PR because i need tha ndarray implementation of this package.

* @return output value
*/
double API_SUFFIX(stdlib_strided_dnanstdev_ndarray)( const CBLAS_INT N, const double correction, const double *X, const CBLAS_INT strideX, const CBLAS_INT offsetX ) {
return API_SUFFIX(stdlib_strided_dnanstdevpn_ndarray)( N, correction, X, strideX, offsetX );
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@0PrashantYadav0 I am working on this package and for that we need to refactor dnanstdevpn which in turn depends on dnanvariancepn till we have those PR's this will be on draft.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ok Thanks for updated will open a draft PR for this and open that we this is merged.

@kgryte
Copy link
Member

kgryte commented May 11, 2025

Ref: #6970

@kgryte kgryte added the Duplicate This issue or pull request already exists. label May 11, 2025
@kgryte kgryte closed this May 11, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Duplicate This issue or pull request already exists. Statistics Issue or pull request related to statistical functionality.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants