Unverified Commit 7c32b79f authored by Gustaf Ahdritz's avatar Gustaf Ahdritz Committed by GitHub
Browse files

Merge pull request #290 from jonathanking/add_missing_warmupdecaylr_setting

Add missing warmup_num_steps parameters to DeepSpeed config generation script.
parents 886494ff 9b2a4f83
......@@ -248,9 +248,10 @@ if(args.scheduler is not None):
params["warmup_max_lr"] = args.warmup_max_lr
params["warmup_num_steps"] = args.warmup_num_steps
elif(args.scheduler == "WarmupDecayLR"):
params["total_num_steps"] = args.warmup_decay_total_num_steps
params["warmup_min_lr"] = args.warmup_decay_min_lr
params["warmup_max_lr"] = args.warmup_decay_max_lr
params["warmup_num_steps"] = args.warmup_decay_num_steps
params["total_num_steps"] = args.warmup_decay_total_num_steps
else:
raise ValueError("Invalid scheduler")
......
Markdown is supported
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment