Repository navigation
HF-150 (alternative): report instruction-count changes above 0.25% instead of 0.5% - #1864
marcin-kordas-hoc wants to merge 2 commits into
Conversation
The workflow threshold of the executed-instruction verdict is halved, so smaller increases are named as regressions in the performance report. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Deploying with
|
| Status | Name | Latest Commit | Preview URL | Updated (UTC) |
|---|---|---|---|---|
| ✅ Deployment successful! View logs |
hyperformula-docs | 1b51b4f | Commit Preview URL Branch Preview URL |
Nov 03 2026, 07:08 AM |
Performance✅ Executed instructions: no regression above +0.25% in 15 scenarios.
Instruction countsInstructions counted by callgrind in the measured region of each scenario. The verdict compares the executed instructions: the count without garbage collection and JIT compilation, whose share moves with the state of the heap and the compiler, not only with the code under test. A change larger than ±0.25% of the base is reported. GC, Compilation and Total give the change of those parts and of the whole count, in percent of the base count.
Wall-clock times
Threshold of an unmarked scenario = max(5%, 3 x the standard error of the difference of the two medians). † = the band of changes observed on identical code in earlier runs, instead of that rule. ‡ = a fixed threshold for a scenario that guards against a change of complexity class. This check is advisory: it reports regressions but does not fail the build. head Executed instructions decide the verdict: they do not depend on the load of the runner. Garbage collection and JIT compilation are counted too but shown apart, in percent of the base count, because they move with the state of the heap and the compiler, not only with the code under test. Wall-clock times vary with the load of the runner and are shown for information. This report does not block merging. |
…-025 Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## feat/hf-150-perf-report #1864 +/- ##
========================================================
Coverage 97.28% 97.28%
========================================================
Files 207 207
Lines 16218 16218
Branches 3556 3556
========================================================
Hits 15777 15777
Misses 433 433
Partials 8 8 🚀 New features to boost your workflow:
|
Context
The PR lowers the threshold of the instruction-count verdict from 0.5% to 0.25%. It is one of four alternatives to #1852, stacked on it, and changes only that value.
setCellContents(HF-150 (throwaway control, do not merge): known extra work for the blocking instruction check #1863, run 41010016546), four scenarios besides the 5 regressions rose by +0.261% to +0.380% in executed instructions and were reportedokat 0.5%..github/workflows/performance.yml:PERF_INSTRUCTIONS_THRESHOLD: '0.25'.How did you test your changes?
feat/hf-150-alt-threshold-025inhyperformula-tests(README only).npm run test:jest -- unit/performancewith the paired tests: 5 suites, 101 tests passed. The tests branch changes no lintable file.Compatibility with Excel and Google Sheets
Not applicable.
Public API changes
None.
Out of scope and follow-ups
Types of changes
CI tooling only.
Related issues
Checklist
dev-docs/pages this change affects – or none needed it.🤖 Generated with Claude Code