Skip to content

Conversation

DaltheCow
Copy link
Collaborator

@DaltheCow DaltheCow commented Oct 1, 2025

Summary

We want to use ITL instead of TPOT. The data we had previously happened to be ITL data, but all of the labels indicate that it is TPOT data. Now the code and labels reflect that it is ITL data.

Test Plan

  • Everything works, tests pass, No use of TPOT in the UI

@DaltheCow DaltheCow added the UI Front-end workstream label Oct 1, 2025
Copy link

github-actions bot commented Oct 1, 2025

🧹 The live preview for this PR has been removed.

github-actions bot pushed a commit that referenced this pull request Oct 1, 2025
github-actions bot pushed a commit that referenced this pull request Oct 1, 2025
sjmonson
sjmonson previously approved these changes Oct 3, 2025
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, I'm going to fast-track a merge

## Summary

Update various places where ttft and itl were abbreviated but other
units were not.

## Details

- update metrics summary section
- update metrics details section

## Test Plan

Visually everything is correct, as long as tests pass it should be fine

## Related Issues

- Resolves #276
github-actions bot pushed a commit that referenced this pull request Oct 3, 2025
@sjmonson sjmonson merged commit 108a657 into main Oct 3, 2025
17 checks passed
@sjmonson sjmonson deleted the fix/itl-use-in-ui branch October 3, 2025 14:35
github-actions bot pushed a commit that referenced this pull request Oct 3, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
UI Front-end workstream
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants