From 9aabaf680d35c36ed905732a702d00554c9f27cc Mon Sep 17 00:00:00 2001 From: Katon Minhas Date: Fri, 6 Feb 2026 15:38:15 -0500 Subject: [PATCH] Resolve comments --- src/crosswalk/crosswalk_builder.py | 1 - src/pipelines/clients/bcbs_promise/prompts/prompt_calls.py | 7 ++----- src/pipelines/clients/clover/prompts/prompt_calls.py | 7 ++----- 3 files changed, 4 insertions(+), 11 deletions(-) diff --git a/src/crosswalk/crosswalk_builder.py b/src/crosswalk/crosswalk_builder.py index 3825b5a..969573b 100644 --- a/src/crosswalk/crosswalk_builder.py +++ b/src/crosswalk/crosswalk_builder.py @@ -165,7 +165,6 @@ class CrosswalkBuilder: else: reverse_dict[target] = [str(source)] - # Then join the lists with commas return {target: sources for target, sources in reverse_dict.items()} def map_value(self, key_value): diff --git a/src/pipelines/clients/bcbs_promise/prompts/prompt_calls.py b/src/pipelines/clients/bcbs_promise/prompts/prompt_calls.py index 4775dc7..aa5d58e 100644 --- a/src/pipelines/clients/bcbs_promise/prompts/prompt_calls.py +++ b/src/pipelines/clients/bcbs_promise/prompts/prompt_calls.py @@ -4,10 +4,7 @@ import src.config as config import src.prompts.prompt_templates as prompt_templates from src.constants.constants import Constants from src.prompts.fieldset import Field, FieldSet -from src.utils import llm_utils, string_utils, timing_utils, rag_utils -from src.pipelines.shared.preprocessing.hybrid_smart_chunking_funcs import ( - format_chunks_for_llm, -) +from src.utils import llm_utils, string_utils import json @@ -317,7 +314,7 @@ def prompt_special_case_breakout(breakout_template, term_answer, filename): logging.warning( f"Expected prompt string, got {type(prompt)} in prompt_special_case_breakout; coercing to str." ) - prompt, _parser = str(prompt) + prompt = str(prompt) # Look for a corresponding _INSTRUCTION() function for caching template_name = getattr(breakout_template, "__name__", "SPECIAL_CASE_BREAKOUT") diff --git a/src/pipelines/clients/clover/prompts/prompt_calls.py b/src/pipelines/clients/clover/prompts/prompt_calls.py index 59e6571..02886c4 100644 --- a/src/pipelines/clients/clover/prompts/prompt_calls.py +++ b/src/pipelines/clients/clover/prompts/prompt_calls.py @@ -4,10 +4,7 @@ import src.config as config import src.prompts.prompt_templates as prompt_templates from src.constants.constants import Constants from src.prompts.fieldset import Field, FieldSet -from src.utils import llm_utils, string_utils, timing_utils, rag_utils -from src.pipelines.shared.preprocessing.hybrid_smart_chunking_funcs import ( - format_chunks_for_llm, -) +from src.utils import llm_utils, string_utils import json @@ -324,7 +321,7 @@ def prompt_special_case_breakout(breakout_template, term_answer, filename): logging.warning( f"Expected prompt string, got {type(prompt)} in prompt_special_case_breakout; coercing to str." ) - prompt, _parser = str(prompt) + prompt = str(prompt) # Look for a corresponding _INSTRUCTION() function for caching template_name = getattr(breakout_template, "__name__", "SPECIAL_CASE_BREAKOUT")