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

Save and load task state to the checkpoint #5328

Merged
merged 1 commit into from
Sep 15, 2023
Merged

Conversation

piyush-kansal
Copy link
Contributor

Before submitting

  • Was this discussed/approved via a Github issue? (no need for typos, doc improvements)
  • Did you read the contributor guideline?
  • Did you make sure to update the docs?
  • Did you write any new necessary tests?

What does this PR do?

This PR adds support for the following:

  • Saving task state/attributes to the checkpoint
  • Repopulating the task state/attributes while loading the checkpoint
  • The respective unit test cases have been added to validate the new functionality

Besides, this PR also removes an unused import.

PR review

CC @cbalioglu

Did you have fun?

Make sure you had fun coding 🙃

@piyush-kansal piyush-kansal changed the title initial revision Save and load task state to the checkpoint Sep 15, 2023
@piyush-kansal piyush-kansal self-assigned this Sep 15, 2023
@piyush-kansal piyush-kansal marked this pull request as ready for review September 15, 2023 19:01
@cbalioglu cbalioglu merged commit e29f53b into main Sep 15, 2023
3 of 7 checks passed
@cbalioglu cbalioglu deleted the preserve_task_checkpoint branch September 15, 2023 19:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants