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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

add checkpoint_max_to_keep to train_loop #10977

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

FirdowsRahaman
Copy link

Description

馃摑 Summary of the change.

  • Sometimes we need more checkpoints to evaluate the trained model. If the user can define that on command line it'll help them to test and get a better model.
  • No dependencies that are required for this change.

Type of change

  • Other (Add checkpoint_max_to_keep argument to train_loop function)

Tests

馃摑 Please describe the tests that you ran to verify your changes.

  • Provide instructions so we can reproduce.
  • Please also list any relevant details for your test configuration.

Test Configuration:

Checklist

@google-cla
Copy link

google-cla bot commented Apr 12, 2023

Thanks for your pull request! It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).

View this failed invocation of the CLA check for more information.

For the most up to date status, view the checks section at the bottom of the pull request.

@laxmareddyp laxmareddyp added the models:research models that come under research directory label Jan 9, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
models:research models that come under research directory
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants