Skip to content

Add BenchmarkDotNet benchmark project and DataLoader benchmark#159

Merged
mattjohnsonpint merged 2 commits intomattjohnsonpint:mainfrom
lahma:benchmarks
Feb 11, 2025
Merged

Add BenchmarkDotNet benchmark project and DataLoader benchmark#159
mattjohnsonpint merged 2 commits intomattjohnsonpint:mainfrom
lahma:benchmarks

Conversation

@lahma
Copy link
Contributor

@lahma lahma commented Feb 11, 2025

No description provided.

@lahma
Copy link
Contributor Author

lahma commented Feb 11, 2025

You could probably drop .NET SDKs below 8 I believe, newer ones build older targets.

@mattjohnsonpint
Copy link
Owner

mattjohnsonpint commented Feb 11, 2025

You could probably drop .NET SDKs below 8 I believe, newer ones build older targets.

They'll build for older targets, but tests will fail if the older runtimes are not installed.

See also actions/setup-dotnet#352

@mattjohnsonpint
Copy link
Owner

This is nice - thanks for the PR! Could you also please add a Github Action (or a workflow step to the existing two actions) that runs the benchmark and puts the output somewhere convenient (like the run summary output)?

@lahma
Copy link
Contributor Author

lahma commented Feb 11, 2025

I'm not sure if it is beneficial as we are running on random CI runners, they would not be comparable. Maybe a separate PR?

@mattjohnsonpint mattjohnsonpint merged commit e3ab4d1 into mattjohnsonpint:main Feb 11, 2025
3 checks passed
@lahma lahma deleted the benchmarks branch February 11, 2025 18:16
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