Skip to content

Commit 090ff22

Browse files
committed
fix
1 parent 8b0d2c9 commit 090ff22

File tree

1 file changed

+11
-17
lines changed

1 file changed

+11
-17
lines changed

src/unitxt/llm_as_judge.py

Lines changed: 11 additions & 17 deletions
Original file line numberDiff line numberDiff line change
@@ -803,12 +803,12 @@ def compute(
803803
"option_b": option_pair[1],
804804
"criteria_name": criterias[i].name,
805805
"criteria_description": criterias[i].description,
806-
"response_variable_name": response_variable_name,
807-
"response_variable_name_title": response_variable_name.capitalize(),
806+
"response_variable_name": response_variable_names[i],
807+
"response_variable_name_title": response_variable_names[i].capitalize(),
808808
"data_classification_policy": ["public"],
809809
}
810-
for i, (response_pairs, option_pairs, response_variable_name) in enumerate(
811-
zip(response_pairs_list, option_pairs_list, response_variable_names)
810+
for i, (response_pairs, option_pairs) in enumerate(
811+
zip(response_pairs_list, option_pairs_list)
812812
)
813813
for response_pair, option_pair in zip(response_pairs, option_pairs)
814814
]
@@ -867,33 +867,27 @@ def compute(
867867
)
868868
self.logger.info("The summary was generated successfully.")
869869

870-
score_option_instruction_list = [
871-
"".join(
870+
score_option_instructions_list = [
871+
["".join(
872872
[
873873
f'Choose "{option}" if Response {option} is better quality.\n'
874874
for option in option_pair
875875
]
876-
)
876+
) for option_pair in option_pairs]
877877
for option_pairs in option_pairs_list
878-
for option_pair in option_pairs
879878
]
880879

881880
option_selection_instances = [
882881
{
883882
"options": [f"Response {option}" for option in option_pair],
884883
"score_option_instruction": score_option_instruction,
885-
"response_variable_name": response_variable_name,
884+
"response_variable_name": response_variable_names[i],
886885
"data_classification_policy": ["public"],
887886
}
888-
for option_pair, score_option_instruction, response_variable_name in zip(
889-
[
890-
option_pair
891-
for option_pairs in option_pairs_list
892-
for option_pair in option_pairs
893-
],
894-
score_option_instruction_list,
895-
response_variable_names
887+
for i, (score_option_instructions, option_pairs) in enumerate(
888+
zip(score_option_instructions_list, option_pairs_list)
896889
)
890+
for score_option_instruction, option_pair in zip(score_option_instructions, option_pairs)
897891
]
898892

899893
previous_messages = [

0 commit comments

Comments
 (0)