Merge pull request #8 from ParallelDots/update-internal-logic

Fix: Update relative radius calculation logic

Author: ahwankumar

.github/workflows/workflow.yml

@@ -1,6 +1,9 @@
 name: Publish RetailTree Python package distribution to PyPI
 
 on:
+  push:
+    branches:
+      - main
   workflow_run:
     workflows: ["Run Tests"]
     types:

retailtree/retailtree.py

@@ -57,17 +57,13 @@ def build_tree(self, dist_func=euclidean):
     def __get_neighbors_radius(self):
         """
         Method to get the radius within which neighbors will be searched for.
-        If number of annotations is greater than 10, random annotations are considered.
-
         Radius is the max of the diagonals of the annotations considered.
         """
         radius = self.__neighbors_radius
         if not radius:
             annotation_bucket = list(self.annotations.values())
-            if len(annotation_bucket) > 10:
-                random_annotations = sample(annotation_bucket, 10)
-            else:
-                random_annotations = annotation_bucket
+
+            random_annotations = annotation_bucket
             distance = [math.sqrt(pow(annotation.width, 2) + pow(
                 annotation.length, 2)) for annotation in random_annotations]
             radius = max(distance)

setup.py

@@ -9,7 +9,7 @@ def read(fname):
 
 setup(
     name="retailtree",
-    version="1.2",
+    version="1.3",
     long_description=read("README.md"),
     long_description_content_type='text/markdown',
     packages=find_packages(),