Skip to content

Commit b063ac9

Browse files
committed
Fix comment clarity in maybe_evaluate function in grpo_fast.py to accurately reflect timeout logic.
1 parent 1986faa commit b063ac9

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

open_instruct/grpo_fast.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2650,7 +2650,7 @@ def maybe_evaluate(
26502650
):
26512651
"""Optionally evaluate the model."""
26522652
try:
2653-
# timeout 0.01 if this is the last training step or we're not evaluating
2653+
# timeout 0.01 if this is not the last training step or we're not evaluating
26542654
# otherwise, wait to get the last evaluation generations (long timeout just in case)
26552655
timeout = 0.01 if (training_step < args.num_training_steps or args.local_eval_every < 0) else 100
26562656

0 commit comments

Comments
 (0)