Skip to content
Merged
Show file tree
Hide file tree
Changes from 1 commit
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
minor fix
  • Loading branch information
0x404 committed Jun 9, 2025
commit c9696823771986e37e7fb66a525d7cbd41744c14
2 changes: 1 addition & 1 deletion verl/trainer/config/ppo_trainer.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -578,7 +578,7 @@ critic:

# checkpoint configs
checkpoint:

# What to include in saved checkpoints
# with 'hf_model' you can save whole model as hf format, now only use sharded model checkpoint to save space
save_contents: ['model', 'optimizer', 'extra']
Expand Down
6 changes: 3 additions & 3 deletions verl/utils/checkpoint/fsdp_checkpoint_manager.py
Original file line number Diff line number Diff line change
Expand Up @@ -45,9 +45,9 @@ class FSDPCheckpointManager(BaseCheckpointManager):
processing_class (PreTrainedTokenizer or ProcessorMixin, optional):
Pre-/post-processing artifact handler.
checkpoint_load_contents (list[str], optional):
Components to load; must contain 'model', 'optimizer', 'extra'.
Components to load; must contain 'model'. Defaults to ['model', 'optimizer', 'extra'].
checkpoint_save_contents (list[str], optional):
Components to save; must contain 'model', 'optimizer', 'extra'.
Components to save; must contain 'model'. Defaults to ['model', 'optimizer', 'extra'].
"""

def __init__(
Expand Down Expand Up @@ -89,7 +89,7 @@ def load_checkpoint(self, local_path: str, hdfs_path: str = None, del_local_afte
"""
if local_path is None:
return

# check if the checkpoint_load_contents is valid
if self.should_load_model:
assert self.model is not None, "model must be provided when checkpoint_contents.load includes ['model']"
Expand Down
Loading