better fix without changing existing interfaces

hubmapconsortium · Dec 13, 2024 · 63f4ff2 · 63f4ff2
1 parent 524cbda
commit 63f4ff2
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 3 deletions.
diff --git a/src/portal_visualization/builder_factory.py b/src/portal_visualization/builder_factory.py
@@ -76,7 +76,7 @@ def get_view_config_builder(entity, get_entity, parent=None, epic_uuid=None):
             return SegImagePyramidViewConfBuilder
 
         elif is_support and is_image:
-            ancestor_assaytype = get_entity(uuid=parent.get('uuid')).get("soft_assaytype")
+            ancestor_assaytype = get_entity(parent).get("soft_assaytype")
             if SEQFISH == ancestor_assaytype:
                 # e.g. parent  = c6a254b2dc2ed46b002500ade163a7cc
                 # e.g. support = 9db61adfc017670a196ea9b3ca1852a0

diff --git a/src/portal_visualization/client.py b/src/portal_visualization/client.py
@@ -246,8 +246,9 @@ def get_vitessce_conf_cells_and_lifted_uuid(
         # Otherwise, just try to visualize the data for the entity itself:
         else:  # pragma: no cover  # We have separate tests for the builder logic
             try:
-                print("YES PARENTHESES")
-                Builder = get_view_config_builder(entity, self.get_entity, parent, epic_uuid)
+                def get_entity(uuid):
+                    return self.get_entity(uuid=uuid)
+                Builder = get_view_config_builder(entity, get_entity, parent, epic_uuid)
                 builder = Builder(entity, self.groups_token, self.assets_endpoint)
                 vitessce_conf = builder.get_conf_cells(marker=marker)
             except Exception as e: