-
Notifications
You must be signed in to change notification settings - Fork 31
feat: Report histogram metrics to Triton metrics server #58
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
Merged
yinggeh
merged 36 commits into
main
from
yinggeh-DLIS-7113-support-histogram-metric-type
Aug 16, 2024
Merged
Changes from 35 commits
Commits
Show all changes
36 commits
Select commit
Hold shift + click to select a range
0686a7c
Add first supported metrics
yinggeh 21e2356
Update comments
yinggeh d95bb2c
Minor update
yinggeh 321faa0
Add metrics test
yinggeh 468539f
Fix copyright
yinggeh 8eba2f0
Remove unused metrics and update comments
yinggeh 6f97f6f
Minor update
yinggeh bf7669e
Minor updates
yinggeh e9d0dbb
Minor fix
yinggeh 7d0dc5b
Remove unused module
yinggeh 979dc02
Fix "metrics not supported error" when building with TRITON_ENABLE_ME…
yinggeh 3dd04c5
Fix "metrics not supported error" when building with TRITON_ENABLE_ME…
yinggeh 07f2575
Simply test
yinggeh 2135145
Completely turn off metrics
yinggeh 56aea05
Add vLLM disable_log_stats config test
yinggeh 0dadc8e
Test metrics are enabled by default if disable_log_stats is not set.
yinggeh 8d8fd2a
Update tests based on comments
yinggeh 4f2e217
Remove _log_gauge
yinggeh d22fd03
Resolve comments
yinggeh c8bdb6e
Merge branch 'main' of github.com:triton-inference-server/vllm_backen…
yinggeh 8280d26
Update
yinggeh 6fa7ae3
Change temp directory
yinggeh 89ca6f4
Disable metrics report by default. Controlled by parameter "REPORT_ME…
yinggeh 1158fee
Test server option set --allow-metrics=false
yinggeh a99d38b
Add docs
yinggeh de8f25b
Minor update
yinggeh b1333ce
Both args checking
yinggeh de7ff8f
Add histogram test
yinggeh 9534298
Longer time for A100
yinggeh 38ac8d6
Update comment
yinggeh ebdf14e
Add histogram metrics to doc
yinggeh 0d67322
Update docs
yinggeh 10d8a69
Make metrics test more robust
yinggeh b7f4002
Merge branch 'main' into yinggeh-DLIS-7113-support-histogram-metric-type
yinggeh bba181e
Remove empty newline
yinggeh 2d7488d
Change assertTrue(a>b) to assertGreater(a, b)
yinggeh File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
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.
Uh oh!
There was an error while loading. Please reload this page.