Fixed a bug in the physical predictor where it tries to calculate accuracy even if the operator produce zero columns

Author: ThatUdeshUK

src/execution/operator/projection/physical_predict.cpp

@@ -148,25 +148,6 @@ OperatorResultType PhysicalPredict::Execute(ExecutionContext &context, DataChunk
 	}
 #endif
 
-	int positives = 0;
-	auto lhs = (float *)input.data[1].GetData();
-
-	auto neg = (float *)predictions.data[0].GetData();
-	auto pos = (float *)predictions.data[1].GetData();
-
-	for (size_t i = 0; i < input.size(); i++) {
-		if ((*pos > *neg && *lhs > 0) || (*pos < *neg && *lhs < 1)) {
-			positives++;
-		}
-		lhs++;
-		// rhs++;
-		neg++;
-		pos++;
-	}
-
-	state.stats->correct += positives;
-	state.stats->total += input.size();
-
 	chunk.Fuse(predictions);
 	return OperatorResultType::NEED_MORE_INPUT;
 }

third_party/predictors/llm_api/duckdb_llm_api.cpp

@@ -140,6 +140,10 @@ void LlmApiPredictor::PredictChunk(DataChunk &input, DataChunk &output, int rows
 		end = std::chrono::steady_clock::now();
 		stats->move_rev += std::chrono::duration_cast<std::chrono::microseconds>(end - begin).count();
 #endif
+
+		// TODO: Calculate the accuracy metric here for the prediction and update the stats.correct and stats.total
+		// stats->correct += <no_of_positives>;
+		// stats->total += rows;
 	}
 }