From 47eefa0ccedffcaf6f93d3f55add41b4bdcc60c9 Mon Sep 17 00:00:00 2001
From: Muriithi Frederick Muriuki
Date: Thu, 22 Jul 2021 12:32:30 +0300
Subject: Check the child lengths are same as length of parent

Issue:
https://github.com/genenetwork/gn-gemtext-threads/blob/main/topics/gn1-migration-to-gn2/clustering.gmi

* Check that the child lists/tuples are the same length as the parent
  list/tuple. If any of the children is not the same length as the parent,
  throw and exception.
---
 gn3/computations/slink.py | 7 +++++++
 1 file changed, 7 insertions(+)

(limited to 'gn3')

diff --git a/gn3/computations/slink.py b/gn3/computations/slink.py
index dc71815..5288908 100644
--- a/gn3/computations/slink.py
+++ b/gn3/computations/slink.py
@@ -19,6 +19,12 @@ def raise_valueerror_if_lists_empty(lists):
     if (empty(lists)) or not all(map(lambda x: not empty(x), lists)):
         raise ValueError("List/Tuple should NOT be empty!")
 
+def raise_lengtherror_if_child_lists_are_not_same_as_parent(lists):
+    def len_is_same_as_parent(lst):
+        return len(lst) == len(lists)
+    if not all(map(len_is_same_as_parent, lists)):
+        raise LengthError("All children lists should be same length as the parent.")
+
 def nearest(lists, i, j):
     """Computes some form of distance.
 This is 'copied' over from genenetwork1, from https://github.com/genenetwork/genenetwork1/blob/master/web/webqtl/heatmap/slink.py#L42-L64.
@@ -28,5 +34,6 @@ This description should be updated once the form/type of 'distance' identified."
     #### Guard Functions: Should we do this a different way? ####
     raise_valueerror_if_data_is_not_lists_or_tuples(lists)
     raise_valueerror_if_lists_empty(lists)
+    raise_lengtherror_if_child_lists_are_not_same_as_parent(lists)
     #### END: Guard Functions ####
     return None
-- 
cgit v1.2.3