python: Fix typos and formatting of docstring parameters (OSGeo#4907)

* python: Fix typos and formatting of docstring parameters

* Fix typo

Author: echoix

python/grass/imaging/images2gif.py

@@ -229,9 +229,9 @@ def getGraphicsControlExt(self, duration=0.1, dispose=2):
 
           * 0 - No disposal specified.
           * 1 - Do not dispose. The graphic is to be left in place.
-          * 2 -	Restore to background color. The area used by the graphic
+          * 2 - Restore to background color. The area used by the graphic
             must be restored to the background color.
-          * 3 -	Restore to previous. The decoder is required to restore the
+          * 3 - Restore to previous. The decoder is required to restore the
             area overwritten by the graphic with what was there prior to
             rendering the graphic.
           * 4-7 -To be defined.
@@ -253,7 +253,6 @@ def handleSubRectangles(self, images, subRectangles):
         """Handle the sub-rectangle stuff. If the rectangles are given by the
         user, the values are checked. Otherwise the subrectangles are
         calculated automatically.
-
         """
 
         if isinstance(subRectangles, (tuple, list)):
@@ -400,9 +399,7 @@ def writeGifToFile(self, fp, images, durations, loops, xys, disposes):
         Requires different handling of palette for PIL and Pillow:
         based on https://github.com/rec/echomesh/blob/master/
         code/python/external/images2gif.py
-
         """
-
         # Obtain palette for all images and count each occurrence
         palettes, occur = ([], [])
         for im in images:
@@ -538,7 +535,7 @@ def writeGifVisvis(
                         integer types, and between 0 and 1 for float types.
     :param duration: scalar or list of scalars The duration for all frames, or
                      (if a list) for each frame.
-    :param repeat: bool or integer The amount of loops. If True, loops infinitetely.
+    :param repeat: bool or integer The amount of loops. If True, loops infinitely.
     :param bool dither: whether to apply dithering
     :param int nq: If nonzero, applies the NeuQuant quantization algorithm to
                    create the color palette. This algorithm is superior, but

python/grass/jupyter/utils.py

@@ -309,14 +309,13 @@ def estimate_resolution(
 ) -> float:
     """Estimates resolution of reprojected raster.
 
-    :param str raster: name of raster
-    :param str mapset: mapset of raster
-    :param str location: name of source location
-    :param str dbase: path to source database
+    :param raster: name of raster
+    :param mapset: mapset of raster
+    :param location: name of source location
+    :param dbase: path to source database
     :param dict env: target environment
 
-    :return float estimate: estimated resolution of raster in destination
-                            environment
+    :return estimate: estimated resolution of raster in destination environment
     """
     output = gs.read_command(
         "r.proj",

python/grass/pygrass/modules/grid/grid.py

@@ -163,8 +163,7 @@ def copy_groups(groups, gisrc_src, gisrc_dst, region=None):
     :param gisrc_dst: path of the GISRC file where the groups will be created
     :type gisrc_dst: str
     :param region: a region like object or a dictionary with the region
-                   parameters that will be used to crop the rasters of the
-                   groups
+                   parameters that will be used to crop the rasters of the groups
     :type region: Region object or dictionary
     :returns: None
 
@@ -204,8 +203,7 @@ def set_region(region, gisrc_src, gisrc_dst, env):
     """Set a region into two different mapsets.
 
     :param region: a region like object or a dictionary with the region
-                   parameters that will be used to crop the rasters of the
-                   groups
+                   parameters that will be used to crop the rasters of the groups
     :type region: Region object or dictionary
     :param gisrc_src: path of the GISRC file from where we want to copy the groups
     :type gisrc_src: str
@@ -238,8 +236,7 @@ def copy_rasters(rasters, gisrc_src, gisrc_dst, region=None):
     :param gisrc_dst: path of the GISRC file where the groups will be created
     :type gisrc_dst: str
     :param region: a region like object or a dictionary with the region
-                   parameters that will be used to crop the rasters of the
-                   groups
+                   parameters that will be used to crop the rasters of the groups
     :type region: Region object or dictionary
     :returns: None
     """
@@ -401,7 +398,7 @@ class GridModule:
     :param overlap: overlap between tiles, in pixel.
     :type overlap: int
     :param processes: number of threads, default value is equal to the number
-                      of processor available.
+                      of processors available.
     :param split: if True use r.tile to split all the inputs.
     :type split: bool
     :param mapset_prefix: if specified created mapsets start with this prefix

python/grass/pygrass/vector/__init__.py

@@ -127,7 +127,7 @@ def write(self, geo_obj, cat=None, attrs=None):
         :param geo_obj: a geometry grass object define in
                         grass.pygrass.vector.geometry
         :type geo_obj: geometry GRASS object
-        :param attrs: a list with the values that will be insert in the
+        :param attrs: a list with the values that will be inserted in the
                       attribute table.
         :type attrs: list
         :param cat: The category of the geometry feature, otherwise the

python/grass/pygrass/vector/abstract.py

@@ -337,12 +337,11 @@ def open(
         :param tab_name: define the name of the table that will be generate
         :type tab_name: str
         :param tab_cols: define the name and type of the columns of the
-                         attribute table of the vecto map
+                         attribute table of the vector map
         :type tab_cols: list of pairs
-        :param link_name: define the name of the link connection with the
-                          database
+        :param link_name: define the name of the link connection with the database
         :type link_name: str
-        :param link_key: define the nema of the column that will be use as
+        :param link_key: define the name of the column that will be use as
                          vector category
         :type link_key: str
         :param link_db: define the database connection parameters

python/grass/pygrass/vector/geometry.py

@@ -591,8 +591,7 @@ def buffer(
         :type angle: num
         :param round_: to make corners round
         :type round_: bool
-        :param tol: fix the maximum distance between theoretical arc and
-                    output segments
+        :param tol: fix the maximum distance between theoretical arc and output segments
         :type tol: float
         :returns: the buffer as Area object
 
@@ -1048,11 +1047,9 @@ def reverse(self):
     def segment(self, start, end):
         """Create line segment. using the ``Vect_line_segment`` C function.
 
-        :param start: distance from the beginning of the line where
-                      the segment start
+        :param start: distance from the beginning of the line where the segment starts
         :type start: float
-        :param end: distance from the beginning of the line where
-                    the segment end
+        :param end: distance from the beginning of the line where the segment ends
         :type end: float
 
         ::
@@ -1155,8 +1152,7 @@ def buffer(
         :type angle: num
         :param round_: to make corners round
         :type round_: bool
-        :param tol: fix the maximum distance between theoretical arc and
-                    output segments
+        :param tol: fix the maximum distance between theoretical arc and output segments
         :type tol: float
         :returns: the buffer as Area object
 
@@ -1717,8 +1713,7 @@ def buffer(
         :type angle: num
         :param round_: to make corners round
         :type round_: bool
-        :param tol: fix the maximum distance between theoretical arc and
-                    output segments
+        :param tol: fix the maximum distance between theoretical arc and output segments
         :type tol: float
         :returns: the buffer as line, centroid, isles object tuple
 

python/grass/pygrass/vector/table.py

@@ -1121,8 +1121,7 @@ def drop(self, cursor=None, force=False):
         :param cursor: the cursor to connect, if None it use the cursor
                        of connection table object
         :type cursor: Cursor object
-        :param force: True to remove the table, by default False to print
-                      advice
+        :param force: True to remove the table, by default False to print advice
         :type force: bool
         """
 
@@ -1161,8 +1160,7 @@ def execute(self, sql_code=None, cursor=None, many=False, values=None):
         """Execute SQL code from a given string or build with filters and
         return a cursor object.
 
-        :param sql_code: the SQL code to execute, if not pass it use filters
-                         variable
+        :param sql_code: the SQL code to execute, if not pass it use filters variable
         :type sql_code: str
         :param cursor: the cursor to connect, if None it use the cursor
                      of connection table object

python/grass/script/raster.py

@@ -218,8 +218,7 @@ def raster_what(map, coord, env=None, localized=False):
     [{'elevation': {'color': '255:214:000', 'label': '', 'value': '102.479'}}]
 
     :param str map: the map name
-    :param list coord: a list of list containing all the point that you want
-                       query
+    :param list coord: a list of list containing all the point that you want to query
     :param env:
     """
     map_list = [map] if isinstance(map, (bytes, str)) else map

python/grass/temporal/abstract_dataset.py

@@ -74,7 +74,7 @@ def get_number_of_relations(self):
         topologies must be build first using the SpatioTemporalTopologyBuilder.
 
         :return: The dictionary with relations as keys and number as values or
-                 None in case the topology  wasn't build
+                 None in case the topology wasn't built
         """
         if self.is_temporal_topology_build() and not self.is_spatial_topology_build():
             return self.get_number_of_temporal_relations()
@@ -341,8 +341,7 @@ def get_spatial_extent_as_tuple(self):
         Top and bottom are set to 0 in case of a two dimensional spatial
         extent.
 
-        :return: A the spatial extent as tuple (north, south, east, west,
-                 top, bottom)
+        :return: A the spatial extent as tuple (north, south, east, west, top, bottom)
         """
         return self.spatial_extent.get_spatial_extent_as_tuple()
 

python/grass/temporal/abstract_map_dataset.py

@@ -176,8 +176,7 @@ def split_name(name: str, layer=None, mapset=None):
         the keyword arguments.
 
         :param name: The name of the map
-        :param layer: The layer of the vector map, use None in case no
-                      layer exists
+        :param layer: The layer of the vector map, use None in case no layer exists
         :param mapset: The mapset in which the map is located
 
         :return: tuple of three elements name, layer, mapset e(:layer)@mapset" while
@@ -244,11 +243,9 @@ def build_id(name: str, mapset, layer=None) -> str:
 
         :param name: The name of the map
         :param mapset: The mapset in which the map is located
-        :param layer: The layer of the vector map, use None in case no
-                      layer exists
+        :param layer: The layer of the vector map, use None in case no layer exists
 
-        :return: the id of the map as "name(:layer)@mapset" where layer is
-                 optional
+        :return: the id of the map as "name(:layer)@mapset" where layer is optional
         """
 
         # Split given name into relevant parts
@@ -421,8 +418,7 @@ def set_absolute_time(self, start_time, end_time=None) -> bool:
          This method only modifies this object and does not commit
          the modifications to the temporal database.
 
-        :param start_time: A datetime object specifying the start time of
-                           the map
+        :param start_time: A datetime object specifying the start time of the map
         :param end_time: A datetime object specifying the end time of the
                          map, None in case or time instance
 
@@ -512,8 +508,7 @@ def update_absolute_time(self, start_time, end_time=None, dbif=None) -> None:
         grass file system based database in addition to the temporal
         database entry.
 
-        :param start_time: A datetime object specifying the start time of
-                                      the map
+        :param start_time: A datetime object specifying the start time of the map
         :param end_time: A datetime object specifying the end time of the
                                      map, None in case or time instance
         :param dbif: The database interface to be used
@@ -669,8 +664,7 @@ def set_temporal_extent(self, extent) -> None:
         """Convenient method to set the temporal extent from a temporal extent
         object
 
-        :param extent: The temporal extent that should be set for
-                                this object
+        :param extent: The temporal extent that should be set for this object
 
         .. code-block: : python
 
@@ -853,8 +847,7 @@ def set_spatial_extent(self, spatial_extent) -> None:
          This method only modifies this object and does not commit
          the modifications to the temporal database.
 
-         :param spatial_extent: An object of type SpatialExtent or its
-                                subclasses
+         :param spatial_extent: An object of type SpatialExtent or its subclasses
 
         .. code-block: : python