Skip to content

Commit d0bd715

Browse files
Merge pull request #3 from GeoJSON-Net/feature/add-benchmarking
Test the benchmark run on pull request
2 parents 170bb3f + 8a6064c commit d0bd715

3 files changed

Lines changed: 5 additions & 22 deletions

File tree

.github/workflows/continous-benchmark.yml

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -23,21 +23,21 @@ jobs:
2323
6.0.x
2424
# Run benchmark with `go test -bench` and stores the output to a file
2525
- name: Run benchmark
26-
run: dotnet run --project ./GeoJSON.Text.Test.Benchmark/GeoJSON.Text.Test.Benchmark.csproj --framework net6 --runtimes net60 -c Release -- --job short -f *SerializeAndDeserialize*
26+
run: dotnet run --project ./GeoJSON.Text.Test.Benchmark/GeoJSON.Text.Test.Benchmark.csproj --framework net6 --runtimes net60 -c Release -- --job medium -f *SerializeAndDeserialize*
2727
# Download previous benchmark result from cache (if exists)
2828
- name: Download previous benchmark data
2929
uses: actions/cache@v1
3030
with:
3131
path: ./cache
3232
key: ${{ runner.os }}-benchmark
3333
# Run `github-action-benchmark` action
34-
- name: Store benchmark result for serialize feature collection
34+
- name: Store benchmark result
3535
uses: Happypig375/github-action-benchmark@v1
3636
with:
3737
# What benchmark tool the output.txt came from
3838
tool: 'benchmarkdotnet'
3939
# Where the output from the benchmark tool is stored
40-
output-file-path: 'src/BenchmarkDotNet.Artifacts/results/GeoJSON.Text.Test.Benchmark.SerializeAndDeserialize-report-full-compressed.json'
40+
output-file-path: 'src/BenchmarkDotNet.Artifacts/results/GeoJSON.Text.Test.Benchmark.SerializeAndDeserialize-report-full.json'
4141
# Where the previous data file is stored
4242
external-data-json-path: ./cache/benchmark-data.json
4343
# Will comment on pull request when an alert happens

src/Directory.Builddss.props

Lines changed: 0 additions & 7 deletions
This file was deleted.

src/GeoJSON.Text.Test.Benchmark/TestConfig.cs

Lines changed: 2 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -20,18 +20,7 @@ public class TestConfig : ManualConfig
2020
{
2121
public TestConfig()
2222
{
23-
AddJob(Job.Default.WithRuntime(CoreRuntime.Core31)
24-
.WithLaunchCount(1) // benchmark process will be launched only once
25-
.WithWarmupCount(3) // 3 warmup iteration
26-
.WithIterationCount(3), // 3 target iteration,
27-
Job.Default.WithRuntime(CoreRuntime.Core50)
28-
.WithLaunchCount(1) // benchmark process will be launched only once
29-
.WithWarmupCount(3) // 3 warmup iteration
30-
.WithIterationCount(3), // 3 target iteration,
31-
Job.Default.WithRuntime(CoreRuntime.Core60)
32-
.WithLaunchCount(1) // benchmark process will be launched only once
33-
.WithWarmupCount(3) // 3 warmup iteration
34-
.WithIterationCount(3)); // 3 target iteration);
23+
WithOrderer(new FastestToSlowestOrderer());
3524

3625
AddColumn(RankColumn.Roman);
3726
AddExporter(CsvMeasurementsExporter.Default,
@@ -41,6 +30,7 @@ public TestConfig()
4130
AddDiagnoser(MemoryDiagnoser.Default);
4231
}
4332

33+
4434
private class FastestToSlowestOrderer : IOrderer
4535
{
4636
public IEnumerable<BenchmarkCase> GetExecutionOrder(ImmutableArray<BenchmarkCase> benchmarksCase) =>

0 commit comments

Comments
 (0)