Skip to content

bench: refactor to use string interpolation in @stdlib/stats/base/dists/discrete-uniform/mean#10985

Closed
rautelaKamal wants to merge 1 commit intostdlib-js:developfrom
rautelaKamal:refactor/discrete-uniform-mean-benchmarks
Closed

bench: refactor to use string interpolation in @stdlib/stats/base/dists/discrete-uniform/mean#10985
rautelaKamal wants to merge 1 commit intostdlib-js:developfrom
rautelaKamal:refactor/discrete-uniform-mean-benchmarks

Conversation

@rautelaKamal
Copy link
Copy Markdown
Contributor

Description

This PR refactors the benchmark files to use string interpolation for benchmark names and corrects the variable sorting order in the main loop to follow the convention len, max, min, i, y.

Checklist

@rautelaKamal rautelaKamal force-pushed the refactor/discrete-uniform-mean-benchmarks branch from 11df45b to 9f11332 Compare March 16, 2026 02:42
@stdlib-bot
Copy link
Copy Markdown
Contributor

Hello! Thank you for your contribution to stdlib.

We noticed that the contributing guidelines acknowledgment is missing from your pull request. Here's what you need to do:

  1. Please read our contributing guidelines.

  2. Update your pull request description to include this checked box:

    - [x] Read, understood, and followed the [contributing guidelines](https://github.com/stdlib-js/stdlib/blob/develop/CONTRIBUTING.md)

This acknowledgment confirms that you've read the guidelines, which include:

  • The developer's certificate of origin
  • Your agreement to license your contributions under the project's terms

We can't review or accept contributions without this acknowledgment.

Thank you for your understanding and cooperation. We look forward to reviewing your contribution!

@stdlib-bot stdlib-bot added Statistics Issue or pull request related to statistical functionality. Needs Review A pull request which needs code review. labels Mar 16, 2026
var discreteUniform = require( '@stdlib/random/base/discrete-uniform' );
var Float64Array = require( '@stdlib/array/float64' );
var isnan = require( '@stdlib/math/base/assert/is-nan' );
var tryRequire = require( '@stdlib/utils/try-require' );
var format = require( '@stdlib/string/format' );
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

you just need to use string interpolation in JS benchmarks as suggested in the issue #8647

@Planeshifter
Copy link
Copy Markdown
Member

#10345 already addressed this, so closing this PR. For future reference, PR title here should have been "bench: refactor to use string interpolation in `stats/base/dists/discrete-uniform/mean`".

@stdlib-bot stdlib-bot removed the Needs Review A pull request which needs code review. label Mar 16, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Statistics Issue or pull request related to statistical functionality.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants