bench: update random value generation

[anandkaranubc]

Resolves none.

Description

What is the purpose of this pull request?

This pull request:

• Refactors random number generation in JS benchmarks for the remaining math/base/special/sqrt* packages
• Replaces randu() with uniform() from @stdlib/random/array/uniform for cleaner and more consistent code.
• Moves the random number generation outside the benchmarking loops.
• Updates the test messages to follow code conventions.

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
math/base/special/sqrt	$\color{green}161/161$ $\color{green}+100.00\%$	$\color{green}4/4$ $\color{green}+100.00\%$	$\color{green}1/1$ $\color{green}+100.00\%$	$\color{green}161/161$ $\color{green}+100.00\%$
math/base/special/sqrt1pm1	$\color{green}203/203$ $\color{green}+100.00\%$	$\color{green}9/9$ $\color{green}+100.00\%$	$\color{green}2/2$ $\color{green}+100.00\%$	$\color{green}203/203$ $\color{green}+100.00\%$
math/base/special/sqrtf	$\color{green}168/168$ $\color{green}+100.00\%$	$\color{green}5/5$ $\color{green}+100.00\%$	$\color{green}2/2$ $\color{green}+100.00\%$	$\color{green}168/168$ $\color{green}+100.00\%$
math/base/special/sqrtpi	$\color{green}168/168$ $\color{green}+100.00\%$	$\color{green}5/5$ $\color{green}+100.00\%$	$\color{green}2/2$ $\color{green}+100.00\%$	$\color{green}168/168$ $\color{green}+100.00\%$
math/base/special/sqrtpif	$\color{green}169/169$ $\color{green}+100.00\%$	$\color{green}5/5$ $\color{green}+100.00\%$	$\color{green}2/2$ $\color{green}+100.00\%$	$\color{green}169/169$ $\color{green}+100.00\%$

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

[anandkaranubc]

/stdlib merge

[anandkaranubc]

I am not sure why the test from stats/base/dists/uniform/stdev is failing in this PR, as it is unrelated here.

The tests from stats/base/dists/uniform/stdev pass though if I test them locally after building the native add-on.

[Planeshifter]

Looks good to me, @anandkaranubc!

As mentioned, failing check is unrelated to this PR. Issue is that opts is not passed in at

stdlib/lib/node_modules/@stdlib/stats/base/dists/uniform/stdev/test/test.native.js

Line 82 in 76b9592

tape( 'the function returns the standard deviation of a uniform distribution', function test( t ) {

so the test runs even when the native add-on is not built. For dependent packages, we currently only run their tests without building native add-ons to keep CI times in check.

[anandkaranubc]

@Planeshifter I see! I will create a PR adding the missing opts right away.

Thanks for explaining this in detail!

[stdlib-bot]

PR Commit Message

bench: update random value generation

PR-URL: https://github.com/stdlib-js/stdlib/pull/5786

Co-authored-by: Karan Anand <[email protected]>
Reviewed-by: Philipp Burckhardt <[email protected]>

Please review the above commit message and make any necessary adjustments.