Skip to content

Commit

Permalink
remove unused attribute lr_scheduler_interval
Browse files Browse the repository at this point in the history
  • Loading branch information
catwell committed Dec 20, 2024
1 parent 51bbd4f commit 9e49bc5
Show file tree
Hide file tree
Showing 3 changed files with 0 additions and 4 deletions.
2 changes: 0 additions & 2 deletions src/refiners/training_utils/clock.py
Original file line number Diff line number Diff line change
Expand Up @@ -22,12 +22,10 @@ def __init__(
self,
training_duration: TimeValue,
gradient_accumulation: Step,
lr_scheduler_interval: Iteration | Epoch,
verbose: bool = True,
) -> None:
self.training_duration = training_duration
self.gradient_accumulation = gradient_accumulation
self.lr_scheduler_interval = lr_scheduler_interval
self.verbose = verbose
self.start_time = None
self.end_time = None
Expand Down
1 change: 0 additions & 1 deletion src/refiners/training_utils/trainer.py
Original file line number Diff line number Diff line change
Expand Up @@ -139,7 +139,6 @@ def clock(self, config: ClockConfig) -> TrainingClock:
return TrainingClock(
training_duration=self.config.training.duration,
gradient_accumulation=self.config.training.gradient_accumulation,
lr_scheduler_interval=self.config.lr_scheduler.update_interval,
verbose=config.verbose,
)

Expand Down
1 change: 0 additions & 1 deletion tests/training_utils/test_trainer.py
Original file line number Diff line number Diff line change
Expand Up @@ -205,7 +205,6 @@ def training_clock() -> TrainingClock:
return TrainingClock(
training_duration=Epoch(5),
gradient_accumulation=Step(1),
lr_scheduler_interval=Epoch(1),
)


Expand Down

0 comments on commit 9e49bc5

Please sign in to comment.