Fix conflict, use built-in os function

fastmachinelearning · Nov 27, 2024 · 92dc849 · 92dc849
1 parent 0c4f958
commit 92dc849
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 3 deletions.
diff --git a/hls4ml/backends/vitis/passes/fifo_depth_optimization.py b/hls4ml/backends/vitis/passes/fifo_depth_optimization.py
@@ -71,7 +71,7 @@ def override_test_bench(model):
         new_test_bench.write(newline)
 
     # replace the old test bench with the new test bench that includes a for-loop
-    os.system(f"mv {path_to_new_test_bench} {path_to_old_test_bench}")
+    os.replace(path_to_new_test_bench, path_to_old_test_bench)
     return
 
 

diff --git a/hls4ml/writer/vitis_writer.py b/hls4ml/writer/vitis_writer.py
@@ -55,6 +55,5 @@ def write_hls(self, model):
         """
         super().write_hls(model)
         self.write_nnet_utils_overrides(model)
-        self.write_board_script(model)
-        os.remove(model.config.get_output_dir() + '.tar.gz')
         self.write_tar(model)
+        self.write_board_script(model)