Skip to content

Merge report stages and add wasm-jit to history/overview - #312

Merged
sjoelund merged 1 commit into
masterfrom
merge-report-stages
Aug 13, 2026
Merged

Merge report stages and add wasm-jit to history/overview#312
sjoelund merged 1 commit into
masterfrom
merge-report-stages

Conversation

@sjoelund

Copy link
Copy Markdown
Member
  • Merge ryzen-5950x-1 and ryzen-5950x-2 report stages into a single stage (Postgres makes this fast)

  • Add wasm-jit to all-reports.py/all-plots.py so history reports are generated for it

  • Include wasm-jit in overview-combined.html and overview-special-jobs.html alongside master and newInst-newBackend

- Merge ryzen-5950x-1 and ryzen-5950x-2 report stages into a single
  stage (Postgres makes this fast)

- Add wasm-jit to all-reports.py/all-plots.py so history reports are
  generated for it

- Include wasm-jit in overview-combined.html and overview-special-jobs.html
  alongside master and newInst-newBackend

Co-authored-by: Qwen 27B <qwen@alibaba.com>
@sjoelund

Copy link
Copy Markdown
Member Author

@coderabbitai - did any report stop getting generated from these changes?

@coderabbitai

coderabbitai Bot commented Aug 13, 2026

Copy link
Copy Markdown

Tip

For best results, initiate chat on the files or code changes.

🧩 Analysis chain

🏁 Script executed:

#!/bin/bash
set -euo pipefail

printf '%s\n' '== PR metadata =='
gh pr view 312 --json baseRefName,headRefName,files --jq '{base: .baseRefName, head: .headRefName, files: [.files[].path]}'

base="$(gh pr view 312 --json baseRefName --jq .baseRefName)"
printf '%s\n' '== Changed files =='
git diff --name-status "origin/${base}...HEAD"

printf '%s\n' '== Changed report-related code map =='
git diff --name-only "origin/${base}...HEAD" |
  while IFS= read -r file; do
    case "$file" in
      *.py|*.sh)
        ast-grep outline "$file" --items all 2>/dev/null || true
        ;;
    esac
  done

printf '%s\n' '== Report-generation changes =='
git diff --unified=80 "origin/${base}...HEAD" -- \
  '*all-reports.py' '*all-plots.py' '*overview-combined.html' '*overview-special-jobs.html' \
  '*report*' '*plot*' '*overview*'

Length of output: 1164


🏁 Script executed:

#!/bin/bash
set -euo pipefail

base="$(gh pr view 312 --json baseRefName --jq .baseRefName)"

printf '%s\n' '== Jenkinsfile diff =='
git diff --unified=120 "origin/${base}...HEAD" -- .CI/Jenkinsfile

printf '%s\n' '== Report and stage references on base =='
git show "origin/${base}:.CI/Jenkinsfile" |
  nl -ba |
  rg -n -i -C 8 'stage\s*\(|report|plot|overview|ryzen-5950x|wasm-jit|newInst|newBackend' || true

printf '%s\n' '== Report and stage references on PR head =='
nl -ba .CI/Jenkinsfile |
  rg -n -i -C 8 'stage\s*\(|report|plot|overview|ryzen-5950x|wasm-jit|newInst|newBackend' || true

Length of output: 30256


@sjoelund No report stopped getting generated.

The merged report stage retains all previous report.py calls and overview-*.html outputs. It also retains the final overview.html output.

The new when condition includes the trigger conditions from both previous report stages.

all-reports.py and all-plots.py retain their previous inputs. They add wasm-jit and generateSymbolicJacobian.

The publisher still uploads overview*.html and history/**.

The only operational change is that the reports now run sequentially in one stage instead of in two parallel stages.

You are interacting with an AI system.

@sjoelund
sjoelund enabled auto-merge (squash) August 13, 2026 09:20
@sjoelund
sjoelund merged commit d82101f into master Aug 13, 2026
7 checks passed
@sjoelund
sjoelund deleted the merge-report-stages branch August 13, 2026 09:39
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.

1 participant