Revert Summary Metrics and Expand Test Coverage to Stabilize Nightly/Main CI #58
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.
Summary:
Reverts the summary metrics logic in
src/guidellm/core/report.py
andsrc/guidellm/core/result.py
that was landed due to failing tests. Additionally, test cases are expanded to ensure full coverage of these changes and to stabilize the nightly and main CI pipelines.Details:
prompt_token
,output_token
) with distribution-based calculations (prompt_token_distribution
,output_token_distribution
).computed_field
annotations for several properties insrc/guidellm/core/result.py
.tests/unit/core/test_report.py
from@pytest.mark.regression
to@pytest.mark.sanity
to better align with the testing standards.Test Plan: