Set force_overwrite to default to True #161

pepkit · Apr 4, 2024 · 86a6483 · 86a6483
1 parent 1fe44eb
commit 86a6483
Show file tree

Hide file tree

Showing 4 changed files with 6 additions and 4 deletions.
diff --git a/pipestat/backends/db_backend/dbbackend.py b/pipestat/backends/db_backend/dbbackend.py
@@ -298,7 +298,7 @@ def report(
         self,
         values: Dict[str, Any],
         record_identifier: str,
-        force_overwrite: bool = False,
+        force_overwrite: bool = True,
         result_formatter: Optional[staticmethod] = None,
     ) -> Union[List[str], bool]:
         """

diff --git a/pipestat/backends/file_backend/filebackend.py b/pipestat/backends/file_backend/filebackend.py
@@ -326,7 +326,7 @@ def report(
         self,
         values: Dict[str, Any],
         record_identifier: Optional[str] = None,
-        force_overwrite: bool = False,
+        force_overwrite: bool = True,
         result_formatter: Optional[staticmethod] = None,
     ) -> Union[List[str], bool]:
         """

diff --git a/pipestat/pipestat.py b/pipestat/pipestat.py
@@ -551,7 +551,7 @@ def report(
         self,
         values: Dict[str, Any],
         record_identifier: Optional[str] = None,
-        force_overwrite: bool = False,
+        force_overwrite: bool = True,
         result_formatter: Optional[staticmethod] = None,
         strict_type: bool = True,
     ) -> Union[List[str], bool]:

diff --git a/tests/test_pipestat.py b/tests/test_pipestat.py
@@ -615,7 +615,9 @@ def test_retrieve_one_single_result_as_list(
             args.update(backend_data)
             psm = SamplePipestatManager(**args)
             psm.report(record_identifier=rec_id, values=val, force_overwrite=True)
-            assert psm.retrieve_one(record_identifier=rec_id, result_identifier="number_of_things") == psm.retrieve_one(record_identifier=rec_id, result_identifier=["number_of_things"])
+            assert psm.retrieve_one(
+                record_identifier=rec_id, result_identifier="number_of_things"
+            ) == psm.retrieve_one(record_identifier=rec_id, result_identifier=["number_of_things"])
 
     @pytest.mark.parametrize(
         ["rec_id", "val"],