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

migrate models on service start #5889

Merged
merged 7 commits into from
Mar 8, 2024
Merged

Conversation

maryhipp
Copy link
Collaborator

@maryhipp maryhipp commented Mar 7, 2024

What type of PR is this? (check all applicable)

  • Refactor
  • Feature
  • Bug Fix
  • Optimization
  • Documentation Update
  • Community Node Submission

Have you discussed this change with the InvokeAI team?

  • Yes
  • No, because:

Have you updated all relevant documentation?

  • Yes
  • No

Description

On model manager startup, this will check if a user has 0 rows in their model table and 1+ entries in their models yaml file. If so, we will attempt to add each of their yaml entries to the models DB. This is designed to run one time, when a user first starts their app after updating to 4.0.

@github-actions github-actions bot added python PRs that change python files services PRs that change app services labels Mar 7, 2024
@maryhipp maryhipp marked this pull request as ready for review March 8, 2024 16:12
@maryhipp maryhipp force-pushed the maryhipp/migrate-models-on-start branch from 271a8c8 to fe4a90f Compare March 8, 2024 16:13
@maryhipp maryhipp changed the title POC: migrate models on service start migrate models on service start Mar 8, 2024
@maryhipp maryhipp force-pushed the maryhipp/migrate-models-on-start branch from 65ed373 to c808e79 Compare March 8, 2024 17:51
@maryhipp maryhipp enabled auto-merge (rebase) March 8, 2024 17:52
@maryhipp maryhipp merged commit d567415 into main Mar 8, 2024
14 checks passed
@maryhipp maryhipp deleted the maryhipp/migrate-models-on-start branch March 8, 2024 18:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
python PRs that change python files services PRs that change app services
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants