Skip to content

Re-enable benchmark workflows with label gating#207

Open
auto-differentiation-dev wants to merge 2 commits intomainfrom
feature/re-enable-benchmarks
Open

Re-enable benchmark workflows with label gating#207
auto-differentiation-dev wants to merge 2 commits intomainfrom
feature/re-enable-benchmarks

Conversation

@auto-differentiation-dev
Copy link
Copy Markdown
Collaborator

@auto-differentiation-dev auto-differentiation-dev commented Apr 1, 2026

Summary

  • Re-enable libor swaption and QuantLib benchmark jobs (were disabled with if: false)
  • Gate benchmark runs via benchmarks label on PRs + workflow_dispatch for reference generation
  • Refactor collect job to upload artifacts instead of commenting directly, enabling benchmarkresults.yml to post PR comments (works for fork PRs)
  • Update QuantLibAAD pin from stale commit hash to main

Closes #161

Test plan

  • Run workflow_dispatch on main to generate fresh S3 reference executables
  • Open a test PR with benchmarks label and verify benchmarks run
  • Verify benchmarkresults.yml posts performance comparison comment

@codacy-production
Copy link
Copy Markdown

codacy-production bot commented Apr 1, 2026

Up to standards ✅

🟢 Issues 0 issues

Results:
0 new issues

View in Codacy

TIP This summary will be updated as you push new changes. Give us feedback

@coveralls
Copy link
Copy Markdown
Collaborator

coveralls commented Apr 1, 2026

Pull Request Test Coverage Report for Build 23858635150

Details

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 0.0%

Totals Coverage Status
Change from base Build 23855818590: 0.0%
Covered Lines: 0
Relevant Lines: 0

💛 - Coveralls

@github-actions
Copy link
Copy Markdown

github-actions bot commented Apr 1, 2026

Test Results

    111 files  ±0      111 suites  ±0   5h 26m 35s ⏱️ - 13m 55s
  2 989 tests ±0    2 985 ✅ ±0    4 💤 ±0  0 ❌ ±0 
217 529 runs  ±0  217 085 ✅ ±0  444 💤 ±0  0 ❌ ±0 

Results for commit e202947. ± Comparison against base commit 3d8f365.

♻️ This comment has been updated with latest results.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add performance tests to the CI/CD pipeline

2 participants