Skip to content

Use PR target branch as synthesis diff baseline#234

Draft
Copilot wants to merge 2 commits into
mainfrom
copilot/update-synthesis-diffs-target-branch
Draft

Use PR target branch as synthesis diff baseline#234
Copilot wants to merge 2 commits into
mainfrom
copilot/update-synthesis-diffs-target-branch

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented May 23, 2026

Synthesis metrics diffs were still framed around main, which is incorrect for PRs targeting non-main branches (e.g. multicycle). This change makes the diff/report context track the PR base branch.

  • Workflow baseline context

    • Added BASE_BRANCH: ${{ github.base_ref }} to synthesis diff jobs in ci.yaml so scripts receive the PR target branch explicitly.
  • Per-metric diff script updates

    • Generalized compare_metrics_diff.sh from main-specific naming/messages to base-branch naming.
    • Updated fallback text to report missing baseline for the actual target branch rather than hardcoded main.
  • Aggregated report wording

    • Updated aggregate_metrics.sh footer text to say comparison is against $BASE_BRANCH instead of main.
# .github/workflows/ci.yaml
env:
  BASE_BRANCH: ${{ github.base_ref }}
# compare_metrics_diff.sh / aggregate_metrics.sh
BASE_BRANCH="${BASE_BRANCH:-base branch}"
echo "*Comparing synthesis results from \`$BASE_BRANCH\` branch vs. this PR*"

Agent-Logs-Url: https://github.com/UTOSS/risc-v/sessions/76a4b27e-aa44-4617-8561-bdc6504860fc

Co-authored-by: TheDeepestSpace <11333378+TheDeepestSpace@users.noreply.github.com>
Copilot AI changed the title [WIP] Update synthesis diffs to use PR target branch Use PR target branch as synthesis diff baseline May 23, 2026
Copilot AI requested a review from TheDeepestSpace May 23, 2026 03:32
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.

Make synthesis diffs use PR's target branch instead of main branch

2 participants