Skip to content

chore(benchmark): add benchmark tests#33

Merged
Lemon73-Computing merged 4 commits into
mainfrom
lemon73/add-benchmark
May 8, 2026
Merged

chore(benchmark): add benchmark tests#33
Lemon73-Computing merged 4 commits into
mainfrom
lemon73/add-benchmark

Conversation

@Lemon73-Computing
Copy link
Copy Markdown
Member

No description provided.

@Lemon73-Computing Lemon73-Computing self-assigned this May 8, 2026
@Lemon73-Computing Lemon73-Computing added enhancement New feature or request good first issue Good for newcomers csharp Implementations of C# labels May 8, 2026
@Lemon73-Computing Lemon73-Computing linked an issue May 8, 2026 that may be closed by this pull request
@github-actions
Copy link
Copy Markdown

github-actions Bot commented May 8, 2026

Code Coverage

Result: success

Show coverage summary

Summary

Generated on: 05/08/2026 - 17:45:58
Coverage date: 05/08/2026 - 17:45:56
Parser: Cobertura
Assemblies: 1
Classes: 1
Files: 3
Line coverage: 100% (71 of 71)
Covered lines: 71
Uncovered lines: 0
Coverable lines: 71
Total lines: 139
Branch coverage: 100% (20 of 20)
Covered branches: 20
Total branches: 20
Method coverage: Feature is only available for sponsors

Risk Hotspots

No risk hotspots found.

Coverage

Name Covered Uncovered Coverable Total Line coverage Covered Total Branch coverage
CommonNovel 71 0 71 139 100% 20 20 100%
CommonNovel.Compiler 71 0 71 139 100% 20 20 100%

@Lemon73-Computing
Copy link
Copy Markdown
Member Author

Actual output:



BenchmarkDotNet v0.15.8, Windows 11 (10.0.26200.8328/25H2/2025Update/HudsonValley2)
13th Gen Intel Core i7-13700H 2.40GHz, 1 CPU, 20 logical and 14 physical cores
.NET SDK 10.0.103
  [Host]     : .NET 10.0.3 (10.0.3, 10.0.326.7603), X64 RyuJIT x86-64-v3
  DefaultJob : .NET 10.0.3 (10.0.3, 10.0.326.7603), X64 RyuJIT x86-64-v3


Method Mean Error StdDev Gen0 Allocated
RunNoder 104.307 ns 1.9910 ns 3.8361 ns 0.0497 624 B
RunParser 465.026 ns 3.9148 ns 3.4704 ns 0.1025 1288 B
RunExecutor 3.349 ns 0.0738 ns 0.0691 ns 0.0032 40 B

Copy link
Copy Markdown

@gemini-code-assist gemini-code-assist Bot left a comment

Choose a reason for hiding this comment

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

Code Review

This pull request introduces a new benchmarking project, CommonNovel.Benchmarks, targeting .NET 10 and utilizing BenchmarkDotNet to measure the performance of the Compiler class's Noder, Parse, and Executor methods. Feedback focuses on correcting an invalid NuGet package version for BenchmarkDotNet, adding a missing namespace import to fix compilation, and refactoring benchmark methods to move input data to class fields to avoid measurement noise from allocations. Additionally, it is recommended to rename the temporary documentation file to a standard README.md.

Comment thread src/CommonNovel.Benchmarks/CommonNovel.Benchmarks.csproj
Comment thread src/CommonNovel.Benchmarks/Program.cs
Comment thread src/CommonNovel.Benchmarks/Program.cs
Comment thread src/CommonNovel.Benchmarks/TEMP.md
@Lemon73-Computing Lemon73-Computing merged commit 3de9e00 into main May 8, 2026
1 check passed
@Lemon73-Computing Lemon73-Computing deleted the lemon73/add-benchmark branch May 8, 2026 17:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

csharp Implementations of C# enhancement New feature or request good first issue Good for newcomers

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add benchmark tests

1 participant