Skip to content

Comments

Add bench-on parameter to use benchmarks from PR#213

Merged
JoshuaLampert merged 3 commits intomainfrom
bench-on
Jun 3, 2025
Merged

Add bench-on parameter to use benchmarks from PR#213
JoshuaLampert merged 3 commits intomainfrom
bench-on

Conversation

@JoshuaLampert
Copy link
Member

To use the benchmark script as explained in MilesCranmer/AirspeedVelocity.jl#97 (comment).

@github-actions
Copy link
Contributor

github-actions bot commented Jun 3, 2025

Benchmark Results (Julia v1.10)

Time benchmarks
main cc172ce... main / cc172ce...
bbm_1d/bbm_1d_basic.jl - rhs!: 13.8 ± 0.31 μs 13.8 ± 0.27 μs 0.998 ± 0.03
bbm_1d/bbm_1d_fourier.jl - rhs!: 0.215 ± 0.0093 ms 0.527 ± 0.25 ms 0.408 ± 0.19
bbm_bbm_1d/bbm_bbm_1d_basic_reflecting.jl - rhs!: 0.0803 ± 0.0004 ms 0.0804 ± 0.00038 ms 0.999 ± 0.0069
bbm_bbm_1d/bbm_bbm_1d_dg.jl - rhs!: 0.0345 ± 0.0005 ms 0.0341 ± 0.00086 ms 1.01 ± 0.029
bbm_bbm_1d/bbm_bbm_1d_relaxation.jl - rhs!: 27.4 ± 0.45 μs 27.5 ± 0.66 μs 0.999 ± 0.029
bbm_bbm_1d/bbm_bbm_1d_upwind_relaxation.jl - rhs!: 0.0484 ± 0.00063 ms 0.0484 ± 0.00076 ms 1 ± 0.02
hyperbolic_serre_green_naghdi_1d/hyperbolic_serre_green_naghdi_dingemans.jl - rhs!: 4.35 ± 0.04 μs 4.43 ± 0.05 μs 0.982 ± 0.014
kdv_1d/kdv_1d_basic.jl - rhs!: 1.52 ± 0.03 μs 1.4 ± 0.02 μs 1.08 ± 0.026
kdv_1d/kdv_1d_implicit.jl - rhs!: 1.7 ± 0.03 μs 1.4 ± 0.011 μs 1.21 ± 0.023
serre_green_naghdi_1d/serre_green_naghdi_well_balanced.jl - rhs!: 0.202 ± 0.0085 ms 0.199 ± 0.0085 ms 1.02 ± 0.061
svaerd_kalisch_1d/svaerd_kalisch_1d_dingemans_relaxation.jl - rhs!: 0.148 ± 0.0051 ms 0.144 ± 0.0038 ms 1.03 ± 0.045
time_to_load 2.03 ± 0.01 s 2.01 ± 0.02 s 1.01 ± 0.011
Memory benchmarks
main cc172ce... main / cc172ce...
bbm_1d/bbm_1d_basic.jl - rhs!: 1 allocs: 4.12 kB 1 allocs: 4.12 kB 1
bbm_1d/bbm_1d_fourier.jl - rhs!: 1 allocs: 4.12 kB 1 allocs: 4.12 kB 1
bbm_bbm_1d/bbm_bbm_1d_basic_reflecting.jl - rhs!: 5 allocs: 1.17 kB 5 allocs: 1.17 kB 1
bbm_bbm_1d/bbm_bbm_1d_dg.jl - rhs!: 10 allocs: 8.62 kB 10 allocs: 8.62 kB 1
bbm_bbm_1d/bbm_bbm_1d_relaxation.jl - rhs!: 2 allocs: 8.25 kB 2 allocs: 8.25 kB 1
bbm_bbm_1d/bbm_bbm_1d_upwind_relaxation.jl - rhs!: 2 allocs: 8.25 kB 2 allocs: 8.25 kB 1
hyperbolic_serre_green_naghdi_1d/hyperbolic_serre_green_naghdi_dingemans.jl - rhs!: 0 allocs: 0 B 0 allocs: 0 B
kdv_1d/kdv_1d_basic.jl - rhs!: 0 allocs: 0 B 0 allocs: 0 B
kdv_1d/kdv_1d_implicit.jl - rhs!: 0 allocs: 0 B 0 allocs: 0 B
serre_green_naghdi_1d/serre_green_naghdi_well_balanced.jl - rhs!: 0.075 k allocs: 0.66 MB 0.075 k allocs: 0.66 MB 1
svaerd_kalisch_1d/svaerd_kalisch_1d_dingemans_relaxation.jl - rhs!: 0.042 k allocs: 0.315 MB 0.042 k allocs: 0.315 MB 1
time_to_load 0.153 k allocs: 14.5 kB 0.153 k allocs: 14.5 kB 1

@JoshuaLampert JoshuaLampert marked this pull request as ready for review June 3, 2025 16:45
@JoshuaLampert JoshuaLampert requested a review from ranocha June 3, 2025 16:46
@JoshuaLampert JoshuaLampert merged commit bb62e23 into main Jun 3, 2025
3 checks passed
@JoshuaLampert JoshuaLampert deleted the bench-on branch June 3, 2025 17:06
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