Display performance graphs with artifact links in GitHub Actions summary#205
Closed
Copilot wants to merge 3 commits into
Closed
Display performance graphs with artifact links in GitHub Actions summary#205Copilot wants to merge 3 commits into
Copilot wants to merge 3 commits into
Conversation
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
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>
Contributor
Author
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: 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
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
The
collect-results.ymlworkflow 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.pytput_vs_intvty_*_{exp_name}.pngandtput_vs_e2el_*_{exp_name}.pngWorkflow update:
.github/workflows/collect-results.yml$GITHUB_STEP_SUMMARYgithub.repository,github.run_id) to the scriptResult
All workflows calling
collect-results.ymlnow display links to performance graphs in the Actions summary: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.