Skip to content

Commit 5192054

Browse files
sbryngelsonclaude
andcommitted
Fix mid_step ignoring t_step_start in restart roundtrip
mid_step = t_step_stop // 2 produced an invalid Phase 1 run when t_step_start > 0, since the midpoint could fall before the start step. Use (t_step_start + t_step_stop) // 2 so Phase 1 always has a valid range. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
1 parent c351f1b commit 5192054

1 file changed

Lines changed: 1 addition & 1 deletion

File tree

toolchain/mfc/test/case.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -145,7 +145,7 @@ def run(self, targets: List[Union[str, MFCTarget]], gpus: Set[int]) -> subproces
145145

146146
def run_restart(self, targets, gpus):
147147
"""Run a restart roundtrip: simulate to midpoint, then restart to end."""
148-
mid_step = self.params['t_step_stop'] // 2
148+
mid_step = (self.params['t_step_start'] + self.params['t_step_stop']) // 2
149149
orig = dict(self.params)
150150

151151
try:

0 commit comments

Comments
 (0)