Minor improvements to benchmarks#1386
Conversation
Bring DifferentiationInterface into the benchmarks env and adopt the flatter markdown layout (no <details> wrapper, no "Gist:" prefix). Released AbstractPPL/Bijectors are used instead of the fork-branch sources from the source branch. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
Pairs with the prior commit's benchmarks.jl markdown changes — the new workflow benches PR head and main side-by-side and wraps main's table in <details> on the CI side. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
Replace the PrettyTables benchmark report with a manual text formatter modeled on posteriordb-bench: top/bottom `=` rules, centered `eval` and `gradient` banners, dashed subgroup underlines, and a stub of Model/dim/linked columns. Keep the current pivoted data shape, with a shared `primal` column and backend ratio columns labelled FwdDiff, RvsDiff, Mooncake, and Enzyme. While there, simplify the renderer by formatting rows once up front and using a single backend key/label table as the source of truth. Update the PR comment caption to explain that `primal` is shared `t(logdensity)` and the backend columns are `t(grad)/t(logdensity)`. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
|
DynamicPPL.jl documentation for PR #1386 is available at: |
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #1386 +/- ##
=======================================
Coverage 82.26% 82.26%
=======================================
Files 50 50
Lines 3535 3535
=======================================
Hits 2908 2908
Misses 627 627 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
Restructure the comment so the table comes first, followed by a
single paragraph explaining what each column means and how to read
the AD backend ratios. Update the surrounding workflow text:
- "## Benchmark Report" + separate PR head/Main lines collapsed into
a single "## Benchmarks @ <sha>" heading.
- Foldout summaries shortened to "Main @ <sha>" and "Environment".
- Comparison hint ("compare against `main`") only appears when the
baseline foldout is actually available.
Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
Benchmarks @ 8f48885Each row times one of DynamicPPL's reference models on this PR's head. Main @ 2691e7cGist: SmorgasbordFull table (68 rows)EnvironmentJulia Version 1.11.9 Commit 53a02c0720c (2026-02-06 00:27 UTC) Build Info: Official https://julialang.org/ release Platform Info: OS: Linux (x86_64-linux-gnu) CPU: 4 × Intel(R) Xeon(R) Platinum 8370C CPU @ 2.80GHz WORD_SIZE: 64 LLVM: libLLVM-16.0.6 (ORCJIT, icelake-server) Threads: 1 default, 0 interactive, 1 GC (on 4 virtual cores) |
| benchmarks/results.md | ||
| benchmarks/version_info.txt | ||
|
|
||
| benchmark-main: |
There was a problem hiding this comment.
This seems to kind of revert the previous PR but now the comparision rartios have been removed? Quite confused about this churn.
No description provided.