Skip to content

Commit 4e1bb32

Browse files
authored
Reasoning tags removal is true by default (#925)
* rm reasoning tags true by default * Update src/lighteval/main_vllm.py
1 parent 4d2bf42 commit 4e1bb32

File tree

6 files changed

+46
-10
lines changed

6 files changed

+46
-10
lines changed

src/lighteval/main_accelerate.py

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -61,7 +61,11 @@ def accelerate( # noqa C901
6161
Optional[str], Option(help="Load responses from details directory.", rich_help_panel=HELP_PANEL_NAME_1)
6262
] = None,
6363
remove_reasoning_tags: Annotated[
64-
bool, Option(help="Remove reasoning tags from responses.", rich_help_panel=HELP_PANEL_NAME_1)
64+
bool | None,
65+
Option(
66+
help="Remove reasoning tags from responses (true to remove, false to leave - true by default).",
67+
rich_help_panel=HELP_PANEL_NAME_1,
68+
),
6569
] = True,
6670
reasoning_tags: Annotated[
6771
str | None,

src/lighteval/main_custom.py

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -54,7 +54,11 @@ def custom(
5454
int, Option(help="Number of seeds to use for few-shot evaluation.", rich_help_panel=HELP_PANEL_NAME_1)
5555
] = 1,
5656
remove_reasoning_tags: Annotated[
57-
bool, Option(help="Remove reasoning tags from responses.", rich_help_panel=HELP_PANEL_NAME_1)
57+
bool | None,
58+
Option(
59+
help="Remove reasoning tags from responses (true to remove, false to leave - true by default).",
60+
rich_help_panel=HELP_PANEL_NAME_1,
61+
),
5862
] = True,
5963
reasoning_tags: Annotated[
6064
str | None,

src/lighteval/main_endpoint.py

Lines changed: 20 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -63,7 +63,11 @@ def inference_endpoint(
6363
Optional[str], Option(help="Load responses from details directory.", rich_help_panel=HELP_PANEL_NAME_1)
6464
] = None,
6565
remove_reasoning_tags: Annotated[
66-
bool, Option(help="Remove reasoning tags from responses.", rich_help_panel=HELP_PANEL_NAME_1)
66+
bool | None,
67+
Option(
68+
help="Remove reasoning tags from responses (true to remove, false to leave - true by default).",
69+
rich_help_panel=HELP_PANEL_NAME_1,
70+
),
6771
] = True,
6872
reasoning_tags: Annotated[
6973
str | None,
@@ -188,7 +192,11 @@ def tgi(
188192
Optional[str], Option(help="Load responses from details directory.", rich_help_panel=HELP_PANEL_NAME_1)
189193
] = None,
190194
remove_reasoning_tags: Annotated[
191-
bool, Option(help="Remove reasoning tags from responses.", rich_help_panel=HELP_PANEL_NAME_1)
195+
bool | None,
196+
Option(
197+
help="Remove reasoning tags from responses (true to remove, false to leave - true by default).",
198+
rich_help_panel=HELP_PANEL_NAME_1,
199+
),
192200
] = True,
193201
reasoning_tags: Annotated[
194202
str | None,
@@ -320,7 +328,11 @@ def litellm(
320328
Optional[str], Option(help="Load responses from details directory.", rich_help_panel=HELP_PANEL_NAME_1)
321329
] = None,
322330
remove_reasoning_tags: Annotated[
323-
bool, Option(help="Remove reasoning tags from responses.", rich_help_panel=HELP_PANEL_NAME_1)
331+
bool | None,
332+
Option(
333+
help="Remove reasoning tags from responses (true to remove, false to leave - true by default).",
334+
rich_help_panel=HELP_PANEL_NAME_1,
335+
),
324336
] = True,
325337
reasoning_tags: Annotated[
326338
str | None,
@@ -486,7 +498,11 @@ def inference_providers(
486498
),
487499
] = False,
488500
remove_reasoning_tags: Annotated[
489-
bool, Option(help="Remove reasoning tags from responses.", rich_help_panel=HELP_PANEL_NAME_1)
501+
bool | None,
502+
Option(
503+
help="Remove reasoning tags from responses (true to remove, false to leave - true by default).",
504+
rich_help_panel=HELP_PANEL_NAME_1,
505+
),
490506
] = True,
491507
reasoning_tags: Annotated[
492508
str | None,

src/lighteval/main_nanotron.py

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -44,7 +44,11 @@ def nanotron(
4444
],
4545
lighteval_config_path: Annotated[str, Option(help="Path to a YAML config to be used for the evaluation.")],
4646
remove_reasoning_tags: Annotated[
47-
bool, Option(help="Remove reasoning tags from responses.", rich_help_panel=HELP_PANEL_NAME_1)
47+
bool | None,
48+
Option(
49+
help="Remove reasoning tags from responses (true to remove, false to leave - true by default).",
50+
rich_help_panel=HELP_PANEL_NAME_1,
51+
),
4852
] = True,
4953
reasoning_tags: Annotated[
5054
str | None,

src/lighteval/main_sglang.py

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -54,7 +54,11 @@ def sglang(
5454
Optional[str], Option(help="Load responses from details directory.", rich_help_panel=HELP_PANEL_NAME_1)
5555
] = None,
5656
remove_reasoning_tags: Annotated[
57-
bool, Option(help="Remove reasoning tags from responses.", rich_help_panel=HELP_PANEL_NAME_1)
57+
bool | None,
58+
Option(
59+
help="Remove reasoning tags from responses (true to remove, false to leave - true by default).",
60+
rich_help_panel=HELP_PANEL_NAME_1,
61+
),
5862
] = True,
5963
reasoning_tags: Annotated[
6064
str | None,

src/lighteval/main_vllm.py

Lines changed: 6 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -57,8 +57,12 @@ def vllm(
5757
Optional[str], Option(help="Load responses from details directory.", rich_help_panel=HELP_PANEL_NAME_1)
5858
] = None,
5959
remove_reasoning_tags: Annotated[
60-
bool, Option(help="Remove reasoning tags from responses.", rich_help_panel=HELP_PANEL_NAME_1)
61-
] = False,
60+
bool | None,
61+
Option(
62+
help="Remove reasoning tags from responses (true to remove, false to leave - true by default).",
63+
rich_help_panel=HELP_PANEL_NAME_1,
64+
),
65+
] = True,
6266
reasoning_tags: Annotated[
6367
str | None,
6468
Option(

0 commit comments

Comments
 (0)