Skip to content

Commit 8ab1e82

Browse files
committed
fix: replace assertion with warning to allow default mismatch metrics
1 parent fd13dcd commit 8ab1e82

File tree

1 file changed

+4
-3
lines changed

1 file changed

+4
-3
lines changed

slime/utils/arguments.py

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1672,9 +1672,10 @@ def slime_validate_args(args):
16721672
assert not args.use_tis, "use_rollout_logprobs and use_tis cannot be set at the same time."
16731673

16741674
if args.get_mismatch_metrics:
1675-
assert (
1676-
args.custom_tis_function_path is not None
1677-
), "custom_tis_function_path must be set when get_mismatch_metrics is set"
1675+
if args.custom_tis_function_path is None:
1676+
logger.warning(
1677+
"get_mismatch_metrics is set but custom_tis_function_path is None. Using default vanilla_tis_function for metrics calculation."
1678+
)
16781679

16791680
if args.use_rollout_logprobs:
16801681
logger.info(

0 commit comments

Comments
 (0)