bench: refactor random number generation in stats/base/dists/bernoulli

[anandkaranubc]

Resolves none.

Description

What is the purpose of this pull request?

This pull request:

• Refactors random number generation in JS benchmarks for stats/base/dists/bernoulli.
• Replaces randu() with uniform() for cleaner and more consistent code.

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.

• Read, understood, and followed the contributing guidelines.

---

@stdlib-js/reviewers

[stdlib-bot]

Coverage Report

Package	Statements	Branches	Functions	Lines
stats/base/dists/bernoulli/cdf	$\color{green}215/215$ $\color{green}+100.00\%$	$\color{green}25/25$ $\color{green}+100.00\%$	$\color{green}3/3$ $\color{green}+100.00\%$	$\color{green}215/215$ $\color{green}+100.00\%$
stats/base/dists/bernoulli/ctor	$\color{green}409/409$ $\color{green}+100.00\%$	$\color{green}26/26$ $\color{green}+100.00\%$	$\color{green}15/15$ $\color{green}+100.00\%$	$\color{green}409/409$ $\color{green}+100.00\%$
stats/base/dists/bernoulli/entropy	$\color{green}178/178$ $\color{green}+100.00\%$	$\color{green}12/12$ $\color{green}+100.00\%$	$\color{green}2/2$ $\color{green}+100.00\%$	$\color{green}178/178$ $\color{green}+100.00\%$
stats/base/dists/bernoulli/kurtosis	$\color{green}109/109$ $\color{green}+100.00\%$	$\color{green}7/7$ $\color{green}+100.00\%$	$\color{green}1/1$ $\color{green}+100.00\%$	$\color{green}109/109$ $\color{green}+100.00\%$
stats/base/dists/bernoulli/mean	$\color{green}165/165$ $\color{green}+100.00\%$	$\color{green}9/9$ $\color{green}+100.00\%$	$\color{green}2/2$ $\color{green}+100.00\%$	$\color{green}165/165$ $\color{green}+100.00\%$
stats/base/dists/bernoulli/median	$\color{green}107/107$ $\color{green}+100.00\%$	$\color{green}9/9$ $\color{green}+100.00\%$	$\color{green}1/1$ $\color{green}+100.00\%$	$\color{green}107/107$ $\color{green}+100.00\%$
stats/base/dists/bernoulli/mgf	$\color{green}195/195$ $\color{green}+100.00\%$	$\color{green}13/13$ $\color{green}+100.00\%$	$\color{green}3/3$ $\color{green}+100.00\%$	$\color{green}195/195$ $\color{green}+100.00\%$
stats/base/dists/bernoulli/mode	$\color{green}177/177$ $\color{green}+100.00\%$	$\color{green}11/11$ $\color{green}+100.00\%$	$\color{green}2/2$ $\color{green}+100.00\%$	$\color{green}177/177$ $\color{green}+100.00\%$
stats/base/dists/bernoulli/pmf	$\color{green}226/226$ $\color{green}+100.00\%$	$\color{green}25/25$ $\color{green}+100.00\%$	$\color{green}3/3$ $\color{green}+100.00\%$	$\color{green}226/226$ $\color{green}+100.00\%$
stats/base/dists/bernoulli/quantile	$\color{green}224/224$ $\color{green}+100.00\%$	$\color{green}25/25$ $\color{green}+100.00\%$	$\color{green}3/3$ $\color{green}+100.00\%$	$\color{green}224/224$ $\color{green}+100.00\%$
stats/base/dists/bernoulli/skewness	$\color{green}116/116$ $\color{green}+100.00\%$	$\color{green}11/11$ $\color{green}+100.00\%$	$\color{green}1/1$ $\color{green}+100.00\%$	$\color{green}116/116$ $\color{green}+100.00\%$
stats/base/dists/bernoulli/stdev	$\color{green}166/166$ $\color{green}+100.00\%$	$\color{green}9/9$ $\color{green}+100.00\%$	$\color{green}2/2$ $\color{green}+100.00\%$	$\color{green}166/166$ $\color{green}+100.00\%$
stats/base/dists/bernoulli/variance	$\color{green}107/107$ $\color{green}+100.00\%$	$\color{green}7/7$ $\color{green}+100.00\%$	$\color{green}1/1$ $\color{green}+100.00\%$	$\color{green}107/107$ $\color{green}+100.00\%$

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

[anandkaranubc]

Hi @kgryte,

I wanted to confirm a couple of things about these refactorings:

1. Are we replacing all randu() expressions with uniform() for consistency, or only those where the minimum cannot be zero (e.g., randu() * 1.0 + EPS)?

2. For expressions like round( randu() ), would it be better to use discreteUniform instead?

Thanks for the help!

[kgryte]

LGTM

[kgryte]

Are we replacing all randu() expressions with uniform() for consistency, or only those where the minimum cannot be zero (e.g., randu() * 1.0 + EPS)?

All would be nice, not just those involving EPS.

For expressions like round( randu() ), would it be better to use discreteUniform instead?

Yes, discrete-uniform would be preferred.

[anandkaranubc]

Are we replacing all randu() expressions with uniform() for consistency, or only those where the minimum cannot be zero (e.g., randu() * 1.0 + EPS)?

All would be nice, not just those involving EPS.

For expressions like round( randu() ), would it be better to use discreteUniform instead?

Yes, discrete-uniform would be preferred.

I meant even the ones like randu() * 20.0 or leave them like that?

[kgryte]

I meant even the ones like randu() * 20.0 or leave them like that?

Yes, even those.

[anandkaranubc]

I meant even the ones like randu() * 20.0 or leave them like that?

Yes, even those.

Open to your feedback, @kgryte. Thanks!

[kgryte]

LGTM. Thanks, @anandkaranubc!