Skip to content

Commit 1bf35a0

Browse files
Jack-Khuumalfet
authored andcommitted
Move compile prefill out of generator parser to generic arg (#892)
* Move Compile Prefill Out of being generation specific * fix typo
1 parent b4f9162 commit 1bf35a0

File tree

1 file changed

+5
-5
lines changed

1 file changed

+5
-5
lines changed

cli.py

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -86,6 +86,11 @@ def add_arguments_for_verb(parser, verb: str) -> None:
8686
action="store_true",
8787
help="Whether to compile the model with torch.compile",
8888
)
89+
parser.add_argument(
90+
"--compile-prefill",
91+
action="store_true",
92+
help="Whether to compile the prefill. Improves prefill perf, but has higher compile times.",
93+
)
8994
parser.add_argument(
9095
"--profile",
9196
type=Path,
@@ -302,11 +307,6 @@ def _add_generation_args(parser) -> None:
302307
generator_parser.add_argument(
303308
"--temperature", type=float, default=0.8, help="Temperature for sampling"
304309
)
305-
generator_parser.add_argument(
306-
"--compile-prefill",
307-
action="store_true",
308-
help="Whether to compile the prefill. Improves prefill perf, but has higher compile times.",
309-
)
310310
generator_parser.add_argument(
311311
"--sequential-prefill",
312312
action="store_true",

0 commit comments

Comments
 (0)