Content-Length: 2170 | pFad | http://github.com/postgresml/postgresml/pull/1010.patch

thub.com From ad41d19ffadbd6bc1387524b6ba1f3ac9e0c1741 Mon Sep 17 00:00:00 2001 From: Montana Low Date: Tue, 12 Sep 2023 11:18:38 -0700 Subject: [PATCH 1/2] handle errors cleanly rather than crashing --- pgml-extension/src/metrics.rs | 8 ++++++-- 1 file changed, 6 insertions(+), 2 deletions(-) diff --git a/pgml-extension/src/metrics.rs b/pgml-extension/src/metrics.rs index 4526d971e..1e726efe3 100644 --- a/pgml-extension/src/metrics.rs +++ b/pgml-extension/src/metrics.rs @@ -51,13 +51,17 @@ impl ConfusionMatrix { y_hat: &ArrayView1, num_classes: usize, ) -> ConfusionMatrix { - assert_eq!(ground_truth.len(), y_hat.len()); + if ground_truth.len() != y_hat.len() { + error!("Can't compute metrics when the ground truth labels are a different size than the predicted labels. {} != {}", ground_truth.len(), y_hat.len()) + }; // Distinct classes. let mut classes = ground_truth.iter().collect::>(); classes.extend(&mut y_hat.iter().collect::>().into_iter()); - assert_eq!(num_classes, classes.len()); + if num_classes != classes.len() { + error!("Can't compute metrics when the number of classes in the test set is different than the number of classes in the training set. {} != {}", num_classes, classes.len()) + }; // Class value = index in the confusion matrix // e.g. class value 5 will be index 4 if there are classes 1, 2, 3 and 4 present. From bf7e1acd10ff1b61034ace964cf1f5bcd573f391 Mon Sep 17 00:00:00 2001 From: Montana Low Date: Tue, 12 Sep 2023 11:22:00 -0700 Subject: [PATCH 2/2] use --- pgml-extension/src/metrics.rs | 1 + 1 file changed, 1 insertion(+) diff --git a/pgml-extension/src/metrics.rs b/pgml-extension/src/metrics.rs index 1e726efe3..c23f27b64 100644 --- a/pgml-extension/src/metrics.rs +++ b/pgml-extension/src/metrics.rs @@ -1,4 +1,5 @@ //github.com/ Module providing various metrics used to rank the algorithms. +use pgrx::*; use std::collections::{BTreeSet, HashMap}; use ndarray::{Array2, ArrayView1};








ApplySandwichStrip

pFad - (p)hone/(F)rame/(a)nonymizer/(d)eclutterfier!      Saves Data!


--- a PPN by Garber Painting Akron. With Image Size Reduction included!

Fetched URL: http://github.com/postgresml/postgresml/pull/1010.patch

Alternative Proxies:

Alternative Proxy

pFad Proxy

pFad v3 Proxy

pFad v4 Proxy