Summary and Per Request Report Generation

[Bslabe123]

We should define the shape of the output report and how to configure what gets reported. Keep in mind the output report may be extremely large or unreadable if all info is included by default such as per request metrics, per message inputs/outputs, etc.

ReportConfig API Proposal:

ReportConfig:
  Requests:
    Summary: null                   # If included, reports summary metrics for all requests (median, p50, p90, p99 for TPOT, input_len, output_len, etc)
    PerRequest:                     # If included, reports per request level metrics (start_time, end_time, input_len, output_len)
      IncludeInputs: boolean        # replace input_len with input request body
      IncludeOutputs: boolean       # replace output_len with output request body
  Prometheus:
    Summary: null                   # If included, report prometheus metrics query results with the window length equal to total experement time
    PerStage: null                  # If included, report prometheus metrics query results for each stage
    Periodic:
      Interval: uint                # Scrape metrics every {interval} seconds, include the results of each scrape including a timestamp

Related issues:

• [Feature] Collect Latency metrics - compute request time percentiles #30

[achandrasekar]

It'd be good to see the report format with filled in values to get an idea of how we expect this to look. Also, for the metrics, let's name them using widely used / agreed upon terminology. For example, you can refer to https://cloud.google.com/kubernetes-engine/docs/concepts/machine-learning/inference#performance.

[Bslabe123]

Current report looks like this, will make changes accordingly

{
  "observed": {
    "summary": {
      "load_summary": {
        "count": 121
      },
      "successes": {
        "count": 121,
        "time_per_request": {
          "mean": 2.07875813514548,
          "min": 0.166026983002666,
          "p10": 1.54146612802288,
          "p50": 1.86517220595852,
          "p90": 3.43353097402724,
          "max": 5.26778485300019
        },
        "prompt_len": {
          "mean": 428.785123966942,
          "min": 2,
          "p10": 11,
          "p50": 101,
          "p90": 1271,
          "max": 3836
        },
        "output_len": {
          "mean": 29.396694214876,
          "min": 0,
          "p10": 23,
          "p50": 32,
          "p90": 32,
          "max": 32
        },
        "per_token_latency": {
          "mean": 0.0728202762444499,
          "min": 0,
          "p10": 0.0535703550631297,
          "p50": 0.0595776435311564,
          "p90": 0.114333859532053,
          "max": 0.513822042674292
        }
      },
      "failures": {
        "count": 0,
        "time_per_request": null
      }
    }
  }
}

[achandrasekar]

/assign @Bslabe123

[achandrasekar]

Marking this as done since the report is now available and covers the initial set of items we wanted to add.

[achandrasekar]

Just realized we are not reporting throughput as a part of this. We need to add throughput in output tokens / sec, input tokens / sec, total tokens / sec and requests / sec.