Skip to content

Commit 28dbe33

Browse files
committed
add more informative messaging on ScheduleProblem
1 parent 7b25526 commit 28dbe33

File tree

2 files changed

+3
-2
lines changed

2 files changed

+3
-2
lines changed

src/virtualship/cli/_run.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -98,7 +98,7 @@ def _run(expedition_dir: str | Path, from_data: Path | None = None) -> None:
9898
)
9999
if isinstance(schedule_results, ScheduleProblem):
100100
print(
101-
"Update your schedule and continue the expedition by running the tool again."
101+
f"SIMULATION PAUSED: update your schedule (`virtualship plan`) and continue the expedition by executing the `virtualship run` command again.\nCheckpoint has been saved to {expedition_dir.joinpath(CHECKPOINT)}."
102102
)
103103
_save_checkpoint(
104104
Checkpoint(

src/virtualship/expedition/simulate_schedule.py

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -123,7 +123,8 @@ def simulate(self) -> ScheduleOk | ScheduleProblem:
123123
if waypoint.time is not None and self._time > waypoint.time:
124124
print(
125125
f"Waypoint {wp_i + 1} could not be reached in time. Current time: {self._time}. Waypoint time: {waypoint.time}."
126-
"\n\nHave you ensured that your schedule includes sufficient time for taking measurements such as CTD casts (in addition to the time it takes to sail between waypoints)?\n\n"
126+
"\n\nHave you ensured that your schedule includes sufficient time for taking measurements, e.g. CTD casts (in addition to the time it takes to sail between waypoints)?\n"
127+
"**Note**, the `virtualship plan` tool will not account for measurement times when verifying the schedule, only the time it takes to sail between waypoints.\n"
127128
)
128129
return ScheduleProblem(self._time, wp_i)
129130
else:

0 commit comments

Comments
 (0)