[CI] Add pre-commit hook vulture to find dead Python code

.pre-commit-config.yaml

@@ -10,6 +10,11 @@ repos:
     hooks:
       - id: identity
       - id: check-hooks-apply
+  - repo: https://github.com/jendrikseipp/vulture
+    rev: v2.13
+    hooks:
+      - id: vulture
+        args: [--min-confidence=100]
   - repo: https://github.com/psf/black-pre-commit-mirror
     rev: 24.10.0
     hooks:

pyproject.toml

@@ -0,0 +1,4 @@
+[tool.vulture]
+min_confidence = 100
+paths = ["python/", "spark-version-converter.py", "docs/usecases/utilities.py"]
+sort_by_size = true

python/sedona/maps/SedonaPyDeck.py

@@ -318,7 +318,7 @@ def _create_default_fill_color_(cls, gdf, plot_col):
         return "[85, 183, 177, ({0} / {1}) * 255 + 15]".format(plot_col, plot_max)
 
     @classmethod
-    def _create_coord_column_(cls, gdf, geometry_col, add_points=False):
+    def _create_coord_column_(cls, gdf, geometry_col):
         """
         Create a coordinate column in a given GeoPandas Dataframe, this coordinate column contains coordinates of a ST_Point in a list format of [longitude, latitude]
         :param gdf: GeoPandas Dataframe

python/sedona/raster/sedona_raster.py

@@ -191,7 +191,7 @@ def close(self):
     def __enter__(self):
         return self
 
-    def __exit__(self, exc_type, exc_val, exc_tb):
+    def __exit__(self):
         self.close()
 
     def __del__(self):

python/sedona/utils/abstract_parser.py

@@ -29,7 +29,7 @@ def name(self):
         raise NotImplementedError
 
     @classmethod
-    def serialize(cls, obj: BaseGeometry, binary_buffer: "BinaryBuffer"):
+    def serialize(cls, obj: BaseGeometry):
         raise NotImplementedError("Parser has to implement serialize method")
 
     @classmethod

python/sedona/utils/spatial_rdd_parser.py

@@ -101,7 +101,7 @@ def __ne__(self, other):
 class AbstractSpatialRDDParser(ABC):
 
     @classmethod
-    def serialize(cls, obj: List[Any], binary_buffer: "BinaryBuffer") -> bytearray:
+    def serialize(cls, obj: List[Any]) -> bytearray:
         raise NotImplemented()
 
     @classmethod
@@ -131,7 +131,7 @@ def deserialize(cls, bin_parser: "BinaryParser"):
         return deserialized_data
 
     @classmethod
-    def serialize(cls, obj: BaseGeometry, binary_buffer: "BinaryBuffer"):
+    def serialize(cls, obj: BaseGeometry):
         raise NotImplementedError("Currently this operation is not supported")
 
 
@@ -147,7 +147,7 @@ def deserialize(cls, bin_parser: "BinaryParser"):
         return left_geom_data
 
     @classmethod
-    def serialize(cls, obj: BaseGeometry, binary_buffer: "BinaryBuffer"):
+    def serialize(cls, obj: BaseGeometry):
         raise NotImplementedError("Currently this operation is not supported")
 
 
@@ -174,7 +174,7 @@ def deserialize(cls, bin_parser: "BinaryParser"):
         return deserialized_data
 
     @classmethod
-    def serialize(cls, obj: BaseGeometry, binary_buffer: "BinaryBuffer"):
+    def serialize(cls, obj: BaseGeometry):
         raise NotImplementedError("Currently this operation is not supported")
 
 

python/tests/spatial_rdd/test_spatial_rdd_writer.py

@@ -74,7 +74,7 @@ def remove_wkb_directory():
 
 class TestSpatialRDDWriter(TestBase):
 
-    def test_save_as_geo_json_with_data(self, remove_wkb_directory):
+    def test_save_as_geo_json_with_data(self):
         spatial_rdd = PointRDD(
             sparkContext=self.sc,
             InputLocation=inputLocation,