Fix CI: make classifications optional in OtherExperimentLogFields, unexport classifierName

Author: aswink

js/src/framework.ts

@@ -884,7 +884,7 @@ export function scorerName(
   return scorer.name || `scorer_${scorer_idx}`;
 }
 
-export function classifierName(
+function classifierName(
   classifier: EvalClassifier<any, any, any, any>,
   classifier_idx: number,
 ) {

js/util/object.ts

@@ -21,7 +21,7 @@ export type OtherExperimentLogFields = {
   error: unknown;
   tags: string[];
   scores: Record<string, number | null>;
-  classifications: Record<string, { id: string; label: string }[]>;
+  classifications?: Record<string, { id: string; label: string }[]>;
   metadata: Record<string, unknown>;
   metrics: Record<string, unknown>;
   datasetRecordId: string;