Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

チェックポイントの実装 #119

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

cehl-kurage
Copy link
Contributor

概要

#103

変更内容

影響範囲

Submit前の確認項目

  • タイトルは一目でわかるようにし、説明文は PR を簡潔に説明するようにしましたか?
  • あなたの PR が、異なる変更を束ねたものではなく、ひとつのことだけを行うものであることを確認しましたか?
  • この PR で導入されたすべての変更点をリストアップしましたか?
  • PR をmake testコマンドでローカルにテストしましたか?
  • make formatコマンドで pre-commit フックを実行しましたか?

補足


def _save_checkpoint(self, trainer: pl.Trainer, filepath: str) -> None:
self.file_paths.append(filepath)
if len(self.file_paths) > self.max_file_num:
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
if len(self.file_paths) > self.max_file_num:
while len(self.file_paths) > self.max_file_num:

が良いかもしれないです

@myxyy
Copy link
Contributor

myxyy commented Oct 3, 2023

src/utils/callbacks.py ってまだどこからも呼ばれないものでしょうか?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants