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

Add wax to benchmark #24

Open
wants to merge 3 commits into
base: main
Choose a base branch
from
Open

Add wax to benchmark #24

wants to merge 3 commits into from

Conversation

milesj
Copy link

@milesj milesj commented Mar 21, 2025

https://docs.rs/wax/latest/wax/

Seems like wax is faster.

@shulaoda
Copy link
Member

I have updated the benchmark data to ensure consistency. I first ran a few iterations for warming up, then executed three runs and took the best result among them.

Copy link

codspeed-hq bot commented Mar 22, 2025

CodSpeed Performance Report

Merging #24 will create unknown performance changes

Comparing milesj:wax (c05457a) with main (c02faef)

Summary

⚠️ No benchmarks were detected in both the base of the PR and the PR.\

@shulaoda shulaoda closed this Mar 22, 2025
@shulaoda shulaoda reopened this Mar 22, 2025
@shulaoda
Copy link
Member

shulaoda commented Mar 22, 2025

CodSpeed Performance Report

Merging #24 will create unknown performance changes

Comparing milesj:wax (c05457a) with main (c02faef)

Summary

🆕 14 new benchmarks

Benchmarks breakdown

  Benchmark BASE HEAD Change
🆕 fast-glob N/A 3.4 µs N/A
🆕 glob-match N/A 5.4 µs N/A
🆕 globset N/A 241.1 µs N/A
🆕 globset-pre-compiled N/A 3.7 µs N/A
🆕 wax N/A 617.7 µs N/A
🆕 wax-pre-compiled N/A 2.9 µs N/A
🆕 fast-glob N/A 1.5 µs N/A
🆕 glob N/A 7.7 µs N/A
🆕 glob-match N/A 3.5 µs N/A
🆕 glob-pre-compiled N/A 2 µs N/A
🆕 globset N/A 192.6 µs N/A
🆕 globset-pre-compiled N/A 3.6 µs N/A
🆕 wax N/A 552.7 µs N/A
🆕 wax-pre-compiled N/A 2.7 µs

@shulaoda shulaoda enabled auto-merge (squash) March 22, 2025 19:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants