Skip to content

Apply Runic formatting to test runtests.jl files - #507

Merged
ChrisRackauckas merged 1 commit into
SciML:masterfrom
ChrisRackauckas-Claude:runic-format-runtests
Jun 10, 2026
Merged

Apply Runic formatting to test runtests.jl files#507
ChrisRackauckas merged 1 commit into
SciML:masterfrom
ChrisRackauckas-Claude:runic-format-runtests

Conversation

@ChrisRackauckas-Claude

Copy link
Copy Markdown
Member

Summary

Fixes the failing format-check (Runic) workflow on master. The six runtests.jl files touched by the monorepo-structure uniformization (#505) were not Runic-formatted:

  • lib/BoundaryValueDiffEq{Ascher,FIRK,MIRK,MIRKN,Shooting}/test/runtests.jl
  • test/runtests.jl

format-check has been red on master since commit 97e8de7 / f86a8c8-era pushes (first red run: 2026-06-05, consistently red since #505 merged).

Pure formatting change produced by Runic.main(["--inplace", "."]); verified locally that Runic.main(["--check", "."]) now exits 0 and all six files still parse.

This PR should be ignored until reviewed by @ChrisRackauckas.

🤖 Generated with Claude Code

Fixes the failing format-check workflow on master: the six runtests.jl
files introduced/edited in the monorepo-structure uniformization (SciML#505)
were not Runic-formatted.

Co-Authored-By: Chris Rackauckas <accounts@chrisrackauckas.com>
@github-actions

Copy link
Copy Markdown
Contributor

Benchmark Results (Julia v1.11)

Time benchmarks
master c2f5fd7... master / c2f5fd7...
Simple Pendulum/IIP/BoundaryValueDiffEqMIRK.MIRK2() 0.558 ± 0.012 s 0.584 ± 0.017 s 0.956 ± 0.035
Simple Pendulum/IIP/BoundaryValueDiffEqMIRK.MIRK3() 11.7 ± 0.7 ms 11.9 ± 0.72 ms 0.983 ± 0.084
Simple Pendulum/IIP/BoundaryValueDiffEqMIRK.MIRK4() 2.66 ± 0.33 ms 2.69 ± 0.48 ms 0.989 ± 0.21
Simple Pendulum/IIP/BoundaryValueDiffEqMIRK.MIRK5() 2.97 ± 0.078 ms 3.23 ± 0.57 ms 0.92 ± 0.16
Simple Pendulum/IIP/BoundaryValueDiffEqMIRK.MIRK6() 1.44 ± 0.039 ms 1.5 ± 0.14 ms 0.957 ± 0.09
Simple Pendulum/IIP/MultipleShooting(10, Tsit5; grid_coarsening = false) 1.47 ± 0.082 ms 1.64 ± 0.57 ms 0.893 ± 0.32
Simple Pendulum/IIP/MultipleShooting(10, Tsit5; grid_coarsening = true) 2.96 ± 0.35 ms 4.05 ± 1.3 ms 0.731 ± 0.25
Simple Pendulum/IIP/MultipleShooting(100, Tsit5; grid_coarsening = false) 0.0392 ± 0.008 s 0.0475 ± 0.014 s 0.825 ± 0.3
Simple Pendulum/IIP/MultipleShooting(100, Tsit5; grid_coarsening = true) 0.0566 ± 0.0037 s 0.0562 ± 0.0025 s 1.01 ± 0.079
Simple Pendulum/IIP/Shooting(Tsit5()) 0.166 ± 0.088 ms 0.216 ± 0.1 ms 0.768 ± 0.54
Simple Pendulum/OOP/BoundaryValueDiffEqMIRK.MIRK2() 0.898 ± 0.1 s 0.72 ± 0.0092 s 1.25 ± 0.14
Simple Pendulum/OOP/BoundaryValueDiffEqMIRK.MIRK3() 21.5 ± 7.6 ms 16.9 ± 4 ms 1.27 ± 0.54
Simple Pendulum/OOP/BoundaryValueDiffEqMIRK.MIRK4() 4.01 ± 1 ms 3.15 ± 0.61 ms 1.28 ± 0.4
Simple Pendulum/OOP/BoundaryValueDiffEqMIRK.MIRK5() 4.54 ± 1.2 ms 3.78 ± 0.45 ms 1.2 ± 0.34
Simple Pendulum/OOP/BoundaryValueDiffEqMIRK.MIRK6() 1.99 ± 0.53 ms 1.76 ± 0.15 ms 1.13 ± 0.31
Simple Pendulum/OOP/MultipleShooting(10, Tsit5; grid_coarsening = false) 3.77 ± 3.5 ms 3.05 ± 1.1 ms 1.24 ± 1.2
Simple Pendulum/OOP/MultipleShooting(10, Tsit5; grid_coarsening = true) 7.23 ± 7 ms 6.3 ± 6.1 ms 1.15 ± 1.6
Simple Pendulum/OOP/MultipleShooting(100, Tsit5; grid_coarsening = false) 0.147 ± 0.07 s 0.0866 ± 0.0096 s 1.7 ± 0.83
Simple Pendulum/OOP/MultipleShooting(100, Tsit5; grid_coarsening = true) 0.204 ± 0.07 s 0.132 ± 0.017 s 1.55 ± 0.57
Simple Pendulum/OOP/Shooting(Tsit5()) 0.601 ± 0.23 ms 0.549 ± 0.032 ms 1.1 ± 0.43
time_to_load 7.2 ± 0.36 s 6.4 ± 0.091 s 1.13 ± 0.058
Memory benchmarks
master c2f5fd7... master / c2f5fd7...
Simple Pendulum/IIP/BoundaryValueDiffEqMIRK.MIRK2() 0.389 M allocs: 0.0442 GB 0.389 M allocs: 0.0442 GB 1
Simple Pendulum/IIP/BoundaryValueDiffEqMIRK.MIRK3() 0.0436 M allocs: 4.85 MB 0.0436 M allocs: 4.85 MB 1
Simple Pendulum/IIP/BoundaryValueDiffEqMIRK.MIRK4() 15.7 k allocs: 1.63 MB 15.7 k allocs: 1.63 MB 1
Simple Pendulum/IIP/BoundaryValueDiffEqMIRK.MIRK5() 22.1 k allocs: 2.02 MB 22.1 k allocs: 2.02 MB 1
Simple Pendulum/IIP/BoundaryValueDiffEqMIRK.MIRK6() 12.8 k allocs: 1.04 MB 12.8 k allocs: 1.04 MB 1
Simple Pendulum/IIP/MultipleShooting(10, Tsit5; grid_coarsening = false) 25.5 k allocs: 1.82 MB 25.5 k allocs: 1.82 MB 1
Simple Pendulum/IIP/MultipleShooting(10, Tsit5; grid_coarsening = true) 0.049 M allocs: 3.38 MB 0.049 M allocs: 3.38 MB 1
Simple Pendulum/IIP/MultipleShooting(100, Tsit5; grid_coarsening = false) 0.553 M allocs: 0.0535 GB 0.553 M allocs: 0.0535 GB 1
Simple Pendulum/IIP/MultipleShooting(100, Tsit5; grid_coarsening = true) 0.833 M allocs: 0.0778 GB 0.833 M allocs: 0.0778 GB 1
Simple Pendulum/IIP/Shooting(Tsit5()) 4.62 k allocs: 0.223 MB 4.62 k allocs: 0.223 MB 1
Simple Pendulum/OOP/BoundaryValueDiffEqMIRK.MIRK2() 0.89 M allocs: 0.984 GB 0.89 M allocs: 0.984 GB 1
Simple Pendulum/OOP/BoundaryValueDiffEqMIRK.MIRK3() 0.0932 M allocs: 24.7 MB 0.0932 M allocs: 24.7 MB 1
Simple Pendulum/OOP/BoundaryValueDiffEqMIRK.MIRK4() 0.0324 M allocs: 3.94 MB 0.0324 M allocs: 3.94 MB 1
Simple Pendulum/OOP/BoundaryValueDiffEqMIRK.MIRK5() 0.045 M allocs: 4.97 MB 0.045 M allocs: 4.97 MB 1
Simple Pendulum/OOP/BoundaryValueDiffEqMIRK.MIRK6() 25.3 k allocs: 2.16 MB 25.3 k allocs: 2.16 MB 1
Simple Pendulum/OOP/MultipleShooting(10, Tsit5; grid_coarsening = false) 0.142 M allocs: 10.2 MB 0.142 M allocs: 10.2 MB 1
Simple Pendulum/OOP/MultipleShooting(10, Tsit5; grid_coarsening = true) 0.266 M allocs: 18.7 MB 0.266 M allocs: 18.7 MB 1
Simple Pendulum/OOP/MultipleShooting(100, Tsit5; grid_coarsening = false) 2.52 M allocs: 0.279 GB 2.52 M allocs: 0.279 GB 1
Simple Pendulum/OOP/MultipleShooting(100, Tsit5; grid_coarsening = true) 3.82 M allocs: 0.404 GB 3.82 M allocs: 0.404 GB 1
Simple Pendulum/OOP/Shooting(Tsit5()) 0.0373 M allocs: 1.69 MB 0.0373 M allocs: 1.69 MB 1
time_to_load 0.161 k allocs: 11.4 kB 0.161 k allocs: 11.4 kB 1

@ChrisRackauckas
ChrisRackauckas marked this pull request as ready for review June 10, 2026 11:25
@ChrisRackauckas
ChrisRackauckas merged commit 0ea6963 into SciML:master Jun 10, 2026
8 of 52 checks passed
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