merge

SpiNNakerManchester · Oct 21, 2022 · c393a58 · c393a58
2 parents 04748ca + 0ca95f0
commit c393a58
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 1 deletion.
diff --git a/spinn_utilities/data/utils_data_view.py b/spinn_utilities/data/utils_data_view.py
@@ -72,6 +72,7 @@ def _clear(self):
         """
         Clears out all data
         """
+        self._report_dir_path = None
         self._hard_reset()
 
     def _hard_reset(self):
@@ -82,7 +83,6 @@ def _hard_reset(self):
         self._dsg_dir_path = None
         self._mapping_dir_path = None
         self._run_dir_path = None
-        self._report_dir_path = None
         self._requires_data_generation = True
         self._requires_mapping = True
         self._temporary_directory = None

diff --git a/spinn_utilities/data/utils_data_writer.py b/spinn_utilities/data/utils_data_writer.py
@@ -277,6 +277,12 @@ def get_report_dir_path(self):
         ..note: In unittest mode this returns a tempdir
         shared by all path methods
 
+        This is the high level directory which in which timestamp directories
+        and then run directories are placed.
+
+        As it is only accessed to create timestamp directories and
+        remove old reports this is not a view method.
+
         :rtpye: str
         :raises SpiNNUtilsException:
             If the simulation_time_step is currently unavailable