-
-
Notifications
You must be signed in to change notification settings - Fork 591
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 stats/base/dists/planck/mean
#3942
Conversation
Coverage Report
The above coverage report was generated for the changes in this PR. |
lib/node_modules/@stdlib/stats/base/dists/planck/mean/README.md
Outdated
Show resolved
Hide resolved
lib/node_modules/@stdlib/stats/base/dists/planck/mean/README.md
Outdated
Show resolved
Hide resolved
lib/node_modules/@stdlib/stats/base/dists/planck/mean/README.md
Outdated
Show resolved
Hide resolved
lib/node_modules/@stdlib/stats/base/dists/planck/mean/benchmark/benchmark.js
Outdated
Show resolved
Hide resolved
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.
Most of the same requested changes in #3896 are applicable here.
If generating rv doesn't need changes then this PR is ready to merge. |
lib/node_modules/@stdlib/stats/base/dists/planck/mean/benchmark/benchmark.js
Outdated
Show resolved
Hide resolved
lib/node_modules/@stdlib/stats/base/dists/planck/mean/examples/index.js
Outdated
Show resolved
Hide resolved
lib/node_modules/@stdlib/stats/base/dists/planck/mean/lib/main.js
Outdated
Show resolved
Hide resolved
lib/node_modules/@stdlib/stats/base/dists/planck/mean/lib/main.js
Outdated
Show resolved
Hide resolved
lib/node_modules/@stdlib/stats/base/dists/planck/mean/test/fixtures/python/REQUIRE
Outdated
Show resolved
Hide resolved
lib/node_modules/@stdlib/stats/base/dists/planck/mean/test/fixtures/python/runner.py
Show resolved
Hide resolved
lib/node_modules/@stdlib/stats/base/dists/planck/mean/test/fixtures/python/runner.py
Outdated
Show resolved
Hide resolved
/stdlib merge |
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 another round of review comments.
/stdlib merge |
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 now LGTM. Thanks for working on this @Jaysukh-409.
Description
This pull request:
mean
for the Planck Discrete Random Variable.Related Issues
This pull request:
Questions
No.
Other
No.
Checklist
@stdlib-js/reviewers