clean up

polytope/datacube/backends/fdb.py

@@ -102,8 +102,6 @@ def get(self, requests: TensorIndexTree):
             request_combis = product(*interm_branch_tuple_values)
 
             # Need to extract the possible requests and add them to the right nodes
-            # complete_list_complete_uncompressed_requests = []
-            # complete_fdb_decoding_info = []
             for combi in request_combis:
                 uncompressed_request = {}
                 for i, key in enumerate(compressed_request[0].keys()):
@@ -286,7 +284,6 @@ def assign_fdb_output_to_nodes(self, output_values, fdb_requests_decoding_info):
                     # If we are here, no data was found for this path in the fdb
                     none_array = [None] * len(n.values)
                     n.result.extend(none_array)
-                    # n.remove_branch()
                 else:
                     interm_request_output_values = request_output_values[0][i][0]
                     n.result.extend(interm_request_output_values)

polytope/engine/hullslicer.py

@@ -194,8 +194,6 @@ def find_compressed_axes(self, datacube, polytopes):
         # add the last axis of the grid always (longitude) as a compressed axis
         k, last_value = _, datacube.axes[k] = datacube.axes.popitem()
         self.compressed_axes.append(k)
-        # if len(datacube.coupled_axes) != 0:
-        #     self.compressed_axes.append(datacube.coupled_axes[0][-1])
 
     def remove_compressed_axis_in_union(self, polytopes):
         for p in polytopes:

tests/test_fdb_unmap_tree.py

@@ -5,9 +5,6 @@
 from polytope.polytope import Polytope, Request
 from polytope.shapes import Box, Select
 
-# import geopandas as gpd
-# import matplotlib.pyplot as plt
-
 
 class TestSlicingFDBDatacube:
     def setup_method(self, method):

tests/test_union_gj.py

@@ -164,6 +164,3 @@ def test_fdb_datacube_complete_overlap_v2(self):
             total_lons += len(leaf.values)
             total_vals += len(leaf.result)
         assert total_lons == 9
-        # TODO: there is a problem somewhere when we assign the values to the nodes
-        # OR the problem is when we remove duplicates before giving to gribjump? Check
-        assert total_vals == 9