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

[FEATURE] Jax Checkpointers #604

Closed
6 tasks
mmorris44 opened this issue Jun 24, 2022 · 0 comments · Fixed by #710
Closed
6 tasks

[FEATURE] Jax Checkpointers #604

mmorris44 opened this issue Jun 24, 2022 · 0 comments · Fixed by #710
Assignees
Labels
enhancement New feature or request
Projects

Comments

@mmorris44
Copy link
Contributor

Feature

Custom checkpointers for jax systems

Proposal

@RuanJohn found a nice easy way to do this.

Testing

Unique tests will need to be created. Checkpointers should also be tested on an example.

Definition of done

Checkpointers component is created, working in example, and tested.

Mandatory checklist before making a PR

  • The success criteria laid down in “Definition of done” are met.
  • Code is documented - docstrings for methods and classes, static types for arguments.
  • Code is tested - unit, integration and/or functional tests are added.
  • Documentation is updated - README, CONTRIBUTING, or other documentation.
  • All functional tests are green.
  • Link experiment/benchmarking after implementation (optional).
@mmorris44 mmorris44 added the enhancement New feature or request label Jun 24, 2022
@arnupretorius arnupretorius added this to To do in Mava Dev via automation Jul 8, 2022
@arnupretorius arnupretorius added this to the Jax Mava release milestone Jul 8, 2022
@DriesSmit DriesSmit assigned AsadJeewa and unassigned AsadJeewa Aug 29, 2022
Mava Dev automation moved this from To do to Done Sep 16, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
No open projects
Development

Successfully merging a pull request may close this issue.

3 participants