Skip to content

Commit

Permalink
Update comments
Browse files Browse the repository at this point in the history
  • Loading branch information
yinggeh committed Nov 4, 2024
1 parent 3f7761d commit 7014fd7
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 3 deletions.
1 change: 1 addition & 0 deletions src/metric_model_reporter.cc
Original file line number Diff line number Diff line change
Expand Up @@ -304,6 +304,7 @@ void
MetricModelReporter::InitializeHistograms(
const std::map<std::string, std::string>& labels)
{
// Update MetricReporterConfig::metric_map_ for new histograms.
// Only create response metrics if decoupled model to reduce metric output
if (config_.latency_histograms_enabled_) {
if (config_.is_decoupled_) {
Expand Down
6 changes: 3 additions & 3 deletions src/metric_model_reporter.h
Original file line number Diff line number Diff line change
Expand Up @@ -80,9 +80,9 @@ struct MetricReporterConfig {

private:
// Maps the metric family fullname to its lookup key. This field is required
// because the users are expected to configure metric configuration "ModelMetrics"
// with the full name displayed from metrics reporting while a different name is used internally .
// All new histograms must update the map.
// because the users are expected to configure metric configuration
// "ModelMetrics" with the full name displayed from metrics reporting while a
// different name is used internally. All new histograms must update the map.
const std::unordered_map<std::string, std::string> metric_map_ = {
{"nv_inference_first_response_histogram_ms", kFirstResponseHistogram}};
#endif // TRITON_ENABLE_METRICS
Expand Down

0 comments on commit 7014fd7

Please sign in to comment.