Skip to content

Commit

Permalink
Revert "Add assert for max_steps to be positive in MegatronGPTSFTModel (
Browse files Browse the repository at this point in the history
NVIDIA#9110)" (NVIDIA#9128)

This reverts commit b92e24c.
  • Loading branch information
pablo-garay authored May 7, 2024
1 parent 451d459 commit 47aadb1
Showing 1 changed file with 0 additions and 1 deletion.
Original file line number Diff line number Diff line change
Expand Up @@ -74,7 +74,6 @@ def __init__(self, cfg: DictConfig, trainer: Trainer):
raise ImportError(
"Apex was not found. Please see the NeMo README for installation instructions: https://github.com/NVIDIA/NeMo#megatron-gpt."
)
assert trainer.max_steps > 0, "max_steps for SFT can't be negative as its required to build the dataset"
super().__init__(cfg, trainer=trainer)
self.sep_id = cfg.get('sep_id', 49704)
if hasattr(self.cfg.data, "validation_ds"):
Expand Down

0 comments on commit 47aadb1

Please sign in to comment.