Skip to content

Conversation

jaredoconnell
Copy link
Collaborator

Summary

This PR gets the CSV output to a state comparable to pre-refactor.

Details

Implements the functions to export the required data to the CSV format.

The goal is to include the required information in the CSV without cluttering it, but also without creating too much of a burden to the future maintainers resulting from referencing specific schema elements.

The following columns are new:

  • Profile
  • Backend
  • Generator Data

You can view these files in the attached output generated by the following test:
guidellm benchmark run --output-path result_7.csv --max-seconds 2 --target=http://localhost:8000 --data "prompt_tokens=256,output_tokens=128" --rate-type constant --rate 1 --output-formats csv

result_7.csv

Test Plan

Run GuideLLM with the following additional args: --output-path result.csv --output-formats csv

The generated file should have all info required.

Related Issues

This is a part of the scheduler refactor.


  • "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 ##)

Copy link
Collaborator

@sjmonson sjmonson left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. Will approve when feature/refactor/main is ready.

@sjmonson sjmonson force-pushed the feature/refactor/draft-main branch from 61943e4 to fac0be4 Compare September 16, 2025 15:02
@jaredoconnell
Copy link
Collaborator Author

Replaced with #359

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.

3 participants