clear some com

Author: MaxNumerique

src/opengeodeweb_viewer/object/object_methods.py

@@ -162,12 +162,10 @@ def SetBlocksVisibility(
         mapper = pipeline.mapper
         if not isinstance(mapper, vtkCompositePolyDataMapper):
             raise Exception("Mapper is not a vtkCompositePolyDataMapper")
-        # Update block visibility attributes on the main mapper
         blocks = pipeline.blockDataSets
         visibility_attributes = mapper.GetCompositeDataDisplayAttributes()
         for block_id in block_ids:
             visibility_attributes.SetBlockVisibility(blocks[block_id], visibility)
-        # Extract output dataset from filter or reader
         dataset = (pipeline.filter or pipeline.reader).GetOutputDataObject(0)
         if not isinstance(dataset, vtkMultiBlockDataSet):
             return

src/opengeodeweb_viewer/rpc/viewer/viewer_protocols.py

@@ -343,26 +343,26 @@ def setHighlight(
         )
         params = schemas.Highlight.from_dict(rpc_params)
 
-        # 1. Clear previous highlights
+        # Clear previous highlights
         self.clear_highlights(params.ids)
         picker = vtkCellPicker(tolerance=0.005)
 
-        # 2. Perform pick operation to identify clicked pipeline and primitive ID
+        # Perform pick operation to identify clicked pipeline and primitive ID
         data_id, id_to_select = self.pick_cell_or_point(
             params.ids, params.x, params.y, params.field_type.value, picker
         )
         if not data_id or id_to_select == -1:
             self.render(-1)
             return {}
 
-        # 3. Retrieve picked composite block information and check block visibility
+        # Retrieve picked composite block information and check block visibility
         pipeline = self.get_vtk_pipeline(data_id)
         dataset, geode_id, is_visible = self.get_composite_block_info(pipeline, picker)
         if not is_visible:
             self.render(-1)
             return {}
 
-        # 4. Update highlight visibility and extract attributes from the picked element
+        # Update highlight visibility and extract attributes from the picked element
         self.update_highlight(pipeline, id_to_select, params.field_type.value, dataset)
         self.render(-1)
         data_attributes = self.extract_picked_attributes(

src/opengeodeweb_viewer/vtk_protocol.py

@@ -213,12 +213,10 @@ def pick_cell_or_point(
         field_type: str,
         picker: vtkCellPicker,
     ) -> tuple[str | None, int]:
-        # Temporarily swap to pick mappers to prevent picking hidden blocks
         self.swap_pick_mappers(data_ids, use_pick_mapper=True)
         try:
             picker.Pick(x, y, 0, self.get_renderer())
         finally:
-            # Swap back to normal mappers
             self.swap_pick_mappers(data_ids, use_pick_mapper=False)
 
         actor = picker.GetActor()
@@ -239,21 +237,18 @@ def pick_cell_or_point(
     def pick_actors_under_coordinate(
         self, data_ids: list[str], x: float, y: float, picker: vtkCellPicker
     ) -> tuple[list[vtkActor], int]:
-        # Swap mappers to ensure picking respects block visibility
         renderer = self.get_renderer()
         self.swap_pick_mappers(data_ids, use_pick_mapper=True)
         actors = []
         viewer_id = -1
         try:
             picker.Pick(x, y, 0, renderer)
             viewer_id = picker.GetFlatBlockIndex()
-            # Iteratively pick overlapping actors by temporarily hiding them
             while actor := picker.GetActor():
                 actors.append(actor)
                 actor.SetPickable(False)
                 picker.Pick(x, y, 0, renderer)
         finally:
-            # Revert pickability and swap back to normal mappers
             for actor in actors:
                 actor.SetPickable(True)
             self.swap_pick_mappers(data_ids, use_pick_mapper=False)