Skip to content

Publish validated DAE benchmark outputs - #43

Merged
ChrisRackauckas merged 3 commits into
SciML:mainfrom
ChrisRackauckas-Claude:publish/dae-pr1670-20260828
Aug 28, 2026
Merged

Publish validated DAE benchmark outputs#43
ChrisRackauckas merged 3 commits into
SciML:mainfrom
ChrisRackauckas-Claude:publish/dae-pr1670-20260828

Conversation

@ChrisRackauckas-Claude

@ChrisRackauckas-Claude ChrisRackauckas-Claude commented Aug 28, 2026

Copy link
Copy Markdown
Member

Important

Ignore this PR until it has been reviewed by @ChrisRackauckas. This output was generated from SciML/SciMLBenchmarks.jl#1670 and must not merge before that source PR.

What changed and why

Publish the complete locally validated DAE benchmark output from SciMLBenchmarks commit a6400cd1100af968fc678fd717f0416f60fbc280. The current output predates the DAE runtime and dependency repairs; this refresh replaces all 14 generated pages and scripts, updates 122 referenced figures, and removes seven obsolete figure files.

This draft lets the already-completed five-hour weave be reviewed and published without consuming one of the three congested CPU runners again. It is intentionally stacked behind the source PR so the hosted output never gets ahead of the benchmark code on master.

Verification

The source weave completed all 14 DAE pages at the exact PR head in 5:16:04 with exit code 0, producing 122 figures. No generated page contains an embedded ERROR:, Stacktrace:, UndefVarError, LoadError, or MethodError.

The output audit reported:

pages=14
references=122
figures=122
missing_references=0
unreferenced_figures=0
embedded_exception_pages=0
appendix_pages=14
script_files=14

All generated scripts are Runic-formatted and parse:

parsed 14 formatted generated scripts

The local documentation build completed successfully with Julia 1.12:

julia +1.12 --project=docs -e 'using Pkg; Pkg.develop(PackageSpec(path=pwd())); Pkg.instantiate()'
julia +1.12 --project=docs docs/make.jl
[ Info: Automatic `version="1.0.0"` for inventory from ../Project.toml
┌ Warning: Documenter could not auto-detect the building environment. Skipping deployment.

git diff --check passes. typos over the generated diff reports solver variables and truncated rendered type dumps such as caf, ba, and Nothin; the authored SciMLBenchmarks source diff passed its spelling check.

Not verified

The generated site was built but not deployed locally. The output should not merge until SciML/SciMLBenchmarks.jl#1670 is reviewed and merged.

🤖 Generated with Claude Code

https://chatgpt.com/codex/tasks/01a03a17-ad6f-7131-82fc-d0fd57ea6512

Co-Authored-By: Chris Rackauckas <accounts@chrisrackauckas.com>
Co-Authored-By: Claude <noreply@anthropic.com>
Claude-Session: https://chatgpt.com/codex/tasks/01a03a17-ad6f-7131-82fc-d0fd57ea6512
ChrisRackauckas and others added 2 commits August 28, 2026 19:29
Co-Authored-By: Chris Rackauckas <accounts@chrisrackauckas.com>
Co-Authored-By: Claude <noreply@anthropic.com>
Claude-Session: https://chatgpt.com/codex/tasks/01a03a17-ad6f-7131-82fc-d0fd57ea6512
Co-Authored-By: Chris Rackauckas <accounts@chrisrackauckas.com>
Co-Authored-By: Claude <noreply@anthropic.com>
Claude-Session: https://chatgpt.com/codex/tasks/01a03a17-ad6f-7131-82fc-d0fd57ea6512
@ChrisRackauckas-Claude

Copy link
Copy Markdown
Member Author

Runic 1.10.0 was released today, and the workflow's runic-version: "1" constraint resolved it while the generated scripts had been formatted with Runic 1.9.0. The latest commit applies the only two added-line changes reported by the workflow.

Local recheck with Julia 1.12.7 / Runic 1.10.0:

pkgversion(Runic) = v"1.10.0"
script/DAE/slider_crank.jl: exit 0
all changed DAE scripts: 14/14, exit 0
git diff --check: exit 0

The repository-wide historical formatter debt remains outside this generated-output PR; the workflow filters it with filter_mode: added.

@ChrisRackauckas
ChrisRackauckas marked this pull request as ready for review August 28, 2026 23:56
@ChrisRackauckas
ChrisRackauckas merged commit 5edbf22 into SciML:main Aug 28, 2026
2 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