Skip to content

Commit 57a988d

Browse files
committed
fix: multistep lr decay epoch bugs
- add milestones arguments - change decay_epochs to milestones variable
1 parent 6d4665b commit 57a988d

File tree

2 files changed

+3
-1
lines changed

2 files changed

+3
-1
lines changed

timm/scheduler/scheduler_factory.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -71,7 +71,7 @@ def create_scheduler(args, optimizer):
7171
elif args.sched == 'multistep':
7272
lr_scheduler = MultiStepLRScheduler(
7373
optimizer,
74-
decay_t=args.decay_epochs,
74+
decay_t=args.milestones,
7575
decay_rate=args.decay_rate,
7676
warmup_lr_init=args.warmup_lr,
7777
warmup_t=args.warmup_epochs,

train.py

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -171,6 +171,8 @@
171171
help='epoch repeat multiplier (number of times to repeat dataset epoch per train epoch).')
172172
parser.add_argument('--start-epoch', default=None, type=int, metavar='N',
173173
help='manual epoch number (useful on restarts)')
174+
parser.add_argument('--milestones', default=[30, 60], type=int, nargs='+', metavar="MILESTONES",
175+
help='list of epoch indices for multistep lr. must be increasing')
174176
parser.add_argument('--decay-epochs', type=float, default=100, metavar='N',
175177
help='epoch interval to decay LR')
176178
parser.add_argument('--warmup-epochs', type=int, default=3, metavar='N',

0 commit comments

Comments
 (0)