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.

Sign up for GitHub

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

Jump to bottom

perf: linear mapping #111

Merged
merged 3 commits into from
Aug 23, 2024
Merged

perf: linear mapping #111

merged 3 commits into from
Aug 23, 2024

Conversation

SyMind
Copy link
Member

@SyMind SyMind commented Aug 23, 2024

No description provided.

@codspeed-hq CodSpeed HQ
Copy link

codspeed-hq bot commented Aug 23, 2024 •
edited
Loading

CodSpeed Performance Report

Merging #111 will improve performances by 6.25%

Comparing SyMind:no-hash-hasher (e9bedfa) with main (7c4ca9f)

Summary

⚡ 1 improvements
✅ 4 untouched benchmarks

Benchmarks breakdown

Benchmark main SyMind:no-hash-hasher Change
⚡ concat_generate_string 53 µs 49.9 µs +6.25%

@SyMind SyMind force-pushed the no-hash-hasher branch 3 times, most recently from fba321b to 0f3c488 Compare August 23, 2024 09:44
@SyMind SyMind mentioned this pull request Aug 23, 2024
Merged
2 tasks
@SyMind SyMind marked this pull request as ready for review August 23, 2024 09:48
@SyMind SyMind merged commit 920711b into web-infra-dev:main Aug 23, 2024
7 checks passed
@SyMind SyMind deleted the no-hash-hasher branch August 23, 2024 09:56
@SyMind SyMind changed the title perf: no hash hasher perf: linear mapping Aug 23, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@JSerFeng JSerFeng JSerFeng approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@SyMind @JSerFeng