Merge pull request #1037 from mantidproject/1036_recon_is_running

Recon: Set recon_is_running earlier
mantidproject · Jul 5, 2021 · 106cf44 · 106cf44
2 parents b8bac04 + 026b6d1
commit 106cf44
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 1 deletion.
diff --git a/docs/release_notes/next.rst b/docs/release_notes/next.rst
@@ -31,6 +31,7 @@ Fixes
 - #1001 : Exception when selecting Log in Load Dataset dialog
 - #991 : Handle longer running previews better
 - #1021 : Prevent simultaneous Astra calls
+- #1036 : Exception in ReconstructWindowView.show_recon_volume prevents recon closing
 
 Developer Changes
 -----------------

diff --git a/mantidimaging/gui/windows/recon/presenter.py b/mantidimaging/gui/windows/recon/presenter.py
@@ -268,8 +268,8 @@ def do_cor_fit(self):
         self.do_preview_reconstruct_slice()
 
     def _on_volume_recon_done(self, task):
-        self.view.show_recon_volume(task.result)
         self.recon_is_running = False
+        self.view.show_recon_volume(task.result)
         self.view.recon_applied.emit()
 
     def do_clear_all_cors(self):