SDK: Fix FileExistsErrors on Windows when a dataset is loaded from …

…cache `atomic_writer` is supposed to overwrite the destination file, but `os.rename` doesn't do that on Windows. We have to use `os.replace` instead.
cvat-ai · Aug 21, 2023 · 1724bfa · 1724bfa
1 parent 5889c43
commit 1724bfa
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 2 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -25,7 +25,8 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ### Fixed
 
-- TBD
+- \[SDK\]: `FileExistsError` exception raised on Windows when a dataset is loaded from cache
+  (<https://github.com/opencv/cvat/pull/6722>)
 
 ### Security
 

diff --git a/cvat-sdk/cvat_sdk/core/utils.py b/cvat-sdk/cvat_sdk/core/utils.py
@@ -75,7 +75,7 @@ def atomic_writer(
     try:
         with tmp_file:
             yield tmp_file
-        os.rename(tmp_path, path)
+        os.replace(tmp_path, path)
     except:
         os.unlink(tmp_path)
         raise