Merge pull request #187 from gachiemchiep/master

Json files will be written using utf-8 encoding

Author: Jorge Núñez Santiago

pyodi/apps/coco/coco_merge.py

@@ -92,7 +92,7 @@ def coco_merge(
         )
     )
 
-    with open(output_file, "w") as f:
-        json.dump(output, f, indent=indent)
+    with open(output_file, "w", encoding="utf-8") as f:
+        json.dump(output, f, indent=indent, ensure_ascii=False)
 
     return output_file

pyodi/apps/coco/coco_split.py

@@ -149,8 +149,8 @@ def property_split(
     output_files = []
     for split_type, split in zip(["train", "val"], [train_split, val_split]):
         output_files.append(output_filename + f"_{split_type}.json")
-        with open(output_files[-1], "w") as f:
-            json.dump(split, f, indent=2)
+        with open(output_files[-1], "w", encoding="utf-8") as f:
+            json.dump(split, f, indent=2, ensure_ascii=False)
 
     return output_files
 
@@ -217,7 +217,7 @@ def random_split(
     output_files = []
     for split_type, split in zip(["train", "val"], [train_split, val_split]):
         output_files.append(output_filename + f"_{split_type}.json")
-        with open(output_files[-1], "w") as f:
-            json.dump(split, f, indent=2)
+        with open(output_files[-1], "w", encoding="utf-8") as f:
+            json.dump(split, f, indent=2, ensure_ascii=False)
 
     return output_files

pyodi/apps/crops/crops_merge.py

@@ -91,7 +91,7 @@ def crops_merge(
             "annotations": annotations,
         }
 
-    with open(output_file, "w") as f:
-        json.dump(new_ground_truth, f, indent=2)
+    with open(output_file, "w", encoding="utf-8") as f:
+        json.dump(new_ground_truth, f, indent=2, ensure_ascii=False)
 
     return output_file

tests/apps/test_coco_merge.py

@@ -84,4 +84,4 @@ def test_coco_merge_with_json_indent(tmp_path, indent):
             output_file=tmp_path / "result.json",
             indent=indent,
         )
-        mock.assert_called_once_with(ANY, ANY, indent=indent)
+        mock.assert_called_once_with(ANY, ANY, indent=indent, ensure_ascii=False)