Skip to content

Commit dc48898

Browse files
sbryngelsonclaude
andcommitted
Add documentation comment to run_restart() about t_step_save constraint
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
1 parent a59eda6 commit dc48898

1 file changed

Lines changed: 4 additions & 0 deletions

File tree

toolchain/mfc/test/case.py

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -145,6 +145,10 @@ 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+
# NOTE: This method overrides t_step_save to produce exactly one save
149+
# per phase (at the boundary step). Tests using restart_check=True
150+
# must not rely on custom t_step_save values, as the straight run's
151+
# save points would not match the restart run's.
148152
mid_step = (self.params['t_step_start'] + self.params['t_step_stop']) // 2
149153
if mid_step <= self.params['t_step_start']:
150154
raise common.MFCException(

0 commit comments

Comments
 (0)