From 78ef2ec19678e65a24fde5e2f1e09b1893047e64 Mon Sep 17 00:00:00 2001 From: RhythmP619 Date: Wed, 26 Nov 2025 22:32:20 +0530 Subject: [PATCH 1/7] fix: replace mutable default arguments to fix Pylint W0102 warnings --- pyproject.toml | 3 +- python/grass/pygrass/messages/__init__.py | 11 +++-- python/grass/temporal/temporal_algebra.py | 41 +++++++++++++------ .../temporal/temporal_raster_base_algebra.py | 31 ++++++++++---- .../grass/temporal/temporal_vector_algebra.py | 12 ++++-- 5 files changed, 67 insertions(+), 31 deletions(-) diff --git a/pyproject.toml b/pyproject.toml index 0dbc7549763..3ea93de36ea 100644 --- a/pyproject.toml +++ b/pyproject.toml @@ -721,7 +721,6 @@ disable = [ "RP0401", # Report: Imports checker: External dependencies "RP0801", # Report: Similarities: Duplication "W0101", # Unreachable code (unreachable) - "W0102", # Dangerous default value %s as argument (dangerous-default-value) "W0104", # Statement seems to have no effect (pointless-statement) "W0106", # Expression "%s" is assigned to nothing (expression-not-assigned) "W0108", # Lambda may not be necessary (unnecessary-lambda) @@ -745,7 +744,7 @@ disable = [ "W0611", # (unused-import) "W0612", # Unused variable %r (unused-variable) "W0613", # Unused argument %r (unused-argument) - "W0614", # Unused import(s) %s from wildcard import of %s (unused-wildcard-import) + #"W0614", # Unused import(s) %s from wildcard import of %s (unused-wildcard-import) "W0621", # Redefining name %r from outer scope (line %s) (redefined-outer-name) "W0621", # Redefining name %r from outer scope (line %s) (redefined-outer-name) "W0622", # Redefining built-in %r (redefined-builtin) diff --git a/python/grass/pygrass/messages/__init__.py b/python/grass/pygrass/messages/__init__.py index 5735398dafc..ad7cf1a2b46 100644 --- a/python/grass/pygrass/messages/__init__.py +++ b/python/grass/pygrass/messages/__init__.py @@ -338,11 +338,9 @@ def test_fatal_error(self, message: str) -> None: self.client_conn.send(["FATAL", message]) time.sleep(1) +_MSGR_INSTANCE: Messenger | None = None def get_msgr( - instance=[ - None, - ], *args, **kwargs, ) -> Messenger: @@ -358,9 +356,10 @@ def get_msgr( >>> msgr0 is msgr2 False """ - if not instance[0]: - instance[0] = Messenger(*args, **kwargs) - return instance[0] + global _MSGR_INSTANCE + if _MSGR_INSTANCE is None: + _MSGR_INSTANCE = Messenger(*args, **kwargs) + return _MSGR_INSTANCE if __name__ == "__main__": diff --git a/python/grass/temporal/temporal_algebra.py b/python/grass/temporal/temporal_algebra.py index 37da1099bc9..7119b2e7999 100644 --- a/python/grass/temporal/temporal_algebra.py +++ b/python/grass/temporal/temporal_algebra.py @@ -1112,7 +1112,7 @@ def overlay_map_extent( returncode = 0 return returncode - def set_temporal_extent_list(self, maplist, topolist=["EQUAL"], temporal="l"): + def set_temporal_extent_list(self, maplist, topolist=None, temporal="l"): """Change temporal extent of map list based on temporal relations to other map list and given temporal operator. @@ -1125,6 +1125,8 @@ def set_temporal_extent_list(self, maplist, topolist=["EQUAL"], temporal="l"): :return: Map list with specified temporal extent. """ + if topolist is None: + topolist = ["EQUAL"] resultdict = {} temporal_topo_list, spatial_topo_list = self._check_topology(topolist=topolist) @@ -1399,7 +1401,7 @@ def build_spatio_temporal_topology_list( self, maplistA, maplistB=None, - topolist=["EQUAL"], + topolist=None, assign_val: bool = False, count_map: bool = False, compare_bool: bool = False, @@ -1573,6 +1575,8 @@ def build_spatio_temporal_topology_list( """ # Check the topology definitions and return the list of temporal and spatial # topological relations that must be fulfilled + if topolist is None: + topolist = ["EQUAL"] temporal_topo_list, spatial_topo_list = self._check_topology(topolist=topolist) resultdict = {} @@ -1624,7 +1628,7 @@ def build_spatio_temporal_topology_list( return sorted(resultlist, key=AbstractDatasetComparisonKeyStartTime) def assign_bool_value( - self, map_i, temporal_topo_list=["EQUAL"], spatial_topo_list=[] + self, map_i, temporal_topo_list=None, spatial_topo_list=None ) -> bool: """Function to assign boolean map value based on the map_values from the compared map list by topological relationships. @@ -1637,7 +1641,10 @@ def assign_bool_value( relation maps that fulfil the topological relationships to maplistB specified in temporal_topo_list. """ - + if temporal_topo_list is None: + temporal_topo_list = ["EQUAL"] + if spatial_topo_list is None: + spatial_topo_list = [] temporal_relations = map_i.get_temporal_relations() condition_value_list = [] for topo in temporal_topo_list: @@ -1669,8 +1676,8 @@ def compare_bool_value( map_i, compop, aggregate, - temporal_topo_list=["EQUAL"], - spatial_topo_list=[], + temporal_topo_list=None, + spatial_topo_list=None, ): """Function to evaluate two map lists with boolean values by boolean comparison operator. @@ -1684,7 +1691,10 @@ def compare_bool_value( :return: Map object with conditional value that has been evaluated by comparison operators. """ - + if temporal_topo_list is None: + temporal_topo_list = ["EQUAL"] + if spatial_topo_list is None: + spatial_topo_list = [] temporal_relations = map_i.get_temporal_relations() # Build conditional list with elements from related maps and given relation @@ -1760,7 +1770,7 @@ def perform_temporal_selection( self, maplistA, maplistB, - topolist=["EQUAL"], + topolist=None, inverse: bool = False, assign_val: bool = False, ): @@ -1828,6 +1838,8 @@ def perform_temporal_selection( Map a4 has no equal relation to mapset mapsB """ + if topolist is None: + topolist = ["EQUAL"] if not inverse: topolist = self.build_spatio_temporal_topology_list( maplistA, maplistB, topolist, assign_val=assign_val @@ -1849,7 +1861,7 @@ def perform_temporal_selection( # Sort list of maps chronological. return sorted(resultlist, key=AbstractDatasetComparisonKeyStartTime) - def set_granularity(self, maplistA, maplistB, toperator="l", topolist=["EQUAL"]): + def set_granularity(self, maplistA, maplistB, toperator="l", topolist=None): """This function sets the temporal extends of a list of maps based on another map list. @@ -1900,6 +1912,8 @@ def set_granularity(self, maplistA, maplistB, toperator="l", topolist=["EQUAL"]) :raises SyntaxError: If an unpermitted temporal relation name is used in ``topolist`` """ + if topolist is None: + topolist = ["EQUAL"] topologylist = [ "EQUAL", "FOLLOWS", @@ -2153,7 +2167,7 @@ def eval_global_var(self, gvar, maplist): map_i.condition_value = boolname return maplist - def eval_map_list(self, maplist, thenlist, topolist=["EQUAL"]): + def eval_map_list(self, maplist, thenlist, topolist=None): """This function transfers boolean values from temporal expression from one map list to another by their topology. These boolean values are added to the maps as condition_value. @@ -2172,11 +2186,13 @@ def eval_map_list(self, maplist, thenlist, topolist=["EQUAL"]): # inverse = True, # topolist = topolist) # Combining the selection and inverse selection list. + if topolist is None: + topolist = ["EQUAL"] return self.perform_temporal_selection( thenlist, maplist, assign_val=True, topolist=topolist ) - def build_condition_list(self, tvarexpr, thenlist, topolist=["EQUAL"]): + def build_condition_list(self, tvarexpr, thenlist, topolist=None): """This function evaluates temporal variable expressions of a conditional expression in two steps. At first it combines stepwise the single conditions by their relations with @@ -2207,7 +2223,8 @@ def build_condition_list(self, tvarexpr, thenlist, topolist=["EQUAL"]): :return: Map list with conditional values for all temporal expressions. """ - + if topolist is None: + topolist = ["EQUAL"] # Evaluate the temporal variable expression and compute the temporal combination # of conditions. diff --git a/python/grass/temporal/temporal_raster_base_algebra.py b/python/grass/temporal/temporal_raster_base_algebra.py index 308edc948c6..f3349f59cf4 100644 --- a/python/grass/temporal/temporal_raster_base_algebra.py +++ b/python/grass/temporal/temporal_raster_base_algebra.py @@ -204,7 +204,7 @@ def build_spatio_temporal_topology_list( self, maplistA, maplistB=None, - topolist=["EQUAL"], + topolist=None, assign_val: bool = False, count_map: bool = False, compare_bool: bool = False, @@ -275,6 +275,8 @@ def build_spatio_temporal_topology_list( a9@B """ + if topolist is None: + topolist = ["EQUAL"] if self.debug: print( topolist, @@ -418,8 +420,8 @@ def compare_cmd_value( map_i, compop, aggregate, - temporal_topo_list=["EQUAL"], - spatial_topo_list=[], + temporal_topo_list=None, + spatial_topo_list=None, convert: bool = False, ): """Function to evaluate two map lists with boolean values by boolean @@ -447,6 +449,10 @@ def compare_cmd_value( """ # Build command list list with elements from related maps and given relation # operator. + if temporal_topo_list is None: + temporal_topo_list = ["EQUAL"] + if spatial_topo_list is None: + spatial_topo_list = [] if convert and "condition_value" in dir(map_i): if map_i.condition_value != []: cmdstring = str(int(map_i.condition_value[0])) @@ -500,7 +506,7 @@ def compare_cmd_value( return cmd_value_str def operator_cmd_value( - self, map_i, operator, temporal_topo_list=["EQUAL"], spatial_topo_list=[] + self, map_i, operator, temporal_topo_list=None, spatial_topo_list=None ): """Function to evaluate two map lists by given arithmetic operator. @@ -512,7 +518,10 @@ def operator_cmd_value( :return: Map object with command list with operators that has been evaluated by implicit aggregation. """ - + if temporal_topo_list is None: + temporal_topo_list = ["EQUAL"] + if spatial_topo_list is None: + spatial_topo_list = [] temporal_relations = map_i.get_temporal_relations() # Build comandlist list with elements from related maps and given relation @@ -553,7 +562,7 @@ def operator_cmd_value( def set_temporal_extent_list( self, maplist, - topolist=["EQUAL"], + topolist=None, temporal="l", cmd_bool: bool = False, cmd_type=None, @@ -575,6 +584,8 @@ def set_temporal_extent_list( :return: Map list with specified temporal extent and optional command string. """ + if topolist is None: + topolist = ["EQUAL"] resultdict = {} temporal_topo_list, spatial_topo_list = self._check_topology(topolist=topolist) @@ -643,8 +654,8 @@ def build_condition_cmd_list( iflist, thenlist, elselist=None, - condition_topolist=["EQUAL"], - conclusion_topolist=["EQUAL"], + condition_topolist=None, + conclusion_topolist=None, temporal="l", null: bool = False, ): @@ -667,6 +678,10 @@ def build_condition_cmd_list( :return: map list with resulting command string for given condition type. """ + if condition_topolist is None: + condition_topolist = ["EQUAL"] + if conclusion_topolist is None: + conclusion_topolist = ["EQUAL"] resultlist = [] # First merge conclusion command maplists or strings. # Check if alternative conclusion map list is given. diff --git a/python/grass/temporal/temporal_vector_algebra.py b/python/grass/temporal/temporal_vector_algebra.py index 95ad181b817..376496e8cec 100644 --- a/python/grass/temporal/temporal_vector_algebra.py +++ b/python/grass/temporal/temporal_vector_algebra.py @@ -186,7 +186,7 @@ def build_spatio_temporal_topology_list( self, maplistA, maplistB=None, - topolist=["EQUAL"], + topolist=None, assign_val: bool = False, count_map: bool = False, compare_bool: bool = False, @@ -223,6 +223,8 @@ def build_spatio_temporal_topology_list( :return: List of maps from maplistA that fulfil the topological relationships to maplistB specified in topolist. """ + if topolist is None: + topolist = ["EQUAL"] topologylist = [ "EQUAL", "FOLLOWS", @@ -285,7 +287,7 @@ def build_spatio_temporal_topology_list( # Sort list of maps chronological. return sorted(resultlist, key=AbstractDatasetComparisonKeyStartTime) - def overlay_cmd_value(self, map_i, tbrelations, function, topolist=["EQUAL"]): + def overlay_cmd_value(self, map_i, tbrelations, function, topolist=None): """Function to evaluate two map lists by given overlay operator. :param map_i: Map object with temporal extent. @@ -298,6 +300,8 @@ def overlay_cmd_value(self, map_i, tbrelations, function, topolist=["EQUAL"]): """ # Build comandlist list with elements from related maps and given relation # operator. + if topolist is None: + topolist = ["EQUAL"] resultlist = [] # Define overlay operation dictionary. overlaydict = {"&": "and", "|": "or", "^": "xor", "~": "not", "+": "disor"} @@ -345,7 +349,7 @@ def overlay_cmd_value(self, map_i, tbrelations, function, topolist=["EQUAL"]): return resultlist - def set_temporal_extent_list(self, maplist, topolist=["EQUAL"], temporal="l"): + def set_temporal_extent_list(self, maplist, topolist=None, temporal="l"): """Change temporal extent of map list based on temporal relations to other map list and given temporal operator. @@ -358,6 +362,8 @@ def set_temporal_extent_list(self, maplist, topolist=["EQUAL"], temporal="l"): :return: Map list with specified temporal extent. """ + if topolist is None: + topolist = ["EQUAL"] resultdict = {} for map_i in maplist: From aa0f03bb954dcc24cf184cd555c5bccc7fc59edf Mon Sep 17 00:00:00 2001 From: RhythmP619 Date: Wed, 26 Nov 2025 23:18:00 +0530 Subject: [PATCH 2/7] fix: enabled W0614 --- pyproject.toml | 1 - 1 file changed, 1 deletion(-) diff --git a/pyproject.toml b/pyproject.toml index 3ea93de36ea..83561606a07 100644 --- a/pyproject.toml +++ b/pyproject.toml @@ -744,7 +744,6 @@ disable = [ "W0611", # (unused-import) "W0612", # Unused variable %r (unused-variable) "W0613", # Unused argument %r (unused-argument) - #"W0614", # Unused import(s) %s from wildcard import of %s (unused-wildcard-import) "W0621", # Redefining name %r from outer scope (line %s) (redefined-outer-name) "W0621", # Redefining name %r from outer scope (line %s) (redefined-outer-name) "W0622", # Redefining built-in %r (redefined-builtin) From 25dee52ca26ed180cf5efff6c62612c4f62f691c Mon Sep 17 00:00:00 2001 From: Rhythm Patel <138519018+RhythmP619@users.noreply.github.com> Date: Thu, 27 Nov 2025 00:00:14 +0530 Subject: [PATCH 3/7] Update python/grass/pygrass/messages/__init__.py Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com> --- python/grass/pygrass/messages/__init__.py | 1 + 1 file changed, 1 insertion(+) diff --git a/python/grass/pygrass/messages/__init__.py b/python/grass/pygrass/messages/__init__.py index ad7cf1a2b46..b8ed18126a8 100644 --- a/python/grass/pygrass/messages/__init__.py +++ b/python/grass/pygrass/messages/__init__.py @@ -338,6 +338,7 @@ def test_fatal_error(self, message: str) -> None: self.client_conn.send(["FATAL", message]) time.sleep(1) + _MSGR_INSTANCE: Messenger | None = None def get_msgr( From dc6436e0683351dd4f95af9a7d42db0d621f21b0 Mon Sep 17 00:00:00 2001 From: Rhythm Patel <138519018+RhythmP619@users.noreply.github.com> Date: Thu, 27 Nov 2025 00:00:26 +0530 Subject: [PATCH 4/7] Update python/grass/pygrass/messages/__init__.py Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com> --- python/grass/pygrass/messages/__init__.py | 1 + 1 file changed, 1 insertion(+) diff --git a/python/grass/pygrass/messages/__init__.py b/python/grass/pygrass/messages/__init__.py index b8ed18126a8..461848f6454 100644 --- a/python/grass/pygrass/messages/__init__.py +++ b/python/grass/pygrass/messages/__init__.py @@ -341,6 +341,7 @@ def test_fatal_error(self, message: str) -> None: _MSGR_INSTANCE: Messenger | None = None + def get_msgr( *args, **kwargs, From 3da5f10972997cc7e2c34ce29e97b5428fdc6207 Mon Sep 17 00:00:00 2001 From: RhythmP619 Date: Sat, 29 Nov 2025 12:39:15 +0530 Subject: [PATCH 5/7] doc: document None default handling for temporal lists --- python/grass/temporal/temporal_algebra.py | 26 +++++++++++++------ .../temporal/temporal_raster_base_algebra.py | 16 +++++++----- .../grass/temporal/temporal_vector_algebra.py | 9 ++++--- 3 files changed, 34 insertions(+), 17 deletions(-) diff --git a/python/grass/temporal/temporal_algebra.py b/python/grass/temporal/temporal_algebra.py index 7119b2e7999..d6eb1ce91b2 100644 --- a/python/grass/temporal/temporal_algebra.py +++ b/python/grass/temporal/temporal_algebra.py @@ -1118,7 +1118,8 @@ def set_temporal_extent_list(self, maplist, topolist=None, temporal="l"): :param maplist: List of map objects for which relations has been build correctly. - :param topolist: List of strings of temporal relations. + :param topolist: List of strings of temporal relations. If None, + defaults to ["EQUAL"] internally. :param temporal: The temporal operator specifying the temporal extent operation (intersection, union, disjoint union, right reference, left reference). @@ -1413,7 +1414,8 @@ def build_spatio_temporal_topology_list( :param maplistA: List of maps. :param maplistB: List of maps. - :param topolist: List of strings of spatio-temporal relations. + :param topolist: List of strings of spatio-temporal relations. If None, + defaults to ["EQUAL"] internally. :param assign_val: Boolean for assigning a boolean map value based on the map_values from the compared map list by topological relationships. @@ -1634,8 +1636,10 @@ def assign_bool_value( compared map list by topological relationships. :param map_i: Map object with temporal extent. - :param temporal_topo_list: List of strings for given temporal relations. - :param spatial_topo_list: List of strings for given spatial relations. + :param temporal_topo_list: List of strings for given temporal relations. If None, + defaults to ["EQUAL"] internally. + :param spatial_topo_list: List of strings for given spatial relations. If None, + defaults to empty list internally. :return: Map object with conditional value that has been assigned by relation maps that fulfil the topological relationships to @@ -1685,8 +1689,10 @@ def compare_bool_value( :param map_i: Map object with temporal extent. :param compop: Comparison operator, && or ||. :param aggregate: Aggregation operator for relation map list, & or \\|. - :param temporal_topo_list: List of strings for given temporal relations. - :param spatial_topo_list: List of strings for given spatial relations. + :param temporal_topo_list: List of strings for given temporal relations. If None, + defaults to ["EQUAL"] internally. + :param spatial_topo_list: List of strings for given spatial relations. If None, + defaults to empty list internally. :return: Map object with conditional value that has been evaluated by comparison operators. @@ -1780,7 +1786,8 @@ def perform_temporal_selection( expression. :param maplistB: List of maps representing the right side of a temporal expression. - :param topolist: List of strings of temporal relations. + :param topolist: List of strings of temporal relations. If None, + defaults to ["EQUAL"] internally. :param inverse: Boolean value that specifies if the selection should be inverted. :param assign_val: Boolean for assigning a boolean map value based on @@ -1868,7 +1875,8 @@ def set_granularity(self, maplistA, maplistB, toperator="l", topolist=None): :param maplistB: List of maps. :param maplistB: List of maps. :param toperator: String containing the temporal operator: l, r, d, i, u. - :param topolist: List of topological relations. + :param topolist: List of topological relations. If None, defaults to ["EQUAL"] + internally. :return: List of maps with the new temporal extends. @@ -2175,6 +2183,8 @@ def eval_map_list(self, maplist, thenlist, topolist=None): :param maplist: List of map objects containing boolean map values. :param thenlist: List of map objects where the boolean values should be added. + :param topolist: List of temporal relations between the two map lists. If None, + defaults to ["EQUAL"] internally. :return: List of maps from thenlist with added conditional boolean values. """ diff --git a/python/grass/temporal/temporal_raster_base_algebra.py b/python/grass/temporal/temporal_raster_base_algebra.py index f3349f59cf4..8413d9f5aa0 100644 --- a/python/grass/temporal/temporal_raster_base_algebra.py +++ b/python/grass/temporal/temporal_raster_base_algebra.py @@ -220,7 +220,8 @@ def build_spatio_temporal_topology_list( :param maplistA: List of maps. :param maplistB: List of maps. - :param topolist: List of strings of temporal relations. + :param topolist: List of strings of temporal relations. If None, + defaults to ["EQUAL"] internally. :param assign_val: Boolean for assigning a boolean map value based on the map_values from the compared map list by topological relationships. @@ -439,8 +440,10 @@ def compare_cmd_value( :param map_i: Map object with temporal extent. :param compop: Comparison operator, && or ||. :param aggregate: Aggregation operator for relation map list, & or \\|. - :param temporal_topo_list: List of strings for given temporal relations. - :param spatial_topo_list: List of strings for given spatial relations. + :param temporal_topo_list: List of strings for given temporal relations. If None, + defaults to ["EQUAL"] internally. + :param spatial_topo_list: List of strings for given spatial relations. If None, + defaults to empty list internally. :param convert: Boolean if conditional values should be converted to r.mapcalc command strings. @@ -573,7 +576,8 @@ def set_temporal_extent_list( :param maplist: List of map objects for which relations has been build correctly. - :param topolist: List of strings of temporal relations. + :param topolist: List of strings of temporal relations. If None, + defaults to ["EQUAL"] internally. :param temporal: The temporal operator specifying the temporal extent operation (intersection, union, disjoint union, right reference, left reference). @@ -668,9 +672,9 @@ def build_condition_cmd_list( :param elselist: Map list with temporal extents and command list or numeric string. :param condition_topolist: List of strings for given temporal relations between - conditions and conclusions. + conditions and conclusions. If None, defaults to ["EQUAL"] internally. :param conclusion_topolist: List of strings for given temporal relations between - conditions (then and else). + conditions (then and else). If None, defaults to ["EQUAL"] internally. :param temporal: The temporal operator specifying the temporal extent operation (intersection, union, disjoint union, right reference, left reference). diff --git a/python/grass/temporal/temporal_vector_algebra.py b/python/grass/temporal/temporal_vector_algebra.py index 376496e8cec..34a72605c3a 100644 --- a/python/grass/temporal/temporal_vector_algebra.py +++ b/python/grass/temporal/temporal_vector_algebra.py @@ -202,7 +202,8 @@ def build_spatio_temporal_topology_list( :param maplistA: List of maps. :param maplistB: List of maps. - :param topolist: List of strings of temporal relations. + :param topolist: List of strings of temporal relations. If None, + defaults to ["EQUAL"] internally. :param assign_val: Boolean for assigning a boolean map value based on the map_values from the compared map list by topological relationships. @@ -292,7 +293,8 @@ def overlay_cmd_value(self, map_i, tbrelations, function, topolist=None): :param map_i: Map object with temporal extent. :param tbrelations: List of temporal relation to map_i. - :param topolist: List of strings for given temporal relations. + :param topolist: List of strings for given temporal relations. If None, + defaults to ["EQUAL"] internally. :param function: Overlay operator, &|+^~. :return: Map object with command list with operators that has been @@ -355,7 +357,8 @@ def set_temporal_extent_list(self, maplist, topolist=None, temporal="l"): :param maplist: List of map objects for which relations has been build correctly. - :param topolist: List of strings of temporal relations. + :param topolist: List of strings of temporal relations. If None, + defaults to ["EQUAL"] internally. :param temporal: The temporal operator specifying the temporal extent operation (intersection, union, disjoint union, right reference, left reference). From 2ce15a02a8620b26daca801c587b82bc0b5cfe6e Mon Sep 17 00:00:00 2001 From: RhythmP619 Date: Sat, 29 Nov 2025 20:55:41 +0530 Subject: [PATCH 6/7] doc: remove trailing whitespaces --- python/grass/temporal/temporal_raster_base_algebra.py | 3 +-- python/grass/temporal/temporal_vector_algebra.py | 6 ++---- 2 files changed, 3 insertions(+), 6 deletions(-) diff --git a/python/grass/temporal/temporal_raster_base_algebra.py b/python/grass/temporal/temporal_raster_base_algebra.py index 8413d9f5aa0..7dbbc93ebd7 100644 --- a/python/grass/temporal/temporal_raster_base_algebra.py +++ b/python/grass/temporal/temporal_raster_base_algebra.py @@ -220,8 +220,7 @@ def build_spatio_temporal_topology_list( :param maplistA: List of maps. :param maplistB: List of maps. - :param topolist: List of strings of temporal relations. If None, - defaults to ["EQUAL"] internally. +what :param topolist: List of strings of temporal relations. If None, defaults to ["EQUAL"] internally. :param assign_val: Boolean for assigning a boolean map value based on the map_values from the compared map list by topological relationships. diff --git a/python/grass/temporal/temporal_vector_algebra.py b/python/grass/temporal/temporal_vector_algebra.py index 34a72605c3a..64c09c24dec 100644 --- a/python/grass/temporal/temporal_vector_algebra.py +++ b/python/grass/temporal/temporal_vector_algebra.py @@ -202,8 +202,7 @@ def build_spatio_temporal_topology_list( :param maplistA: List of maps. :param maplistB: List of maps. - :param topolist: List of strings of temporal relations. If None, - defaults to ["EQUAL"] internally. + :param topolist: List of strings of temporal relations. If None, defaults to ["EQUAL"] internally. :param assign_val: Boolean for assigning a boolean map value based on the map_values from the compared map list by topological relationships. @@ -357,8 +356,7 @@ def set_temporal_extent_list(self, maplist, topolist=None, temporal="l"): :param maplist: List of map objects for which relations has been build correctly. - :param topolist: List of strings of temporal relations. If None, - defaults to ["EQUAL"] internally. + :param topolist: List of strings of temporal relations. If None, defaults to ["EQUAL"] internally. :param temporal: The temporal operator specifying the temporal extent operation (intersection, union, disjoint union, right reference, left reference). From 3d58171937d3d790e3904af46a8e0e4794b49403 Mon Sep 17 00:00:00 2001 From: RhythmP619 Date: Sun, 30 Nov 2025 16:13:15 +0530 Subject: [PATCH 7/7] doc: fix doctring --- python/grass/temporal/temporal_raster_base_algebra.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/python/grass/temporal/temporal_raster_base_algebra.py b/python/grass/temporal/temporal_raster_base_algebra.py index 7dbbc93ebd7..06c8301be86 100644 --- a/python/grass/temporal/temporal_raster_base_algebra.py +++ b/python/grass/temporal/temporal_raster_base_algebra.py @@ -220,7 +220,7 @@ def build_spatio_temporal_topology_list( :param maplistA: List of maps. :param maplistB: List of maps. -what :param topolist: List of strings of temporal relations. If None, defaults to ["EQUAL"] internally. + :param topolist: List of strings of temporal relations. If None, defaults to ["EQUAL"] internally. :param assign_val: Boolean for assigning a boolean map value based on the map_values from the compared map list by topological relationships.