Skip to content

Commit 7258bd2

Browse files
fix: clear occursin_info cache in @be macro
1 parent b4bf625 commit 7258bd2

File tree

2 files changed

+4
-4
lines changed

2 files changed

+4
-4
lines changed

benchmarks/Symbolics/BCR.jmd

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -93,8 +93,7 @@ function run_and_time_construct!(rhs, vars, pars, iv, N, i, jac_times, jac_alloc
9393
jac_nohc = Symbolics.sparsejacobian(outputs, vars)
9494

9595
SymbolicUtils.ENABLE_HASHCONSING[] = true
96-
SymbolicUtils.clear_cache!(Symbolics.occursin_info)
97-
jac_result = @be Symbolics.sparsejacobian(outputs, vars)
96+
jac_result = @be (SymbolicUtils.clear_cache!(Symbolics.occursin_info); Symbolics.sparsejacobian(outputs, vars))
9897
jac_times[2][i] = minimum(x -> x.time, jac_result.samples)
9998
jac_allocs[2][i] = minimum(x -> x.bytes, jac_result.samples)
10099
jac_hc = Symbolics.sparsejacobian(outputs, vars)
@@ -147,6 +146,7 @@ end
147146
# Run benchmark
148147

149148
```julia
149+
Chairmarks.DEFAULTS.seconds = 15.0
150150
N = [10, 20, 40, 80, 160, 320]
151151
jacobian_times = [zeros(Float64, length(N)), zeros(Float64, length(N))]
152152
jacobian_allocs = copy.(jacobian_times)

benchmarks/Symbolics/ThermalFluid.jmd

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -264,8 +264,7 @@ function run_and_time_construction!(jacobian_times, jacobian_gctimes, jacobian_a
264264
jacobian_allocs[1][i] = mean(x -> x.bytes, jac_result.samples)
265265
@info "times" jacobian_times[1][i] jacobian_gctimes[1][i] jacobian_allocs[1][i]
266266
SymbolicUtils.ENABLE_HASHCONSING[] = true
267-
SymbolicUtils.clear_cache!(Symbolics.occursin_info)
268-
jac_result = @be calculate_jacobian(sys; sparse = true)
267+
jac_result = @be (SymbolicUtils.clear_cache!(Symbolics.occursin_info); calculate_jacobian(sys; sparse = true))
269268
@info "Hashconsing benchmark"
270269
jac_cse = calculate_jacobian(sys; sparse = true)
271270
@info "Hashconsing result"
@@ -350,6 +349,7 @@ second_call_times = copy.(jacobian_times)
350349
## Timings
351350

352351
```julia
352+
Chairmarks.DEFAULTS.seconds = 15.0
353353
# compile
354354
run_and_time_construction!(jacobian_times, jacobian_gctimes, jacobian_allocs, build_times, functions, 1, 5)
355355
run_and_time_call!(functions, first_call_times, second_call_times, 1)

0 commit comments

Comments
 (0)