Fix Pylint errors

Author: f-PLT

geospatial_tools/planetary_computer/sentinel_2.py

@@ -72,7 +72,7 @@ def __init__(
 
     @property
     def max_cloud_cover(self):
-        """"""
+        """Max % of cloud cover used for Sentinel 2 product search."""
         return self._max_cloud_cover
 
     @max_cloud_cover.setter
@@ -90,7 +90,7 @@ def max_cloud_cover(self, max_cloud_cover: int):
 
     @property
     def date_ranges(self):
-        """"""
+        """Date range used to search for Sentinel 2 products."""
         return self._date_ranges
 
     @date_ranges.setter
@@ -181,7 +181,7 @@ def find_best_complete_products(self, max_cloud_cover: int | None = None, max_no
         return self.successful_results
 
     def select_best_products_per_feature(self) -> GeoDataFrame:
-        """"""
+        """Return a GeoDataFrame containing the best products for each Sentinel 2 tile."""
         spatial_join_results = spatial_join_within(
             polygon_features=self.sentinel2_tiling_grid,
             polygon_column=self.sentinel2_tiling_grid_column,

geospatial_tools/stac.py

@@ -91,7 +91,11 @@ def list_available_catalogs(logger: logging.Logger = LOGGER) -> frozenset[str]:
 
 
 class AssetSubItem:
-    """"""
+    """
+    Class that represent a STAC asset sub item.
+
+    Generally represents a single satellite image band.
+    """
 
     def __init__(self, asset, item_id: str, band: str, filename: str | Path):
         """
@@ -111,7 +115,7 @@ def __init__(self, asset, item_id: str, band: str, filename: str | Path):
 
 
 class Asset:
-    """"""
+    """Represents a STAC asset."""
 
     def __init__(
         self,
@@ -240,32 +244,30 @@ def reproject_merged_asset(
         return None
 
     def delete_asset_sub_items(self):
-        """"""
+        """Delete all asset sub items that belong to this asset."""
         self.logger.info(f"Deleting asset sub items from asset [{self.asset_id}]")
         if self.list:
             for item in self.list:
                 self.logger.info(f"Deleting [{item.filename}] ...")
                 item.filename.unlink()
 
     def delete_merged_asset(self):
-        """"""
+        """Delete merged asset."""
         self.logger.info(f"Deleting merged asset file for [{self.merged_asset_path}]")
         self.merged_asset_path.unlink()
 
     def delete_reprojected_asset(self):
-        """"""
+        """Delete reprojected asset."""
         self.logger.info(f"Deleting reprojected asset file for [{self.reprojected_asset_path}]")
         self.reprojected_asset_path.unlink()
 
     def _create_merged_asset_metadata(self):
-        """"""
         self.logger.info("Creating merged asset metadata")
         file_list = [asset.filename for asset in self.list]
         meta = create_merged_raster_bands_metadata(file_list)
         return meta
 
     def _get_asset_total_bands(self):
-        """"""
         downloaded_file_list = [asset.filename for asset in self.list]
         total_band_count = get_total_band_count(downloaded_file_list)
         return total_band_count

geospatial_tools/vector.py

@@ -412,8 +412,8 @@ def select_all_within_feature(polygon_feature: gpd.GeoSeries, vector_features: g
     This function is quite small and simple, but exists mostly as a.
 
     Args:
-      polygon_feature: Polygon feature that will be used to find which features of `vector_features` are contained within it.
-        In this function, it is expected to be a GeoSeries, so a single row from a GeoDataFrame.
+      polygon_feature: Polygon feature that will be used to find which features of `vector_features` are contained
+        within it. In this function, it is expected to be a GeoSeries, so a single row from a GeoDataFrame.
       vector_features: The dataframe containing the features that will be grouped by polygon_feature.
       polygon_feature:
       vector_features: