Skip to content

Commit 8a5a9d2

Browse files
committed
don't omit fields for MongoDB storage
1 parent 356512b commit 8a5a9d2

File tree

1 file changed

+4
-2
lines changed

1 file changed

+4
-2
lines changed

tests/accuracy/sdk/accuracyResultStorage/mongodbStorage.ts

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -8,8 +8,10 @@ import {
88
ModelResponse,
99
} from "./resultStorage.js";
1010

11-
// Omitting these as they might contain large chunk of texts
12-
const OMITTED_MODEL_RESPONSE_FIELDS: (keyof ModelResponse)[] = ["messages", "text"];
11+
// We could decide to omit some fields from the model response to reduce the size of the stored results. Since
12+
// so far, the responses are not too big, we do not omit any fields, but if we decide to do so in the future,
13+
// we could add `"messages"` and `"text"` to this list.
14+
const OMITTED_MODEL_RESPONSE_FIELDS: (keyof ModelResponse)[] = [];
1315

1416
export class MongoDBBasedResultStorage implements AccuracyResultStorage {
1517
private client: MongoClient;

0 commit comments

Comments
 (0)