Movatterモバイル変換

[0]ホーム
URL:

Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

 Sign up
Appearance settings

improve perf with array pre-allocate#403

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Merged
JoviDeCroock merged 1 commit intomainfrompreallocate-arrays
Dec 12, 2024
Merged

Conversation

@JoviDeCroock
Copy link
Member

Text:baseline x 434.471 ops/sec ±2.09 (89 runs sampled)current x 455.816 ops/sec ±1.35 (88 runs sampled)Fastest is currentSearchResults:baseline x 2,124.763 ops/sec ±3.50 (92 runs sampled)current x 2,174.318 ops/sec ±4.29 (88 runs sampled)Fastest is currentColorPicker:baseline x 12,934.781 ops/sec ±1.34 (95 runs sampled)current x 13,121.543 ops/sec ±0.87 (97 runs sampled)Fastest is current

@changeset-bot
Copy link

🦋 Changeset detected

Latest commit:7fcc557

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
NameType
preact-render-to-stringPatch

Not sure what this means?Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

Copy link
Member

@rschristianrschristian left a comment

Choose a reason for hiding this comment

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

Damn, where else do we use arrays? Month of free perf boosts!

Might look at copying over some of the benching setup from the core repo, having to bump the baseline-rts pkg ain't ideal

@JoviDeCroockJoviDeCroock merged commited720ef intomainDec 12, 2024
1 check passed
@JoviDeCroockJoviDeCroock deleted the preallocate-arrays branchDecember 12, 2024 08:24
@github-actionsgithub-actionsbot mentioned this pull requestOct 20, 2024
Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment

Reviewers

@rschristianrschristianrschristian approved these changes

@marvinhagemeistermarvinhagemeisterAwaiting requested review from marvinhagemeister

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@JoviDeCroock@rschristian
[8]ページ先頭
©2009-2025 Movatter.jp