Avoid printing ModelCheckpoint log with monitor=None and verbose=True (…

…#6109)
Lightning-AI · Feb 22, 2021 · 57215b7 · 57215b7
1 parent 1d28d11
commit 57215b7
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 1 deletion.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -33,6 +33,9 @@ The format is based on [Keep a Changelog](http://keepachangelog.com/en/1.0.0/).
 - Move lightning module to correct device type when using LightningDistributedWrapper ([#6070](https://github.com/PyTorchLightning/pytorch-lightning/pull/6070))
 
 
+- Do not print top-k verbose log with `ModelCheckpoint(monitor=None)` ([#6109](https://github.com/PyTorchLightning/pytorch-lightning/pull/6109))
+
+
 - Fixed error message for AMP + CPU incompatibility ([#6107](https://github.com/PyTorchLightning/pytorch-lightning/pull/6107))
 
 

diff --git a/pytorch_lightning/callbacks/model_checkpoint.py b/pytorch_lightning/callbacks/model_checkpoint.py
@@ -564,7 +564,7 @@ def _save_top_k_checkpoints(self, trainer, pl_module, metrics):
 
  if self.check_monitor_top_k(current):
  self._update_best_and_save(current, epoch, step, trainer, pl_module, metrics)
- elif self.verbose:
+ elif self.monitor is not None and self.verbose:
  rank_zero_info(f"Epoch {epoch:d}, step {step:d}: {self.monitor} was not in top {self.save_top_k}")
 
  def _is_valid_monitor_key(self, metrics):