-
-
Notifications
You must be signed in to change notification settings - Fork 908
Description
This commit has 6 comment(s) from core contributors that require attention.
Commit: be656a143216896aa6b1ce0df89646e6a3565ec9
Comments:
-
Line 124: @stdlib-bot This change should be reverted. The use of
logpdf
was intentional here.stdlib/lib/node_modules/@stdlib/stats/base/dists/triangular/logcdf/README.md
Lines 121 to 127 in be656a1
## Notes - In virtually all cases, using the `logcdf` or `logcdf` functions is preferable to manually computing the logarithm of the `pdf` or `cdf`, respectively, since the latter is prone to overflow and underflow. </section> -
Line 189: @stdlib-bot Missing Oxford comma after
(upper limit)
and beforeand
.stdlib/lib/node_modules/@stdlib/stats/base/dists/triangular/logcdf/README.md
Lines 186 to 192 in be656a1
#### stdlib_base_dists_geometric_logcdf( x, a, b, c ) Evaluates the natural logarithm of the [cumulative distribution function][cdf] (CDF) for a [triangular][triangular-distribution] distribution with parameters `a` (lower limit), `b` (upper limit) and `c` (mode). ```c double y = stdlib_base_dists_geometric_logcdf( 0.5, -1.0, 1.0, 0.0 ); -
Line 230: @stdlib-bot This header is not used and can be removed.
stdlib/lib/node_modules/@stdlib/stats/base/dists/triangular/logcdf/README.md
Lines 227 to 233 in be656a1
#include "stdlib/constants/float64/eps.h" #include <stdlib.h> #include <stdio.h> #include <math.h> static double random_uniform( const double min, const double max ) { double v = (double)rand() / ( (double)RAND_MAX + 1.0 ); -
Line 23: @stdlib-bot This header is not used and can be removed.
stdlib/lib/node_modules/@stdlib/stats/base/dists/triangular/logcdf/examples/c/example.c
Lines 20 to 26 in be656a1
#include "stdlib/constants/float64/eps.h" #include <stdlib.h> #include <stdio.h> #include <math.h> static double random_uniform( const double min, const double max ) { double v = (double)rand() / ( (double)RAND_MAX + 1.0 ); -
Line 29: @stdlib-bot This description should be updated to use the description used elsewhere.
stdlib/lib/node_modules/@stdlib/stats/base/dists/triangular/logcdf/lib/native.js
Lines 26 to 32 in be656a1
// MAIN // /** * Evaluates the natural logarithm of the cumulative distribution function (CDF) for a triangular distribution with lower limit `a` and upper limit `b` and mode `c` at a value `x`. * * @private * @param {number} x - input value -
Line 55: @stdlib-bot This should be
if ( x < b ) {
.stdlib/lib/node_modules/@stdlib/stats/base/dists/triangular/logcdf/src/main.c
Lines 52 to 58 in be656a1
if ( x <= c ) { return ( 2.0 * stdlib_base_ln( x - a ) - stdlib_base_ln( ( b - a ) * ( c - a ) ) ); } if( x < b ){ return stdlib_base_ln( 1.0 - ( stdlib_base_pow( b - x, 2.0 ) / ( ( b - a ) * ( b - c ) ) ) ); } return 0.0;
Interested in helping improve the project? If you are, the comment linked to above has 6 comment(s) from core contributors that could use your attention.
What do you need to do?
- Open the above linked comments mentioning @stdlib-bot.
- Review the suggested changes or follow-up tasks (e.g., formatting improvements, small refactorings, or clean-up).
- If you are a first-time contributor, follow the contributing and development guides to setup your local environment for contributing to stdlib. If you are already a seasoned stdlib contributor, create a new branch on your local fork for making the changes.
- Make all the desired changes and commit those changes to a local branch.
- Push the changes to GitHub and open a new pull request against the
develop
branch of the main stdlib development repository.
Once you've opened a pull request, a stdlib maintainer will review your work and suggest any follow-up changes.
And that's it!
Thank you for your help in reducing the project backlog and in improving the quality of stdlib. 🙌
Notes
- For older commits, there is a chance that comments will have been already been addressed due to other refactorings. If you find that to be true, don't worry! Just move on to addressing the next comment, and, when opening your pull request and describing your proposed changes, be sure to link to the comment and mention that it has been addressed. This will help reviewers when reviewing your code!
This issue was created automatically to address commit comments tagging @stdlib-bot.