Skip to content

Custom checkpoint #258

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

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

Custom checkpoint #258

wants to merge 10 commits into from

Conversation

kozlov721
Copy link
Collaborator

@kozlov721 kozlov721 commented Jul 11, 2025

Purpose

Better flexibility in the future regarding backwards compatibility.

Specification

Added a few custom keys to the saved checkpoint:

  • "execution_order" - saves the execution order of modules in a model
  • "config": - the configuration
  • "dataset_metadata": Dumped DatasetMetadata containing info about number / names of classes, keypoints, etc.

Dependencies & Potential Impact

None / not applicable

Deployment Plan

None / not applicable

Testing & Validation

Added relevant tests

@kozlov721 kozlov721 requested a review from a team as a code owner July 11, 2025 00:47
@kozlov721 kozlov721 requested review from klemen1999, tersekmatija and conorsim and removed request for a team July 11, 2025 00:47
@github-actions github-actions bot added the enhancement New feature or request label Jul 11, 2025
@kozlov721
Copy link
Collaborator Author

Adds a few custom keys to the saved checkpoint:

Feel free to suggest more useful stuff that could be good to save in the checkpoints.

@klemen1999
Copy link
Collaborator

Any rough estimation of how much does this bloat the ckpt? E.g. percentage size increase of the checkpoint after this is added?

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
None yet
Development

Successfully merging this pull request may close these issues.

2 participants