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

Appearance settings

feat: add StaticPropertyInitializationGeneratorBenchmarks - #4639

#4639
Merged
thomhurst merged 1 commit into
thomhurst:mainthomhurst/TUnit:mainfrom
TimothyMakkison:static_prop_init_generator_benchTimothyMakkison/TUnit:static_prop_init_generator_benchCopy head branch name to clipboard
Feb 1, 2026
Merged

feat: add StaticPropertyInitializationGeneratorBenchmarks#4639
thomhurst merged 1 commit into
thomhurst:mainthomhurst/TUnit:mainfrom
TimothyMakkison:static_prop_init_generator_benchTimothyMakkison/TUnit:static_prop_init_generator_benchCopy head branch name to clipboard

Conversation

@TimothyMakkison

Copy link
Copy Markdown
Contributor

Add StaticPropertyInitializationGeneratorBenchmarks

Method Mean Error StdDev Median Gen0 Gen1 Allocated
RunGenerator 183.7 ms 6.12 ms 16.66 ms 176.5 ms 10000.0000 4000.0000 99.35 MB

@TimothyMakkison

TimothyMakkison commented Feb 1, 2026

Copy link
Copy Markdown
Contributor Author

After rebasing, these are the new benchmarks. Were more tests that used static property initializers added recently?

Method Mean Error StdDev Gen0 Gen1 Allocated
RunGenerator 290.6 ms 5.78 ms 10.42 ms 11000.0000 3000.0000 102.34 MB

@thomhurst

Copy link
Copy Markdown
Owner

@TimothyMakkison nope! Might be environment rated?

@TimothyMakkison

TimothyMakkison commented Feb 1, 2026

Copy link
Copy Markdown
Contributor Author

@TimothyMakkison nope! Might be environment rated?

Porbably! I reran the benchmarks under similar conditions on older commits and got the same results of around 290ms. Seems like I had a weird fluke earlier

@thomhurst
thomhurst merged commit 5f682ff into thomhurst:main Feb 1, 2026
8 of 10 checks passed
This was referenced Feb 3, 2026
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

Morty Proxy This is a proxified and sanitized view of the page, visit original site.