Unverified Commit 2669a0d7 authored by Seiji Eicher's avatar Seiji Eicher Committed by GitHub
Browse files

Fix ValueError: Missing value for tag key(s): model_name,engine. (#19113)


Signed-off-by: default avatarSeiji Eicher <seiji@anyscale.com>
parent 8e972d9c
...@@ -47,12 +47,15 @@ def test_engine_log_metrics_ray( ...@@ -47,12 +47,15 @@ def test_engine_log_metrics_ray(
engine_args, stat_loggers=[RayPrometheusStatLogger]) engine_args, stat_loggers=[RayPrometheusStatLogger])
for i, prompt in enumerate(example_prompts): for i, prompt in enumerate(example_prompts):
engine.generate( results = engine.generate(
request_id=f"request-id-{i}", request_id=f"request-id-{i}",
prompt=prompt, prompt=prompt,
sampling_params=SamplingParams(max_tokens=max_tokens), sampling_params=SamplingParams(max_tokens=max_tokens),
) )
async for _ in results:
pass
# Create the actor and call the async method # Create the actor and call the async method
actor = EngineTestActor.remote() # type: ignore[attr-defined] actor = EngineTestActor.remote() # type: ignore[attr-defined]
ray.get(actor.run.remote()) ray.get(actor.run.remote())
...@@ -31,6 +31,16 @@ class RayPrometheusMetric: ...@@ -31,6 +31,16 @@ class RayPrometheusMetric:
self.metric.set_default_tags(labelskwargs) self.metric.set_default_tags(labelskwargs)
if labels:
if len(labels) != len(self.metric._tag_keys):
raise ValueError(
"Number of labels must match the number of tag keys. "
f"Expected {len(self.metric._tag_keys)}, got {len(labels)}"
)
self.metric.set_default_tags(
dict(zip(self.metric._tag_keys, labels)))
return self return self
......
Markdown is supported
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment