From e8d7e093f718aa2d59a2d5901d923fcba1bfd376 Mon Sep 17 00:00:00 2001 From: "jinli.yl" Date: Fri, 20 Jun 2025 12:36:51 +0800 Subject: [PATCH] bugfix --- .../module/context_generator/step_context_generator.py | 2 +- experiencemaker/module/summarizer/step_summarizer.py | 7 ++++++- 2 files changed, 7 insertions(+), 2 deletions(-) diff --git a/experiencemaker/module/context_generator/step_context_generator.py b/experiencemaker/module/context_generator/step_context_generator.py index 6a16bd72..31041483 100644 --- a/experiencemaker/module/context_generator/step_context_generator.py +++ b/experiencemaker/module/context_generator/step_context_generator.py @@ -291,7 +291,7 @@ class StepContextGenerator(BaseContextGenerator, PromptMixin): return [int(num) for num in numbers] except Exception as e: - logger.error(f"Error parsing rerank response: {e}") + logger.exception(f"Error parsing rerank response: {e}") return [] def _format_experiences_for_context(self, experiences: List[VectorStoreNode]) -> str: diff --git a/experiencemaker/module/summarizer/step_summarizer.py b/experiencemaker/module/summarizer/step_summarizer.py index ae7afaa7..90333417 100644 --- a/experiencemaker/module/summarizer/step_summarizer.py +++ b/experiencemaker/module/summarizer/step_summarizer.py @@ -362,7 +362,12 @@ class StepSummarizer(BaseSummarizer, PromptMixin): # Parse validation result is_valid = "valid" in response_content.lower() and "invalid" not in response_content.lower() score_match = re.search(r'score[:\s]*([0-9.]+)', response_content.lower()) - score = float(score_match.group(1)) if score_match else 0.5 + + # add by jinli 0620 + try: + score = float(score_match.group(1)) if score_match else 0.5 + except Exception: + score = -1 return { "is_valid": is_valid and score > 0.3,