Merge pull request #2899 from CliMA/ck/results_before_tests

Print benchmark results before tests
CliMA · Apr 12, 2024 · e9aba77 · e9aba77
2 parents c2e8bae + 9ec9269
commit e9aba77
Showing 1 changed file with 7 additions and 6 deletions.
diff --git a/perf/benchmark.jl b/perf/benchmark.jl
@@ -43,6 +43,13 @@ trials["step!"] = get_trial(SciMLBase.step!, (integrator, ), "step!");
 
 using Test
 using ClimaComms
+
+table_summary = OrderedCollections.OrderedDict()
+for k in keys(trials)
+    table_summary[k] = get_summary(trials[k])
+end
+tabulate_summary(table_summary)
+
 are_boundschecks_forced = Base.JLOptions().check_bounds == 1
 # Benchmark allocation tests
 @testset "Benchmark allocation tests" begin
@@ -64,12 +71,6 @@ are_boundschecks_forced = Base.JLOptions().check_bounds == 1
     end
 end
 
-table_summary = OrderedCollections.OrderedDict()
-for k in keys(trials)
-    table_summary[k] = get_summary(trials[k])
-end
-tabulate_summary(table_summary)
-
 if get(ENV, "BUILDKITE", "") == "true"
     # Export table_summary
     import JSON