Skip to content

Commit

Permalink
Internal change.
Browse files Browse the repository at this point in the history
PiperOrigin-RevId: 592911220
  • Loading branch information
niketkumar authored and t5-copybara committed Dec 21, 2023
1 parent f54468c commit 133d0b1
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions t5x/checkpoints.py
Original file line number Diff line number Diff line change
Expand Up @@ -2249,7 +2249,7 @@ class OrbaxCheckpointManagerInterface:
class _CheckpointManagerImpl(ocp.CheckpointManager):
"""CheckpointManager implementation to deal with metrics update."""

def _get_old_steps_to_remove(self):
def _get_old_steps_to_remove(self) -> List[int]:
"""Update metrics for Orbax management, if available."""
if self._track_best:
metric_name_to_monitor = self._options.metric_name_to_monitor # pytype: disable=attribute-error
Expand All @@ -2262,7 +2262,7 @@ def _get_old_steps_to_remove(self):
if info.step in step_to_metric:
metrics = {metric_name_to_monitor: step_to_metric[info.step]}
info.metrics = metrics
super()._get_old_steps_to_remove()
return super()._get_old_steps_to_remove()

def __init__(
self,
Expand Down

0 comments on commit 133d0b1

Please sign in to comment.