Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

benchmark: update iterations in benchmark/util/normalize-encoding.js #50934

Merged
merged 1 commit into from
Dec 4, 2023

Conversation

lucshi
Copy link
Contributor

@lucshi lucshi commented Nov 27, 2023

Refs: #50571

Belowing are the score improvement after 10X iteration change:

util/normalize-encoding.js n=1000000
util/normalize-encoding.js input="":  n=1000000 percent=301.90%
util/normalize-encoding.js input="utf8":  n=1000000 percent=257.18%
util/normalize-encoding.js input="utf-8":  n=1000000 percent=247.71%
util/normalize-encoding.js input="UTF-8":  n=1000000 percent=158.92%
util/normalize-encoding.js input="UTF8":  n=1000000 percent=205.67%
util/normalize-encoding.js input="Utf8":  n=1000000 percent=163.09%
util/normalize-encoding.js input="ucs2":  n=1000000 percent=192.82%
util/normalize-encoding.js input="UCS2":  n=1000000 percent=179.22%
util/normalize-encoding.js input="utf16le":  n=1000000 percent=239.94%
util/normalize-encoding.js input="UTF16LE":  n=1000000 percent=207.57%
util/normalize-encoding.js input="binary":  n=1000000 percent=183.19%
util/normalize-encoding.js input="BINARY":  n=1000000 percent=150.46%
util/normalize-encoding.js input="latin1":  n=1000000 percent=198.72%
util/normalize-encoding.js input="base64":  n=1000000 percent=237.31%
util/normalize-encoding.js input="BASE64":  n=1000000 percent=166.36%
util/normalize-encoding.js input="hex":  n=1000000 percent=231.76%
util/normalize-encoding.js input="HEX":  n=1000000 percent=198.36%
util/normalize-encoding.js input="foo":  n=1000000 percent=192.08%
util/normalize-encoding.js input="undefined":  n=1000000 percent=383.12%
util/normalize-encoding.js input="group_common":  n=1000000 percent=160.35%
util/normalize-encoding.js input="group_upper":  n=1000000 percent=155.27%
util/normalize-encoding.js input="group_uncommon":  n=1000000 percent=250.76%
util/normalize-encoding.js input="group_misc":  n=1000000 percent=169.54%

@nodejs-github-bot nodejs-github-bot added benchmark Issues and PRs related to the benchmark subsystem. util Issues and PRs related to the built-in util module. labels Nov 27, 2023
@H4ad H4ad added the author ready PRs that have at least one approval, no pending requests for changes, and a CI started. label Dec 2, 2023
@H4ad H4ad added the commit-queue Add this label to land a pull request using GitHub Actions. label Dec 4, 2023
@nodejs-github-bot nodejs-github-bot removed the commit-queue Add this label to land a pull request using GitHub Actions. label Dec 4, 2023
@nodejs-github-bot nodejs-github-bot merged commit c510e26 into nodejs:main Dec 4, 2023
30 checks passed
@nodejs-github-bot
Copy link
Collaborator

Landed in c510e26

RafaelGSS pushed a commit that referenced this pull request Dec 15, 2023
Fixes: #50571
PR-URL: #50934
Refs: #50571
Reviewed-By: Vinícius Lourenço Claro Cardoso <contact@viniciusl.com.br>
@RafaelGSS RafaelGSS mentioned this pull request Dec 15, 2023
richardlau pushed a commit that referenced this pull request Mar 25, 2024
Fixes: #50571
PR-URL: #50934
Refs: #50571
Reviewed-By: Vinícius Lourenço Claro Cardoso <contact@viniciusl.com.br>
@richardlau richardlau mentioned this pull request Mar 25, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
author ready PRs that have at least one approval, no pending requests for changes, and a CI started. benchmark Issues and PRs related to the benchmark subsystem. util Issues and PRs related to the built-in util module.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants