Skip to content

Commit

Permalink
chore: address commit comments
Browse files Browse the repository at this point in the history
PR-URL: #5757
Closes: #5747

Reviewed-by: Philipp Burckhardt <[email protected]>
Reviewed-by: Gururaj Gurram <[email protected]>
  • Loading branch information
jalajk3004 authored Mar 4, 2025
1 parent befe02c commit 1ea2bdf
Show file tree
Hide file tree
Showing 3 changed files with 3 additions and 3 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -200,7 +200,7 @@ The function accepts the following arguments:
- **n**: `[in] int32_t` number of draws.

```c
double stdlib_base_dists_hypergeometric_logpmf ( const double x, const int32_t N, const int32_t K, const int32_t n );
double stdlib_base_dists_hypergeometric_logpmf( const double x, const int32_t N, const int32_t K, const int32_t n );
```
</section>
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -26,7 +26,7 @@
#include <stdint.h>

/**
* Evaluates the natural logarithm of the probability mass function (PMF) for a hypergeometric distribution with population size `N`, subpopulation size `K` and number of draws `n`.
* Evaluates the natural logarithm of the probability mass function (PMF) for a hypergeometric distribution with population size `N`, subpopulation size `K`, and number of draws `n`.
*
* @param x input value
* @param N population size
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -68,7 +68,7 @@ tape( 'if provided an integer `x` greater than `min( n, K )`, the function retur
t.equal( y, NINF, 'returns expected value' );

y = logpmf( 100, 20, 20, 10 );
t.equal( y, NINF, 'returns -Infinity' );
t.equal( y, NINF, 'returns expected value' );

t.end();
});
Expand Down

1 comment on commit 1ea2bdf

@stdlib-bot
Copy link
Contributor

Choose a reason for hiding this comment

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

Coverage Report

Package Statements Branches Functions Lines
stats/base/dists/hypergeometric/logpmf $\color{green}381/381$
$\color{green}+100.00\%$
$\color{green}40/40$
$\color{green}+100.00\%$
$\color{green}4/4$
$\color{green}+100.00\%$
$\color{green}381/381$
$\color{green}+100.00\%$

The above coverage report was generated for the changes in this push.

Please sign in to comment.