Skip to content

Display performance graphs with artifact links in GitHub Actions summary#205

Closed
Copilot wants to merge 3 commits into
mainfrom
copilot/display-tput-graphs-summary
Closed

Display performance graphs with artifact links in GitHub Actions summary#205
Copilot wants to merge 3 commits into
mainfrom
copilot/display-tput-graphs-summary

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Nov 9, 2025

The collect-results.yml workflow generates tput_vs_intvty and tput_vs_e2el performance graphs but only uploads them as artifacts, requiring users to download files to view results.

Changes

  • New utility: utils/display_graphs.py

    • Finds generated PNG files matching tput_vs_intvty_*_{exp_name}.png and tput_vs_e2el_*_{exp_name}.png
    • Generates markdown links to the GitHub Actions artifacts page
    • Groups graphs by type (interactivity vs e2e latency) and model family
    • Accepts repository info, run ID, and artifact name to construct proper URLs
  • Workflow update: .github/workflows/collect-results.yml

    • Added step after artifact upload to display graph links in $GITHUB_STEP_SUMMARY
    • Passes GitHub context variables (github.repository, github.run_id) to the script
    • Preserves existing artifact upload for archival

Result

All workflows calling collect-results.yml now display links to performance graphs in the Actions summary:

## Performance Graphs

### Throughput vs Interactivity
#### 70B
📊 [tput_vs_intvty_70b_all.png](https://github.com/InferenceMAX/InferenceMAX/actions/runs/12345/artifacts) (download `graphs_all` artifact)

### Throughput vs End-to-End Latency
#### 70B
📊 [tput_vs_e2el_70b_all.png](https://github.com/InferenceMAX/InferenceMAX/actions/runs/12345/artifacts) (download `graphs_all` artifact)

This approach provides direct links to the artifacts page with clear download instructions, avoiding issues with large embedded images.

View original Slack conversation

View original Slack conversation


💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

Co-authored-by: functionstackx <47992694+functionstackx@users.noreply.github.com>
Copilot AI changed the title [WIP] Add tput vs intvt and e2el graphs to GitHub summary Display performance graphs inline in GitHub Actions summary Nov 9, 2025
Copilot AI requested a review from functionstackx November 9, 2025 21:25
@functionstackx
Copy link
Copy Markdown
Collaborator

@copilot doing base64 encoding didn't work, can u try embedding the link into the markdown of the artifact?

https://github.com/InferenceMAX/InferenceMAX/actions/runs/19214637041?pr=205

Co-authored-by: functionstackx <47992694+functionstackx@users.noreply.github.com>
Copy link
Copy Markdown
Contributor Author

Copilot AI commented Nov 9, 2025

@copilot doing base64 encoding didn't work, can u try embedding the link into the markdown of the artifact?

https://github.com/InferenceMAX/InferenceMAX/actions/runs/19214637041?pr=205

Changed to use artifact links instead of base64 encoding. The script now generates markdown links to the artifacts page with clear download instructions. Commit: c596fec

The summary will now display:

📊 [filename.png](artifact_url) (download `graphs_all` artifact)

The "Display graphs in summary" step now runs after uploading artifacts and receives the repository info, run ID, and artifact name to construct proper links.

Copilot AI changed the title Display performance graphs inline in GitHub Actions summary Display performance graphs with artifact links in GitHub Actions summary Nov 9, 2025
@functionstackx functionstackx deleted the copilot/display-tput-graphs-summary branch December 4, 2025 21:31
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.

2 participants