Skip to content

Commit

Permalink
Merge #898
Browse files Browse the repository at this point in the history
898: Suppress allocation table r=charleskawczynski a=charleskawczynski

This PR suppresses the allocation table job to reduce our CI times, since it takes almost twice as long as the other jobs. For now, we can use the allocation tests in the flame script to help ensure that we keep an eye on allocations. Most of the remaining allocations (at least in the performance target) seem to be due to allocations in dependencies. I'll open issues for each of those items from the table (related issue #686).

Co-authored-by: Charles Kawczynski <[email protected]>
  • Loading branch information
bors[bot] and charleskawczynski authored Oct 5, 2022
2 parents ce21644 + 0d19d60 commit 4de4930
Showing 1 changed file with 5 additions and 4 deletions.
9 changes: 5 additions & 4 deletions .buildkite/pipeline.yml
Original file line number Diff line number Diff line change
Expand Up @@ -418,10 +418,11 @@ steps:
- label: ":rocket: benchmark: baroclinic wave (ρe_tot)"
command: "julia --color=yes --project=perf perf/benchmark.jl --job_id bm_sphere_baroclinic_wave_rhoe"

- label: ":mag: Allocations: perf target"
command: "julia --color=yes --project=perf perf/allocs.jl"
agents:
slurm_mem: 20GB
# TODO: temporarily suppressed to reduce CI times
# - label: ":mag: Allocations: perf target"
# command: "julia --color=yes --project=perf perf/allocs.jl"
# agents:
# slurm_mem: 20GB

- wait: ~
continue_on_failure: true
Expand Down

0 comments on commit 4de4930

Please sign in to comment.