Skip to content

feat(evals): run judges at low inference priority - #6794

Merged
u9g merged 1 commit into
mainfrom
fix/drop-hotel-judge-group
Aug 11, 2026
Merged

feat(evals): run judges at low inference priority#6794
u9g merged 1 commit into
mainfrom
fix/drop-hotel-judge-group

Conversation

@u9g

@u9g u9g commented Aug 11, 2026

Copy link
Copy Markdown
Contributor

Judges fan out concurrently the moment a session ends (asyncio.gather over every judge), so each run adds a burst of gateway requests that no caller is waiting on. Today those requests carry no priority header at all — JudgeGroup never passes inference_class, and the header is only set when it is truthy — so they land in the same lane as live voice traffic and eat into the same RPM/TPM quota.

  • InferenceClass gains low alongside priority / standard.
  • JudgeGroup passes inference_class="low" when it builds the LLM from a model string. Passing a configured LLM instance still wins, so callers keep full control.

This is the framework half of the problem raised in #team-inference on 2026-07-20: simulations hitting the LLM requests-per-minute limit, with LLM-as-a-judge named as the driver. Gateway-side scheduling is tracked separately in LKINF-387.

Needs confirming before merge: the low value matches the header value quoted in #team-inference (X-LiveKit-Inference-Priority: low), but I have not verified the gateway accepts it. If unknown values are ignored this is inert until LKINF-387 lands; if the gateway validates and rejects, it would break judge runs.

Cross-SDK parity: agents-js has the same union at agents/src/inference/llm.ts and needs the same widening. It has no JudgeGroup, so the default has no counterpart there.

@u9g
u9g requested a review from a team as a code owner August 11, 2026 19:38

@devin-ai-integration devin-ai-integration Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

✅ Devin Review: No Issues Found

Devin Review analyzed this PR and found no bugs or issues to report.

Open in Devin Review

Judges fan out concurrently once a session ends, so a run adds a burst of
gateway requests that no caller is waiting on. Tagging them `low` keeps
them out of the quota that live voice traffic competes for.
@u9g
u9g force-pushed the fix/drop-hotel-judge-group branch from b91d0d5 to 58f01b3 Compare August 11, 2026 19:59
@u9g u9g changed the title chore(hotel_receptionist): drop the judge group from on_session_end feat(evals): run judges at low inference priority Aug 11, 2026
@u9g
u9g merged commit 2f29ed8 into main Aug 11, 2026
24 checks passed
@u9g
u9g deleted the fix/drop-hotel-judge-group branch August 11, 2026 20:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants