Skip to content

Commit df997ad

Browse files
SudipSinhaclaude
andcommitted
fix(api): add deprecated id alias in MMD and streaming KS schedule list responses
The id alias was added to compare_means, kolmogorov_smirnov, and jensen_shannon in PR #182 but missed in mmd.py and kolmogorov_smirnov_streaming.py, causing KeyError: 'id' in integration tests for approxkstest and fouriermmd schedules. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
1 parent 6b1179a commit df997ad

2 files changed

Lines changed: 2 additions & 0 deletions

File tree

src/endpoints/metrics/drift/kolmogorov_smirnov_streaming.py

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -336,6 +336,7 @@ async def list_ksteststreaming_requests() -> dict[str, list[dict[str, Any]]]:
336336
and hasattr(request, "fit_columns")
337337
):
338338
request_dict = {
339+
"id": str(request_id), # deprecated: use requestId
339340
"requestId": str(request_id),
340341
"modelId": request.model_id,
341342
"metricName": METRIC_NAME,

src/endpoints/metrics/drift/mmd.py

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -307,6 +307,7 @@ async def list_mmd_requests() -> dict[str, list[dict[str, Any]]]:
307307
):
308308
requests_list.append(
309309
{
310+
"id": str(request_id), # deprecated: use requestId
310311
"requestId": str(request_id),
311312
"modelId": request.model_id,
312313
"metricName": METRIC_NAME,

0 commit comments

Comments
 (0)