-
-
Notifications
You must be signed in to change notification settings - Fork 822
feat: add C implementation for stats/base/dists/bernoulli/pmf
#3974
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. |
/stdlib update-copyright-years |
672d072
to
bebfdee
Compare
2eb4571
to
8b7894f
Compare
/stdlib update-copyright-years |
Signed-off-by: Neeraj Pathak <[email protected]>
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.
Left a bunch of comments that should be addressed before this PR can land.
lib/node_modules/@stdlib/stats/base/dists/bernoulli/pmf/README.md
Outdated
Show resolved
Hide resolved
lib/node_modules/@stdlib/stats/base/dists/bernoulli/pmf/README.md
Outdated
Show resolved
Hide resolved
lib/node_modules/@stdlib/stats/base/dists/bernoulli/pmf/benchmark/benchmark.js
Outdated
Show resolved
Hide resolved
lib/node_modules/@stdlib/stats/base/dists/bernoulli/pmf/benchmark/benchmark.js
Outdated
Show resolved
Hide resolved
lib/node_modules/@stdlib/stats/base/dists/bernoulli/pmf/benchmark/benchmark.native.js
Outdated
Show resolved
Hide resolved
lib/node_modules/@stdlib/stats/base/dists/bernoulli/pmf/lib/native.js
Outdated
Show resolved
Hide resolved
lib/node_modules/@stdlib/stats/base/dists/bernoulli/pmf/src/main.c
Outdated
Show resolved
Hide resolved
lib/node_modules/@stdlib/stats/base/dists/bernoulli/pmf/src/main.c
Outdated
Show resolved
Hide resolved
lib/node_modules/@stdlib/stats/base/dists/bernoulli/pmf/src/main.c
Outdated
Show resolved
Hide resolved
lib/node_modules/@stdlib/stats/base/dists/bernoulli/pmf/test/test.native.js
Outdated
Show resolved
Hide resolved
@Planeshifter Updated the changes here and elsewhere!! |
lib/node_modules/@stdlib/stats/base/dists/bernoulli/pmf/README.md
Outdated
Show resolved
Hide resolved
lib/node_modules/@stdlib/stats/base/dists/bernoulli/pmf/benchmark/benchmark.js
Outdated
Show resolved
Hide resolved
lib/node_modules/@stdlib/stats/base/dists/bernoulli/pmf/benchmark/benchmark.js
Outdated
Show resolved
Hide resolved
lib/node_modules/@stdlib/stats/base/dists/bernoulli/pmf/benchmark/benchmark.native.js
Outdated
Show resolved
Hide resolved
Signed-off-by: Philipp Burckhardt <[email protected]>
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.
Let's get this PR finally in. Thanks, @Neerajpathak07!
PR Commit Message
Please review the above commit message and make any necessary adjustments. |
Resolves #3418
Description
This pull request:
stats/base/dists/bernoulli/pmf
Related Issues
This pull request:
@stdlib/stats/base/dists/bernoulli/pmf
#3418Questions
No.
Other
No.
Checklist
@stdlib-js/reviewers