Skip to content

Conversation

jaredoconnell
Copy link
Collaborator

Summary

Fixes HTML output in the refactor branch.

Details

This PR updates the references for the new data, and corrects a problem that prevented the injection of the data into the HTML. They broke in the refactor.

Here is a screenshot of the current output using this branch:
image

There will be a follow up PR to make HTML and CSV outputs default in the CLI.

Test Plan

To test this you need to locally host the web server as defined in the readme, then run the following command, with variables replaced for your setup:
guidellm benchmark run --target <your target> --data "prompt_tokens=256,output_tokens=128" --rate-type constant --rate 2 --max-seconds 10 --output-path <your output path> --output-formats html


  • "I certify that all code in this PR is my own, except as noted below."

Use of AI

  • Includes AI-assisted code completion
  • Includes code generated by an AI application
  • Includes AI-generated tests (NOTE: AI written tests should have a docstring that includes ## WRITTEN BY AI ##)

@jaredoconnell jaredoconnell force-pushed the features/refactor-fix-html branch from 05bca97 to 54556ae Compare September 26, 2025 15:38
@sjmonson sjmonson requested a review from DaltheCow September 26, 2025 21:34
@markurtz markurtz deleted the branch vllm-project:features/refactor/working September 29, 2025 16:37
@markurtz markurtz closed this Sep 29, 2025
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