Skip to content

Commit

Permalink
fixed issue with canceling dialog during open analysis
Browse files Browse the repository at this point in the history
  • Loading branch information
Philip Colangelo committed Jan 30, 2025
1 parent 2d46474 commit 821d840
Showing 1 changed file with 7 additions and 2 deletions.
9 changes: 7 additions & 2 deletions src/digest/multi_model_selection_page.py
Original file line number Diff line number Diff line change
Expand Up @@ -47,7 +47,9 @@ def run(self):

for file, model in self.model_dict.items():
if self.user_canceled:
break
self.close_progress.emit()
self.completed.emit([])
return
self.step_progress.emit()
if model:
continue
Expand All @@ -65,7 +67,7 @@ def run(self):

self.close_progress.emit()

model_list = [model for model in self.model_dict.values()]
model_list = list(self.model_dict.values())

self.completed.emit(model_list)

Expand Down Expand Up @@ -362,6 +364,9 @@ def start_analysis(self):
def open_analysis(
self, model_list: List[Union[DigestOnnxModel, DigestReportModel]]
):
if not model_list:
return

multi_model_analysis = MultiModelAnalysis(model_list)
self.analysis_window.setCentralWidget(multi_model_analysis)
self.analysis_window.setWindowIcon(QIcon(":/assets/images/digest_logo_500.jpg"))
Expand Down

0 comments on commit 821d840

Please sign in to comment.