Rather call done twice instead of skip, as it produces empty results …

…sometimes otherwise
ls1intum · Jan 17, 2025 · cdc0430 · cdc0430
1 parent 274f640
commit cdc0430
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 4 deletions.
diff --git a/app/pipeline/chat_gpt_wrapper_pipeline.py b/app/pipeline/chat_gpt_wrapper_pipeline.py
@@ -98,5 +98,5 @@ def __call__(
         )
 
         response = (prompt | self.pipeline).invoke({})
+        self.callback.done()
         self.callback.done(final_result=response)
-        self.callback.skip("Skipping suggestion generation as no output was generated.")
diff --git a/app/web/routers/pipelines.py b/app/web/routers/pipelines.py
@@ -75,12 +75,12 @@ def run_exercise_chat_pipeline(
         description="Exercise Chat Pipeline Execution DTO"
     ),
 ):
-    if variant == "default":
+    if variant == "chat-gpt-wrapper":
+        thread = Thread(target=run_chatgpt_wrapper_pipeline_worker, args=(dto, variant))
+    else:
         thread = Thread(
             target=run_exercise_chat_pipeline_worker, args=(dto, variant, event)
         )
-    else:
-        thread = Thread(target=run_chatgpt_wrapper_pipeline_worker, args=(dto, variant))
     thread.start()