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

Separate artifacts from wrappers #193

Merged
merged 16 commits into from
Jul 15, 2023
Merged

Conversation

Oxid15
Copy link
Owner

@Oxid15 Oxid15 commented Jul 15, 2023

  • Adds save_artifact/load_artifact method to each model to save standalone artifacts separately from cascade wrappers
  • Model's save method now accepts only folders
  • Regular save methods now save only wrappers, the same with load
  • Adds ModelLine.load method that allows loading models with and without artifacts
  • getitem now loads only wrappers
  • Line saves models in the folders and to save artifacts creates artifacts folder inside of them
  • Trainer was also updated accordingly

Important part of it is that those changes are breaking previous repo structure and loading models from old repos seems to be impossible since 0.13.0

@Oxid15 Oxid15 merged commit 413b4d1 into 0.13.0 Jul 15, 2023
@Oxid15 Oxid15 deleted the separate_artifacts_from_wrappers branch July 15, 2023 20:33
@Oxid15 Oxid15 mentioned this pull request Nov 6, 2023
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.

1 participant