Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merge LLM Metrics changes to main #855

Merged
merged 7 commits into from
Apr 8, 2024
Merged

Merge LLM Metrics changes to main #855

merged 7 commits into from
Apr 8, 2024

Conversation

nv-braf
Copy link
Contributor

@nv-braf nv-braf commented Apr 8, 2024

Merges the LLM Metrics changes from the branch to main

nv-braf and others added 7 commits April 8, 2024 15:58
* Adding new LLM metrics

* Adding base class for perf, inter_token, and time_to_first latency records
* Adding CLI hook for LLM

* Changing to use --model-type
* Successfully reading from LLM CSV

* General cleanup

* All unit tests passing

* Fixing metric table typos

* Fixing typos
* Adding LLM values to list of possible constraints

* Fixing typo
* Initial changes to run genai-perf in MA

* Gating call to get LLM records

* Fixing captilization issue

* Removing debug

* Adding TODO

---------

Co-authored-by: root <[email protected]>
* Added support for genai_perf CLI

* Remove dead code

* Removing genai_perf collateral

* Fixing codeQL issue

* Adding streaming to genai_perf_config
@nv-braf nv-braf requested a review from debermudez April 8, 2024 22:41
@nv-braf nv-braf merged commit 8298d83 into main Apr 8, 2024
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants